Unity 8

Merge lp://staging/~lukas-kde/unity8/relaxedThreeFingerDrag into lp://staging/unity8

  1. relaxedThreeFingerDrag
  2. Merge into trunk
Proposed by Lukáš Tinkl
Status: Needs review
Proposed branch: lp://staging/~lukas-kde/unity8/relaxedThreeFingerDrag
Merge into: lp://staging/unity8
Diff against target: 38 lines (+5/-5)
2 files modified
plugins/Ubuntu/Gestures/TouchGestureArea.h (+2/-2)
qml/Stages/WindowControlsOverlay.qml (+3/-3)
To merge this branch: bzr merge lp://staging/~lukas-kde/unity8/relaxedThreeFingerDrag
Reviewer Review Type Date Requested Status
Unity8 CI Bot continuous-integration Needs Fixing
Michael Terry Needs Fixing
Review via email: mp+298534@code.staging.launchpad.net

Commit message

More relaxed 3-finger drag controls

Description of the change

More relaxed 3-finger drag controls

We allow 1..max fingers being used for the drag operation, as long as it's underway. With the last finger lifted, the drag is ended. See the attached BR for more details.

* Are there any related MPs required for this MP to build/function as expected? Please list.

No

* Did you perform an exploratory manual test run of your code change and any related functionality?

Yes

* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?

N/A

* If you changed the UI, has there been a design review?

N/A

To post a comment you must log in.
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

PASSED: Continuous integration, rev:2498
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1651/
Executed test runs:

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1651/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
Michael Terry (mterry) wrote :

Good catch on operator=.

Looks like CI was taking a vacation there. I've restarted it.

As for the actual 3-finger change, it does work for me. In the sense that I can continue the drag motion with less fingers.

But it doesn't feel right. Lifting some fingers makes the window jump around (looks like it is "rebasing" on the remaining finger). And putting the fingers back also makes it jump around.

Easy to do accidentally and looks jarring. Can we smooth that out somehow? Prevent rebasing the drag focus?

review: Needs Fixing
Reply
Revision history for this message
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2498
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1655/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/2196/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/2224
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=vivid+overlay/2130
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=xenial+overlay/2130
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-1-sourcepkg/release=yakkety/2130
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=vivid+overlay/2121/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2121
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=xenial+overlay/2121/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=amd64,release=yakkety/2121/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2121
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=vivid+overlay/2121/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2121
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=xenial+overlay/2121/artifact/output/*zip*/output.zip
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2121
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=armhf,release=yakkety/2121/artifact/output/*zip*/output.zip
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=vivid+overlay/2121/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=xenial+overlay/2121/console
    SUCCESS: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2121
        deb: https://unity8-jenkins.ubuntu.com/job/build-2-binpkg/arch=i386,release=yakkety/2121/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1655/rebuild

review: Needs Fixing (continuous-integration)
Reply

Unmerged revisions

2498. By Lukáš Tinkl

reduce diff

2497. By Lukáš Tinkl

WIP: more relaxed drag handling

2496. By Lukáš Tinkl

fix operator== and operator!=

2495. By Launchpad Translations on behalf of unity-team

Launchpad automatic translations update.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches