Merge lp:~paulliu/unity8/noninteractive20140804 into lp:unity8
- noninteractive20140804
- Merge into trunk
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Albert Astals Cid | ||||
Approved revision: | 1129 | ||||
Merged at revision: | 1387 | ||||
Proposed branch: | lp:~paulliu/unity8/noninteractive20140804 | ||||
Merge into: | lp:unity8 | ||||
Diff against target: |
158 lines (+51/-2) 9 files modified
plugins/Dash/CardCreator.js (+3/-1) tests/plugins/Dash/cardcreator/1.res (+2/-0) tests/plugins/Dash/cardcreator/2.res (+2/-0) tests/plugins/Dash/cardcreator/3.res (+2/-0) tests/plugins/Dash/cardcreator/4.res (+2/-0) tests/plugins/Dash/cardcreator/5.res (+3/-1) tests/plugins/Dash/cardcreator/6.res (+2/-0) tests/plugins/Dash/cardcreator/7.res (+2/-0) tests/qmltests/Dash/tst_GenericScopeView.qml (+33/-0) |
||||
To merge this branch: | bzr merge lp:~paulliu/unity8/noninteractive20140804 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Needs Fixing | |
Albert Astals Cid (community) | Approve | ||
Review via email: mp+231746@code.launchpad.net |
Commit message
Add non-interactive code into GenericScopeView.
Description of the change
* Are there any related MPs required for this MP to build/function as expected?
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
- 1118. By Ying-Chun Liu
-
Merge trunk.
PS Jenkins bot (ps-jenkins) wrote : | # |
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1118
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Albert Astals Cid (aacid) wrote : | # |
What's this for? Can you please link at the spec/use for this flag?
Albert Astals Cid (aacid) wrote : | # |
Text conflict in qml/Dash/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
4 conflicts encountered.
- 1119. By Ying-Chun Liu
-
merge trunk
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1119
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/plugins/
Text conflict in tests/qmltests/
8 conflicts encountered.
Albert Astals Cid (aacid) wrote : | # |
- 1120. By Ying-Chun Liu
-
merge trunk
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1120
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1120
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
Albert Astals Cid (aacid) wrote : | # |
Paul, there's something weird in your tests/plugins/
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:1120
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1121. By Ying-Chun Liu
-
Fix results
- 1122. By Ying-Chun Liu
-
remove empty lines
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1122
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Albert Astals Cid (aacid) wrote : | # |
Wouldn't it be easier to just set the enabled property of the button to false?
- 1123. By Ying-Chun Liu
-
merge trunk
- 1124. By Ying-Chun Liu
-
Use enabled property
- 1125. By Ying-Chun Liu
-
Fix card result
Ying-Chun Liu (paulliu) wrote : | # |
Seems using enabled property of the AbstractButton not working as expected.
Inside the AbstractButton, there's MouseArea. And it only looks private __acceptEvents for accepting or not. Not the enabled property inherit from ActionItem.
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1125
http://
Executed test runs:
FAILURE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Albert Astals Cid (aacid) wrote : | # |
> Seems using enabled property of the AbstractButton not working as expected.
>
> Inside the AbstractButton, there's MouseArea. And it only looks private
> __acceptEvents for accepting or not. Not the enabled property inherit from
> ActionItem.
That's strange i just did an AbstractButton and set it to disabled and it's internal mouse area is also disabled.
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1125
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1126. By Ying-Chun Liu
-
Fix unit tests
- 1127. By Ying-Chun Liu
-
Merge trunk
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1127
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Albert Astals Cid (aacid) wrote : | # |
Do we need the change for "visible:" in CardCreator? Since the card is never enabled root.pressed should never be true, so i think we don't really need that change, no?
- 1128. By Ying-Chun Liu
-
Don't modify touchdown visible
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1128
http://
Executed test runs:
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1129. By Ying-Chun Liu
-
Save objectName for rendererLoader.
Albert Astals Cid (aacid) : | # |
Albert Astals Cid (aacid) 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, unrelated
* Did you make sure that the branch does not contain spurious tags?
Yes
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1128
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1130. By Ying-Chun Liu
-
Merge trunk
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1128
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Preview Diff
1 | === modified file 'plugins/Dash/CardCreator.js' | |||
2 | --- plugins/Dash/CardCreator.js 2014-10-10 11:13:26 +0000 | |||
3 | +++ plugins/Dash/CardCreator.js 2014-10-27 13:45:30 +0000 | |||
4 | @@ -313,7 +313,9 @@ | |||
5 | 313 | readonly property string title: cardData && cardData["title"] || ""; \n\ | 313 | readonly property string title: cardData && cardData["title"] || ""; \n\ |
6 | 314 | property bool asynchronous: true; \n\ | 314 | property bool asynchronous: true; \n\ |
7 | 315 | property bool showHeader: true; \n\ | 315 | property bool showHeader: true; \n\ |
9 | 316 | implicitWidth: childrenRect.width; \n'; | 316 | implicitWidth: childrenRect.width; \n\ |
10 | 317 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); \n\ | ||
11 | 318 | \n'; | ||
12 | 317 | 319 | ||
13 | 318 | var hasArt = components["art"] && components["art"]["field"] || false; | 320 | var hasArt = components["art"] && components["art"]["field"] || false; |
14 | 319 | var hasSummary = components["summary"] || false; | 321 | var hasSummary = components["summary"] || false; |
15 | 320 | 322 | ||
16 | === modified file 'tests/plugins/Dash/cardcreator/1.res' | |||
17 | --- tests/plugins/Dash/cardcreator/1.res 2014-10-06 08:01:08 +0000 | |||
18 | +++ tests/plugins/Dash/cardcreator/1.res 2014-10-27 13:45:30 +0000 | |||
19 | @@ -13,6 +13,8 @@ | |||
20 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
21 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
22 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
23 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
24 | 17 | |||
25 | 16 | onArtShapeBorderSourceChanged: { if (artShapeBorderSource !== undefined && artShapeLoader.item) artShapeLoader.item.borderSource = artShapeBorderSource; } | 18 | onArtShapeBorderSourceChanged: { if (artShapeBorderSource !== undefined && artShapeLoader.item) artShapeLoader.item.borderSource = artShapeBorderSource; } |
26 | 17 | readonly property size artShapeSize: artShapeLoader.item ? Qt.size(artShapeLoader.item.width, artShapeLoader.item.height) : Qt.size(-1, -1); | 19 | readonly property size artShapeSize: artShapeLoader.item ? Qt.size(artShapeLoader.item.width, artShapeLoader.item.height) : Qt.size(-1, -1); |
27 | 18 | Item { | 20 | Item { |
28 | 19 | 21 | ||
29 | === modified file 'tests/plugins/Dash/cardcreator/2.res' | |||
30 | --- tests/plugins/Dash/cardcreator/2.res 2014-10-10 11:13:26 +0000 | |||
31 | +++ tests/plugins/Dash/cardcreator/2.res 2014-10-27 13:45:30 +0000 | |||
32 | @@ -13,6 +13,8 @@ | |||
33 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
34 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
35 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
36 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
37 | 17 | |||
38 | 16 | Loader { | 18 | Loader { |
39 | 17 | id: backgroundLoader; | 19 | id: backgroundLoader; |
40 | 18 | objectName: "backgroundLoader"; | 20 | objectName: "backgroundLoader"; |
41 | 19 | 21 | ||
42 | === modified file 'tests/plugins/Dash/cardcreator/3.res' | |||
43 | --- tests/plugins/Dash/cardcreator/3.res 2014-10-06 08:01:08 +0000 | |||
44 | +++ tests/plugins/Dash/cardcreator/3.res 2014-10-27 13:45:30 +0000 | |||
45 | @@ -13,6 +13,8 @@ | |||
46 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
47 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
48 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
49 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
50 | 17 | |||
51 | 16 | onArtShapeBorderSourceChanged: { if (artShapeBorderSource !== undefined && artShapeLoader.item) artShapeLoader.item.borderSource = artShapeBorderSource; } | 18 | onArtShapeBorderSourceChanged: { if (artShapeBorderSource !== undefined && artShapeLoader.item) artShapeLoader.item.borderSource = artShapeBorderSource; } |
52 | 17 | readonly property size artShapeSize: artShapeLoader.item ? Qt.size(artShapeLoader.item.width, artShapeLoader.item.height) : Qt.size(-1, -1); | 19 | readonly property size artShapeSize: artShapeLoader.item ? Qt.size(artShapeLoader.item.width, artShapeLoader.item.height) : Qt.size(-1, -1); |
53 | 18 | Item { | 20 | Item { |
54 | 19 | 21 | ||
55 | === modified file 'tests/plugins/Dash/cardcreator/4.res' | |||
56 | --- tests/plugins/Dash/cardcreator/4.res 2014-10-06 08:01:08 +0000 | |||
57 | +++ tests/plugins/Dash/cardcreator/4.res 2014-10-27 13:45:30 +0000 | |||
58 | @@ -13,6 +13,8 @@ | |||
59 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
60 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
61 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
62 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
63 | 17 | |||
64 | 16 | readonly property size artShapeSize: Qt.size(-1, -1); | 18 | readonly property size artShapeSize: Qt.size(-1, -1); |
65 | 17 | readonly property int headerHeight: row.height; | 19 | readonly property int headerHeight: row.height; |
66 | 18 | Row { | 20 | Row { |
67 | 19 | 21 | ||
68 | === modified file 'tests/plugins/Dash/cardcreator/5.res' | |||
69 | --- tests/plugins/Dash/cardcreator/5.res 2014-10-10 11:13:26 +0000 | |||
70 | +++ tests/plugins/Dash/cardcreator/5.res 2014-10-27 13:45:30 +0000 | |||
71 | @@ -13,6 +13,8 @@ | |||
72 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
73 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
74 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
75 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
76 | 17 | |||
77 | 16 | onArtShapeBorderSourceChanged: { if (artShapeBorderSource !== undefined && artShapeLoader.item) artShapeLoader.item.borderSource = artShapeBorderSource; } | 18 | onArtShapeBorderSourceChanged: { if (artShapeBorderSource !== undefined && artShapeLoader.item) artShapeLoader.item.borderSource = artShapeBorderSource; } |
78 | 17 | readonly property size artShapeSize: artShapeLoader.item ? Qt.size(artShapeLoader.item.width, artShapeLoader.item.height) : Qt.size(-1, -1); | 19 | readonly property size artShapeSize: artShapeLoader.item ? Qt.size(artShapeLoader.item.width, artShapeLoader.item.height) : Qt.size(-1, -1); |
79 | 18 | Item { | 20 | Item { |
80 | @@ -143,7 +145,7 @@ | |||
81 | 143 | id: touchdown; | 145 | id: touchdown; |
82 | 144 | objectName: "touchdown"; | 146 | objectName: "touchdown"; |
83 | 145 | anchors { fill: artShapeHolder } | 147 | anchors { fill: artShapeHolder } |
85 | 146 | visible: root.pressed; | 148 | visible: root.pressed; |
86 | 147 | radius: "medium"; | 149 | radius: "medium"; |
87 | 148 | borderSource: "radius_pressed.sci" | 150 | borderSource: "radius_pressed.sci" |
88 | 149 | } | 151 | } |
89 | 150 | 152 | ||
90 | === modified file 'tests/plugins/Dash/cardcreator/6.res' | |||
91 | --- tests/plugins/Dash/cardcreator/6.res 2014-09-29 08:19:06 +0000 | |||
92 | +++ tests/plugins/Dash/cardcreator/6.res 2014-10-27 13:45:30 +0000 | |||
93 | @@ -13,6 +13,8 @@ | |||
94 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
95 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
96 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
97 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
98 | 17 | |||
99 | 16 | Loader { | 18 | Loader { |
100 | 17 | id: backgroundLoader; | 19 | id: backgroundLoader; |
101 | 18 | objectName: "backgroundLoader"; | 20 | objectName: "backgroundLoader"; |
102 | 19 | 21 | ||
103 | === modified file 'tests/plugins/Dash/cardcreator/7.res' | |||
104 | --- tests/plugins/Dash/cardcreator/7.res 2014-10-10 11:13:26 +0000 | |||
105 | +++ tests/plugins/Dash/cardcreator/7.res 2014-10-27 13:45:30 +0000 | |||
106 | @@ -13,6 +13,8 @@ | |||
107 | 13 | property bool asynchronous: true; | 13 | property bool asynchronous: true; |
108 | 14 | property bool showHeader: true; | 14 | property bool showHeader: true; |
109 | 15 | implicitWidth: childrenRect.width; | 15 | implicitWidth: childrenRect.width; |
110 | 16 | enabled: root.template == null ? true : (root.template["non-interactive"] !== undefined ? !root.template["non-interactive"] : true); | ||
111 | 17 | |||
112 | 16 | Loader { | 18 | Loader { |
113 | 17 | id: backgroundLoader; | 19 | id: backgroundLoader; |
114 | 18 | objectName: "backgroundLoader"; | 20 | objectName: "backgroundLoader"; |
115 | 19 | 21 | ||
116 | === modified file 'tests/qmltests/Dash/tst_GenericScopeView.qml' | |||
117 | --- tests/qmltests/Dash/tst_GenericScopeView.qml 2014-10-10 11:13:26 +0000 | |||
118 | +++ tests/qmltests/Dash/tst_GenericScopeView.qml 2014-10-27 13:45:30 +0000 | |||
119 | @@ -603,6 +603,39 @@ | |||
120 | 603 | spy.wait() | 603 | spy.wait() |
121 | 604 | compare(spy.count, 1) | 604 | compare(spy.count, 1) |
122 | 605 | } | 605 | } |
123 | 606 | |||
124 | 607 | function test_item_noninteractive() { | ||
125 | 608 | waitForRendering(genericScopeView); | ||
126 | 609 | |||
127 | 610 | var categoryListView = findChild(genericScopeView, "categoryListView"); | ||
128 | 611 | waitForRendering(categoryListView); | ||
129 | 612 | |||
130 | 613 | var category0 = findChild(categoryListView, "dashCategory0"); | ||
131 | 614 | waitForRendering(category0); | ||
132 | 615 | |||
133 | 616 | var cardTool = findChild(category0, "cardTool"); | ||
134 | 617 | var cardGrid = category0.item; | ||
135 | 618 | |||
136 | 619 | cardTool.template["non-interactive"] = true; | ||
137 | 620 | compare(cardGrid.cardTool.template["non-interactive"], true); | ||
138 | 621 | |||
139 | 622 | var item0 = findChild(cardGrid, "delegate0"); | ||
140 | 623 | waitForRendering(item0); | ||
141 | 624 | item0.template = cardTool.template; | ||
142 | 625 | compare(item0.template["non-interactive"], true); | ||
143 | 626 | compare(item0.enabled, false); | ||
144 | 627 | var touchdown = findChild(item0, "touchdown"); | ||
145 | 628 | |||
146 | 629 | compare(touchdown.visible, false); | ||
147 | 630 | mouseClick(item0, item0.width / 2, item0.height / 2); | ||
148 | 631 | compare(touchdown.visible, false); | ||
149 | 632 | |||
150 | 633 | cardTool.template["non-interactive"] = false; | ||
151 | 634 | compare(cardGrid.cardTool.template["non-interactive"], false); | ||
152 | 635 | item0.template = cardTool.template; | ||
153 | 636 | compare(item0.template["non-interactive"], false); | ||
154 | 637 | compare(item0.enabled, true); | ||
155 | 638 | } | ||
156 | 606 | } | 639 | } |
157 | 607 | } | 640 | } |
158 | 608 | } | 641 | } |
FAILED: Continuous integration, rev:1117 jenkins. qa.ubuntu. com/job/ unity8- ci/4279/ jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- utopic- touch/4730/ console jenkins. qa.ubuntu. com/job/ unity-phablet- qmluitests- utopic/ 1280 jenkins. qa.ubuntu. com/job/ unity8- utopic- amd64-ci/ 1373 jenkins. qa.ubuntu. com/job/ unity8- utopic- i386-ci/ 1373 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- runner- mako/4500/ console jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/5982 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- utopic- armhf/5982/ artifact/ work/output/ *zip*/output. zip s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 13071
http://
Executed test runs:
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/unity8- ci/4279/ rebuild
http://