| 2016-10-08 17:25:23 |
Stefano Verzegnassi |
description |
Tested on Yakkety, inside a Unity8 environment.
ubuntu-terminal-app version: the one from the repos.
Related to bug #1631628
Long story short:
The merge proposal[1] from the revision 210, "Update QMLTermWidget to latest version and remove duplicate konsole application code.", actually replaced our version of the QMLTermWidget with the fork done by the guys from PapyrOS.
It has been approved without checking/testing the effective changes of the MP.
Some months ago I've talked with the guys from the Mir team. The problem was that ubuntu-terminal-app used to create a new Mir surface during text selection, since the drag&drop feature of QMLTermWidget was enabled.
At the time Mir/QtMir wasn't able to handle more than one surface for a single application, so we all agreed to disable the drag&drop gesture.
Now that our QMLTermWidget has been replaced with the fork I mentioned above, and Mir can handle more than one surface per app, we anyway get something terrible.
See attached screenshot.
The new surface stay visible on screen even if the drag&drop has been completed. Also, copy-pasting a string through a drag&drop gesture doesn't work anyway.
Link to the Mir patch that has been removed:
http://bazaar.launchpad.net/~ubuntu-terminal-dev/ubuntu-terminal-app/reboot/revision/194 |
Tested on Yakkety, inside a Unity8 environment.
ubuntu-terminal-app version: the one from the repos.
Related to bug #1631628
Long story short:
The merge proposal[1] from the revision 210, "Update QMLTermWidget to latest version and remove duplicate konsole application code.", actually replaced our version of the QMLTermWidget with the fork done by the guys from PapyrOS.
It has been approved without checking/testing the effective changes of the MP.
Some months ago I've talked with the guys from the Mir team. The problem was that ubuntu-terminal-app used to create a new Mir surface during text selection, since the drag&drop feature of QMLTermWidget was enabled.
At the time Mir/QtMir wasn't able to handle more than one surface for a single application, so we all agreed to disable the drag&drop gesture.
Now that our QMLTermWidget has been replaced with the fork I mentioned above, and Mir can handle more than one surface per app, we anyway get something terrible.
See attached screenshot.
The new surface stay visible on screen even if the drag&drop has been completed. Also, copy-pasting a string through a drag&drop gesture doesn't work anyway.
Link to the Mir patch that has been removed:
http://bazaar.launchpad.net/~ubuntu-terminal-dev/ubuntu-terminal-app/reboot/revision/194
[1] https://code.launchpad.net/~larryprice/ubuntu-terminal-app/update-qmltermwidget |
|
