Merge lp:~mzanetti/unity8/drop-ignoredMice into lp:unity8

Proposed by Michael Zanetti on 2015-12-01
Status: Merged
Approved by: Daniel d'Andrada on 2015-12-01
Approved revision: 2076
Merged at revision: 2090
Proposed branch: lp:~mzanetti/unity8/drop-ignoredMice
Merge into: lp:unity8
Diff against target: 51 lines (+2/-7)
2 files modified
qml/DeviceConfiguration.qml (+0/-5)
qml/OrientedShell.qml (+2/-2)
To merge this branch: bzr merge lp:~mzanetti/unity8/drop-ignoredMice
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing on 2015-12-01
Daniel d'Andrada (community) 2015-12-01 Approve on 2015-12-01
Review via email: mp+279121@code.launchpad.net

Commit Message

drop the ignoredMice hack again

the new inputinfo backend seems to be clever enough to not
wrongly detect the hardware in there as real a real mouse

Description of the Change

 * Are there any related MPs required for this MP to build/function as expected? Please list.
n
 * Did you perform an exploratory manual test run of your code change and any related functionality?
y
 * Did you make sure that your branch does not contain spurious tags?
y
 * 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.
Daniel d'Andrada (dandrader) wrote :

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

* Did CI run pass? If not, please explain why.
No results yet.

* Did you make sure that the branch does not contain spurious tags?
Yes.

review: Approve
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:2076
http://jenkins.qa.ubuntu.com/job/unity8-ci/6856/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/5526
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/271/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1567
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/270/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1462
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1462
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/269
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/268
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4328
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/5540
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/5540/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25669
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-xenial-touch/94/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/270
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/270/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/25668

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/6856/rebuild

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qml/DeviceConfiguration.qml'
2--- qml/DeviceConfiguration.qml 2015-11-04 14:57:45 +0000
3+++ qml/DeviceConfiguration.qml 2015-12-01 14:01:21 +0000
4@@ -34,8 +34,6 @@
5
6 readonly property alias category: priv.category
7
8- readonly property alias ignoredMice: priv.ignoredMice
9-
10 readonly property var priv: StateGroup {
11 id: priv
12
13@@ -55,8 +53,6 @@
14 // "phone", "tablet" or "desktop"
15 property string category: "phone"
16
17- property int ignoredMice: 0
18-
19 states: [
20 State {
21 name: "mako"
22@@ -83,7 +79,6 @@
23 supportedOrientations: Qt.PortraitOrientation
24 | Qt.LandscapeOrientation
25 | Qt.InvertedLandscapeOrientation
26- ignoredMice: 1
27 }
28 },
29 State {
30
31=== modified file 'qml/OrientedShell.qml'
32--- qml/OrientedShell.qml 2015-11-20 18:21:02 +0000
33+++ qml/OrientedShell.qml 2015-12-01 14:01:21 +0000
34@@ -188,7 +188,7 @@
35 nativeWidth: root.width
36 nativeHeight: root.height
37 mode: applicationArguments.mode
38- hasMouse: miceModel.count + touchPadModel.count > deviceConfiguration.ignoredMice
39+ hasMouse: miceModel.count + touchPadModel.count > 0
40
41 // TODO: Factor in the connected input devices (eg: physical keyboard, mouse, touchscreen),
42 // what's the output device (eg: big TV, desktop monitor, phone display), etc.
43@@ -202,7 +202,7 @@
44 return "tablet";
45 }
46 } else { // automatic
47- if (miceModel.count + touchPadModel.count > deviceConfiguration.ignoredMice) {
48+ if (miceModel.count + touchPadModel.count > 0) {
49 return "desktop";
50 } else {
51 return deviceConfiguration.category;

Subscribers

People subscribed via source and target branches