Merge lp:~saviq/unity8/dash-activity-indicator into lp:unity8

Proposed by Michał Sawicz
Status: Merged
Approved by: Andrea Cimitan
Approved revision: 1087
Merged at revision: 1131
Proposed branch: lp:~saviq/unity8/dash-activity-indicator
Merge into: lp:unity8
Prerequisite: lp:~unity-team/unity8/scopes-v4
Diff against target: 473 lines (+150/-98)
12 files modified
plugins/Dash/ScopeStyle.qml (+11/-8)
qml/Components/PageHeader.qml (+1/-14)
qml/Dash/Dash.qml (+68/-1)
qml/Dash/DashContent.qml (+2/-0)
qml/Dash/DashNavigationList.qml (+0/-5)
qml/Dash/GenericScopeView.qml (+2/-1)
qml/Dash/PreviewListView.qml (+27/-31)
tests/autopilot/unity8/shell/emulators/dash.py (+1/-1)
tests/plugins/Dash/tst_ScopeStyle.qml (+14/-7)
tests/qmltests/Components/tst_PageHeader.qml (+0/-23)
tests/qmltests/Dash/tst_Dash.qml (+23/-6)
tests/qmltests/Dash/tst_GenericScopeView.qml (+1/-1)
To merge this branch: bzr merge lp:~saviq/unity8/dash-activity-indicator
Reviewer Review Type Date Requested Status
Andrea Cimitan (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Albert Astals Cid Pending
Review via email: mp+229805@code.launchpad.net

This proposal supersedes a proposal from 2014-07-29.

Commit message

Refactor dash activity indicator.

Description of the change

 * 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.

 * Did you make sure that your branch does not contain spurious tags?
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?
Yes, there was. And will be again.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Andrea Cimitan (cimi) wrote : Posted in a previous version of this proposal

See inline

review: Needs Fixing
Revision history for this message
Michał Sawicz (saviq) : Posted in a previous version of this proposal
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote : Posted in a previous version of this proposal

Text conflict in tests/qmltests/Dash/tst_Dash.qml
1 conflicts encountered.

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Albert Astals Cid (aacid) wrote : Posted in a previous version of this proposal

There are autopilot failures related to autopilot, can you have a look?

review: Needs Fixing
Revision history for this message
Albert Astals Cid (aacid) wrote : Posted in a previous version of this proposal

err, autopilot failures related to indicators i mean :D

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Andrea Cimitan (cimi) wrote :

Apart seeing if CI will pass, I tested locally twice and works fine

 * 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.
Waiting CI

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/Dash/ScopeStyle.qml'
--- plugins/Dash/ScopeStyle.qml 2014-07-18 18:09:13 +0000
+++ plugins/Dash/ScopeStyle.qml 2014-08-06 14:50:51 +0000
@@ -38,14 +38,20 @@
38 /// Color used for text and symbolic icons38 /// Color used for text and symbolic icons
39 readonly property color foreground: "foreground-color" in style ? style["foreground-color"] : d.defaultDark39 readonly property color foreground: "foreground-color" in style ? style["foreground-color"] : d.defaultDark
4040
41 /// Luminance of the foreground color
42 readonly property real foregroundLuminance: foreground ? luminance(foreground) : d.defaultForegroundLuminance
43
41 /// Color used for the overall background44 /// Color used for the overall background
42 readonly property color background: "background-color" in style ? style["background-color"] : "transparent"45 readonly property color background: "background-color" in style ? style["background-color"] : "transparent"
4346
47 /// Luminance of the foreground color
48 readonly property real backgroundLuminance: background ? luminance(background) : d.defaultBackgroundLuminance
49
44 /*! \brief Luminance threshold for switching between fore and background color50 /*! \brief Luminance threshold for switching between fore and background color
4551
46 \note If background colour is not fully opaque, it's not taken into account.52 \note If background colour is not fully opaque, it's not taken into account.
47 */53 */
48 readonly property real threshold: background.a !== 1.0 ? d.foregroundLuminance : (d.foregroundLuminance + d.backgroundLuminance) / 254 readonly property real threshold: background.a !== 1.0 ? foregroundLuminance : (foregroundLuminance + backgroundLuminance) / 2
4955
50 /*! \brief The lighter of foreground and background colors56 /*! \brief The lighter of foreground and background colors
5157
@@ -53,8 +59,8 @@
53 and defaults to the theme-provided light color.59 and defaults to the theme-provided light color.
54 */60 */
55 readonly property color light: {61 readonly property color light: {
56 if (background.a !== 1.0) return d.foregroundLuminance > d.defaultLightLuminance ? foreground : d.defaultLight;62 if (background.a !== 1.0) return foregroundLuminance > d.defaultLightLuminance ? foreground : d.defaultLight;
57 return d.foregroundLuminance > d.backgroundLuminance ? foreground : background;63 return foregroundLuminance > backgroundLuminance ? foreground : background;
58 }64 }
5965
60 /*! \brief The darker of foreground and background colors66 /*! \brief The darker of foreground and background colors
@@ -63,8 +69,8 @@
63 and defaults to the theme-provided dark color.69 and defaults to the theme-provided dark color.
64 */70 */
65 readonly property color dark: {71 readonly property color dark: {
66 if (background.a !== 1.0) return d.foregroundLuminance < d.defaultDarkLuminance ? foreground : d.defaultDark;72 if (background.a !== 1.0) return foregroundLuminance < d.defaultDarkLuminance ? foreground : d.defaultDark;
67 return d.foregroundLuminance < d.backgroundLuminance ? foreground : background;73 return foregroundLuminance < backgroundLuminance ? foreground : background;
68 }74 }
6975
70 /// Source of the logo image for the header76 /// Source of the logo image for the header
@@ -78,9 +84,6 @@
7884
79 //! @cond85 //! @cond
80 property var d: QtObject {86 property var d: QtObject {
81 readonly property real foregroundLuminance: luminance(foreground)
82 readonly property real backgroundLuminance: luminance(background)
83
84 // FIXME: should be taken from the theme87 // FIXME: should be taken from the theme
85 readonly property color defaultLight: "white"88 readonly property color defaultLight: "white"
86 readonly property color defaultDark: "grey"89 readonly property color defaultDark: "grey"
8790
=== modified file 'qml/Components/PageHeader.qml'
--- qml/Components/PageHeader.qml 2014-07-31 16:49:50 +0000
+++ qml/Components/PageHeader.qml 2014-08-06 14:50:51 +0000
@@ -32,7 +32,6 @@
32 property bool searchEntryEnabled: false32 property bool searchEntryEnabled: false
33 property ListModel searchHistory: SearchHistoryModel33 property ListModel searchHistory: SearchHistoryModel
34 property alias searchQuery: searchTextField.text34 property alias searchQuery: searchTextField.text
35 property bool searchInProgress: false
3635
37 property alias bottomItem: bottomContainer.children36 property alias bottomItem: bottomContainer.children
3837
@@ -200,25 +199,13 @@
200 anchors.fill: parent199 anchors.fill: parent
201 anchors.margins: units.gu(.75)200 anchors.margins: units.gu(.75)
202 source: "image://theme/clear"201 source: "image://theme/clear"
203 opacity: searchTextField.text.length > 0 && !searchActivityIndicator.running202 opacity: searchTextField.text.length > 0
204 visible: opacity > 0203 visible: opacity > 0
205 Behavior on opacity {204 Behavior on opacity {
206 UbuntuNumberAnimation { duration: UbuntuAnimation.FastDuration }205 UbuntuNumberAnimation { duration: UbuntuAnimation.FastDuration }
207 }206 }
208 }207 }
209208
210 ActivityIndicator {
211 id: searchActivityIndicator
212 objectName: "searchIndicator"
213 anchors.fill: parent
214 anchors.margins: units.gu(.75)
215 running: root.searchInProgress
216 opacity: running ? 1 : 0
217 Behavior on opacity {
218 UbuntuNumberAnimation { duration: UbuntuAnimation.FastDuration }
219 }
220 }
221
222 onClicked: {209 onClicked: {
223 root.resetSearch(true);210 root.resetSearch(true);
224 root.openSearchHistory();211 root.openSearchHistory();
225212
=== modified file 'qml/Dash/Dash.qml'
--- qml/Dash/Dash.qml 2014-08-06 14:50:51 +0000
+++ qml/Dash/Dash.qml 2014-08-06 14:50:51 +0000
@@ -14,7 +14,7 @@
14 * along with this program. If not, see <http://www.gnu.org/licenses/>.14 * along with this program. If not, see <http://www.gnu.org/licenses/>.
15 */15 */
1616
17import QtQuick 2.017import QtQuick 2.2
18import Ubuntu.Components 0.118import Ubuntu.Components 0.1
19import Unity 0.219import Unity 0.2
20import Utils 0.120import Utils 0.1
@@ -132,4 +132,71 @@
132 }132 }
133 }133 }
134 }134 }
135
136 Rectangle {
137 id: indicator
138 objectName: "processingIndicator"
139 anchors {
140 left: parent.left
141 right: parent.right
142 bottom: parent.bottom
143 }
144 height: units.dp(3)
145 color: scopeStyle.backgroundLuminance > 0.7 ? "#50000000" : "#50ffffff"
146 opacity: 0
147 visible: opacity > 0
148
149 readonly property bool processing: dashContent.processing || scopeItem.processing
150
151 Behavior on opacity {
152 UbuntuNumberAnimation { duration: UbuntuAnimation.FastDuration }
153 }
154
155 onProcessingChanged: {
156 if (processing) delay.start();
157 else if (!persist.running) indicator.opacity = 0;
158 }
159
160 Timer {
161 id: delay
162 interval: 200
163 onTriggered: if (indicator.processing) {
164 persist.restart();
165 indicator.opacity = 1;
166 }
167 }
168
169 Timer {
170 id: persist
171 interval: 2 * UbuntuAnimation.SleepyDuration - UbuntuAnimation.FastDuration
172 onTriggered: if (!indicator.processing) indicator.opacity = 0
173 }
174
175 Rectangle {
176 id: orange
177 anchors { top: parent.top; bottom: parent.bottom }
178 width: parent.width / 4
179 color: Theme.palette.selected.foreground
180
181 SequentialAnimation {
182 running: indicator.visible
183 loops: Animation.Infinite
184 XAnimator {
185 from: -orange.width / 2
186 to: indicator.width - orange.width / 2
187 duration: UbuntuAnimation.SleepyDuration
188 easing.type: Easing.InOutSine
189 target: orange
190 }
191 XAnimator {
192 from: indicator.width - orange.width / 2
193 to: -orange.width / 2
194 duration: UbuntuAnimation.SleepyDuration
195 easing.type: Easing.InOutSine
196 target: orange
197 }
198 }
199 }
200 }
201
135}202}
136203
=== modified file 'qml/Dash/DashContent.qml'
--- qml/Dash/DashContent.qml 2014-08-06 14:50:51 +0000
+++ qml/Dash/DashContent.qml 2014-08-06 14:50:51 +0000
@@ -25,6 +25,8 @@
2525
26 property alias scopes: dashContentList.model26 property alias scopes: dashContentList.model
27 readonly property alias currentIndex: dashContentList.currentIndex27 readonly property alias currentIndex: dashContentList.currentIndex
28 readonly property bool processing: dashContentList.currentItem && dashContentList.currentItem.item
29 && dashContentList.currentItem.item.processing || false
2830
29 signal scopeLoaded(string scopeId)31 signal scopeLoaded(string scopeId)
30 signal gotoScope(string scopeId)32 signal gotoScope(string scopeId)
3133
=== modified file 'qml/Dash/DashNavigationList.qml'
--- qml/Dash/DashNavigationList.qml 2014-08-06 14:50:51 +0000
+++ qml/Dash/DashNavigationList.qml 2014-08-06 14:50:51 +0000
@@ -35,11 +35,6 @@
35 anchors.fill: parent35 anchors.fill: parent
36 }36 }
3737
38 ActivityIndicator {
39 id: loadingIndicator
40 anchors.centerIn: parent
41 running: !(navigation && navigation.loaded)
42 }
43 clip: true38 clip: true
4439
45 Behavior on height {40 Behavior on height {
4641
=== modified file 'qml/Dash/GenericScopeView.qml'
--- qml/Dash/GenericScopeView.qml 2014-08-06 14:50:51 +0000
+++ qml/Dash/GenericScopeView.qml 2014-08-06 14:50:51 +0000
@@ -38,6 +38,8 @@
38 style: scope ? scope.customizations : {}38 style: scope ? scope.customizations : {}
39 }39 }
4040
41 readonly property bool processing: scope ? scope.searchInProgress || previewListView.processing : false
42
41 signal backClicked()43 signal backClicked()
4244
43 function positionAtBeginning() {45 function positionAtBeginning() {
@@ -379,7 +381,6 @@
379 title: scopeView.scope ? scopeView.scope.name : ""381 title: scopeView.scope ? scopeView.scope.name : ""
380 showBackButton: scopeView.hasBackAction382 showBackButton: scopeView.hasBackAction
381 searchEntryEnabled: true383 searchEntryEnabled: true
382 searchInProgress: scopeView.scope ? scopeView.scope.searchInProgress : false
383 scopeStyle: scopeView.scopeStyle384 scopeStyle: scopeView.scopeStyle
384385
385 bottomItem: DashNavigation {386 bottomItem: DashNavigation {
386387
=== modified file 'qml/Dash/PreviewListView.qml'
--- qml/Dash/PreviewListView.qml 2014-07-17 13:10:45 +0000
+++ qml/Dash/PreviewListView.qml 2014-08-06 14:50:51 +0000
@@ -32,6 +32,9 @@
32 property alias currentItem: previewListView.currentItem32 property alias currentItem: previewListView.currentItem
33 property alias count: previewListView.count33 property alias count: previewListView.count
3434
35 readonly property bool processing: currentItem && (!currentItem.previewModel.loaded
36 || currentItem.previewModel.processingAction)
37
35 PageHeader {38 PageHeader {
36 id: header39 id: header
37 objectName: "pageHeader"40 objectName: "pageHeader"
@@ -83,39 +86,32 @@
83 }86 }
84 }87 }
8588
86 delegate: Item {89 delegate: Previews.Preview {
87 objectName: "previewItem" + index90 id: preview
91 objectName: "preview" + index
88 height: previewListView.height92 height: previewListView.height
89 width: previewListView.width93 width: previewListView.width
9094
91 readonly property bool ready: preview.previewModel.loaded95 isCurrent: ListView.isCurrentItem
9296
93 Previews.Preview {97 previewModel: {
94 id: preview98 var previewStack = root.scope.preview(result);
95 objectName: "preview" + index99 return previewStack.getPreviewModel(0);
96 anchors.fill: parent100 }
97101 scopeStyle: root.scopeStyle
98 isCurrent: parent.ListView.isCurrentItem102 }
99103 }
100 previewModel: {104
101 var previewStack = root.scope.preview(result);105 MouseArea {
102 return previewStack.getPreviewModel(0);106 id: processingMouseArea
103 }107 objectName: "processingMouseArea"
104 scopeStyle: root.scopeStyle108 anchors {
105 }109 left: parent.left
106110 right: parent.right
107 MouseArea {111 top: pageHeader.bottom
108 id: processingMouseArea112 bottom: parent.bottom
109 objectName: "processingMouseArea"113 }
110 anchors.fill: parent114
111 enabled: !preview.previewModel.loaded || preview.previewModel.processingAction115 enabled: root.processing
112
113 ActivityIndicator {
114 anchors.centerIn: parent
115 visible: root.open && parent.enabled
116 running: visible
117 }
118 }
119 }
120 }116 }
121}117}
122118
=== modified file 'tests/autopilot/unity8/shell/emulators/dash.py'
--- tests/autopilot/unity8/shell/emulators/dash.py 2014-07-29 19:49:36 +0000
+++ tests/autopilot/unity8/shell/emulators/dash.py 2014-08-06 14:50:51 +0000
@@ -150,7 +150,7 @@
150 headerContainer.contentY.wait_for(0)150 headerContainer.contentY.wait_for(0)
151 search_text_field = self._get_search_text_field()151 search_text_field = self._get_search_text_field()
152 search_text_field.write(query)152 search_text_field.write(query)
153 current_header.select_single(objectName="searchIndicator").running.wait_for(False)153 self.select_single(objectName="processingIndicator").visible.wait_for(False)
154154
155 def _get_search_text_field(self):155 def _get_search_text_field(self):
156 page_header = self._get_current_page_header()156 page_header = self._get_current_page_header()
157157
=== modified file 'tests/plugins/Dash/tst_ScopeStyle.qml'
--- tests/plugins/Dash/tst_ScopeStyle.qml 2014-07-19 07:56:55 +0000
+++ tests/plugins/Dash/tst_ScopeStyle.qml 2014-08-06 14:50:51 +0000
@@ -54,6 +54,9 @@
54 { tag: "lightgrey", luminance: 0.8275 },54 { tag: "lightgrey", luminance: 0.8275 },
55 { tag: "grey", luminance: 0.5020 },55 { tag: "grey", luminance: 0.5020 },
56 { tag: "darkgrey", luminance: 0.6627 },56 { tag: "darkgrey", luminance: 0.6627 },
57 { tag: "red", luminance: 0.2126 },
58 { tag: "green", luminance: 0.3590 },
59 { tag: "blue", luminance: 0.0722 },
57 ];60 ];
58 }61 }
5962
@@ -64,10 +67,10 @@
6467
65 function test_foreground_data() {68 function test_foreground_data() {
66 return [69 return [
67 { tag: "default", index: 0, foreground: "grey" },70 { tag: "default", index: 0, foreground: "grey", luminance: 0.5020 },
68 { tag: "red on black", index: 1, foreground: "red" },71 { tag: "red on black", index: 1, foreground: "red", luminance: 0.2126 },
69 { tag: "green on white", index: 2, foreground: "green" },72 { tag: "green on white", index: 2, foreground: "green", luminance: 0.3590 },
70 { tag: "blue on darkgrey", index: 3, foreground: "blue" },73 { tag: "blue on darkgrey", index: 3, foreground: "blue", luminance: 0.0722 },
71 ];74 ];
72 }75 }
7376
@@ -75,14 +78,15 @@
75 tool.style = testCase.styles[data.index];78 tool.style = testCase.styles[data.index];
76 verify(Qt.colorEqual(tool.foreground, data.foreground),79 verify(Qt.colorEqual(tool.foreground, data.foreground),
77 "Foreground color not equal: %1 != %2".arg(tool.foreground).arg(data.foreground));80 "Foreground color not equal: %1 != %2".arg(tool.foreground).arg(data.foreground));
81 compare(tool.foregroundLuminance.toFixed(4), data.luminance.toFixed(4));
78 }82 }
7983
80 function test_background_data() {84 function test_background_data() {
81 return [85 return [
82 { tag: "default", index: 0, background: "transparent" },86 { tag: "default", index: 0, background: "transparent" },
83 { tag: "red on black", index: 1, background: "black" },87 { tag: "red on black", index: 1, background: "black", luminance: 0 },
84 { tag: "green on white", index: 2, background: "white" },88 { tag: "green on white", index: 2, background: "white", luminance: 1 },
85 { tag: "blue on darkgrey", index: 3, background: "darkgrey" },89 { tag: "blue on darkgrey", index: 3, background: "darkgrey", luminance: 0.6627 },
86 ];90 ];
87 }91 }
8892
@@ -90,6 +94,9 @@
90 tool.style = testCase.styles[data.index];94 tool.style = testCase.styles[data.index];
91 verify(Qt.colorEqual(tool.background, data.background),95 verify(Qt.colorEqual(tool.background, data.background),
92 "Background color not equal: %1 != %2".arg(tool.background).arg(data.background));96 "Background color not equal: %1 != %2".arg(tool.background).arg(data.background));
97 if (data.hasOwnProperty("luminance")) {
98 compare(tool.backgroundLuminance.toFixed(4), data.luminance.toFixed(4));
99 }
93 }100 }
94101
95 function test_threshold_data() {102 function test_threshold_data() {
96103
=== modified file 'tests/qmltests/Components/tst_PageHeader.qml'
--- tests/qmltests/Components/tst_PageHeader.qml 2014-07-28 09:14:23 +0000
+++ tests/qmltests/Components/tst_PageHeader.qml 2014-08-06 14:50:51 +0000
@@ -119,29 +119,6 @@
119 compare(pageHeader.searchHistory.get(0).query, "humppa4")119 compare(pageHeader.searchHistory.get(0).query, "humppa4")
120 }120 }
121121
122 function test_search_indicator() {
123 var searchIndicator = findChild(pageHeader, "searchIndicator")
124 var clearIcon = findChild(pageHeader, "clearIcon")
125
126 pageHeader.triggerSearch()
127
128 tryCompare(clearIcon, "visible", false)
129 pageHeader.searchQuery = "ubuntu"
130 tryCompare(clearIcon, "visible", true)
131
132 pageHeader.searchInProgress = false
133 compare(searchIndicator.running, false, "Search indicator is running.")
134 tryCompare(clearIcon, "visible", true)
135
136 pageHeader.searchInProgress = true
137 compare(searchIndicator.running, true, "Search indicator isn't running.")
138 tryCompare(clearIcon, "visible", false)
139
140 pageHeader.searchInProgress = false;
141 compare(searchIndicator.running, false, "Search indicator is running.")
142 tryCompare(clearIcon, "visible", true)
143 }
144
145 function test_titleImage() {122 function test_titleImage() {
146123
147 var titleImage = findChild(pageHeader, "titleImage");124 var titleImage = findChild(pageHeader, "titleImage");
148125
=== modified file 'tests/qmltests/Dash/tst_Dash.qml'
--- tests/qmltests/Dash/tst_Dash.qml 2014-08-06 14:50:51 +0000
+++ tests/qmltests/Dash/tst_Dash.qml 2014-08-06 14:50:51 +0000
@@ -45,12 +45,8 @@
45 name: "Dash"45 name: "Dash"
46 when: windowShown46 when: windowShown
4747
48 property var scopes48 readonly property Item dashContent: findChild(dash, "dashContent");
4949 readonly property var scopes: dashContent.scopes
50 Component.onCompleted: {
51 var dashContent = findChild(dash, "dashContent");
52 scopes = dashContent.scopes;
53 }
5450
55 function init() {51 function init() {
56 // clear and reload the scopes.52 // clear and reload the scopes.
@@ -106,5 +102,26 @@
106 dashCommunicatorService.mockSetCurrentScope("clickscope", true, true);102 dashCommunicatorService.mockSetCurrentScope("clickscope", true, true);
107 tryCompare(dashContentList, "currentIndex", 1)103 tryCompare(dashContentList, "currentIndex", 1)
108 }104 }
105
106 function test_processing_indicator() {
107 tryCompare(scopes, "loaded", true);
108
109 var processingIndicator = findChild(dash, "processingIndicator");
110 verify(processingIndicator, "Can't find the processing indicator.");
111
112 verify(!processingIndicator.visible, "Processing indicator should be visible.");
113
114 tryCompareFunction(function() {
115 return scopes.getScope(dashContent.currentIndex) != null;
116 }, true);
117 var currentScope = scopes.getScope(dashContent.currentIndex);
118 verify(currentScope, "Can't find the current scope.");
119
120 currentScope.setSearchInProgress(true);
121 tryCompare(processingIndicator, "visible", true);
122
123 currentScope.setSearchInProgress(false);
124 tryCompare(processingIndicator, "visible", false);
125 }
109 }126 }
110}127}
111128
=== modified file 'tests/qmltests/Dash/tst_GenericScopeView.qml'
--- tests/qmltests/Dash/tst_GenericScopeView.qml 2014-07-24 20:40:57 +0000
+++ tests/qmltests/Dash/tst_GenericScopeView.qml 2014-08-06 14:50:51 +0000
@@ -319,7 +319,7 @@
319 units.gu(2),319 units.gu(2),
320 testCase.previewListView.height / 2);320 testCase.previewListView.height / 2);
321 tryCompare(previewListViewList, "moving", false);321 tryCompare(previewListViewList, "moving", false);
322 tryCompare(testCase.previewListView.currentItem, "objectName", "previewItem" + i);322 tryCompare(testCase.previewListView.currentItem, "objectName", "preview" + i);
323323
324 }324 }
325 closePreview();325 closePreview();

Subscribers

People subscribed via source and target branches