Merge lp:~mterry/unity8/less-ubuntucolors into lp:unity8
- less-ubuntucolors
- Merge into trunk
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~mterry/unity8/less-ubuntucolors | ||||
Merge into: | lp:unity8 | ||||
Diff against target: |
719 lines (+138/-143) 23 files modified
plugins/Dash/AudioProgressBar.qml (+1/-1) plugins/Dash/ScopeStyle.qml (+1/-1) qml/Components/Dialogs.qml (+2/-2) qml/Components/Header.qml (+0/-28) qml/Components/ModeSwitchWarningDialog.qml (+1/-1) qml/Components/WindowControlButtons.qml (+1/-1) qml/Dash/DashPageHeader.qml (+32/-37) qml/Dash/GenericScopeView.qml (+5/-0) qml/Dash/Previews/PreviewActions.qml (+1/-1) qml/Dash/Previews/PreviewPayments.qml (+1/-1) qml/Launcher/LauncherDelegate.qml (+2/-2) qml/Panel/Panel.qml (+1/-1) qml/Stages/DecoratedWindow.qml (+1/-1) qml/Stages/DesktopSpread.qml (+1/-1) qml/Stages/DesktopSpreadDelegate.qml (+1/-1) qml/Stages/Splash.qml (+7/-19) qml/Stages/SpreadDelegate.qml (+1/-1) tests/plugins/Dash/tst_ScopeStyle.qml (+1/-1) tests/qmltests/Dash/tst_Dash.qml (+9/-9) tests/qmltests/Dash/tst_DashContent.qml (+64/-5) tests/qmltests/Dash/tst_DashPageHeader.qml (+0/-24) tests/qmltests/Dash/tst_GenericScopeView.qml (+2/-2) tests/qmltests/Dash/tst_PreviewView.qml (+3/-3) |
||||
To merge this branch: | bzr merge lp:~mterry/unity8/less-ubuntucolors | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Andrea Cimitan (community) | Needs Fixing | ||
Unity8 CI Bot | continuous-integration | Needs Fixing | |
Review via email: mp+294514@code.launchpad.net |
This proposal has been superseded by a proposal from 2016-05-16.
Commit message
Change some dash button colors from orange to green.
And switch other colors from hardcoded values to the same semantic theme value.
Description of the change
UbuntuColors should be avoided when possible. Instead we should use semantic color names from the theme. These are future-proof and potentially locale-proof (for locales where green and red mean different things -- which I heard was a thing, but can't find anything on it in a quick search).
Half this branch simply replaces some color names with the correct theme color. So no visual change there. But it also changes some colors:
plugins/
plugins/
qml/Dash/
qml/Dash/
qml/Dash/
I didn't change any uses of the various "grey" colors (slate, jet, charcoal, etc). That seemed a bit more fiddly and less user visible. But they should in theory be fixed down the road.
* 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?
Not yet.. will do in a bit
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
NA
* If you changed the UI, has there been a design review?
No... Do we need one for just using the correct theme colors? The only border case I'd see is the purchase button -- maybe that's supposed to be branding-colored?
Unity8 CI Bot (unity8-ci-bot) wrote : | # |
Andrea Cimitan (cimi) wrote : | # |
Please rebase on https:/
- 2331. By Michael Terry
-
Use PageHeader instead Ambiance's PageHeadStyle
- 2332. By Michael Terry
-
Fix some uses of UbuntuColors
Unmerged revisions
Preview Diff
1 | === modified file 'plugins/Dash/AudioProgressBar.qml' | |||
2 | --- plugins/Dash/AudioProgressBar.qml 2016-01-18 22:56:16 +0000 | |||
3 | +++ plugins/Dash/AudioProgressBar.qml 2016-05-16 14:05:53 +0000 | |||
4 | @@ -49,6 +49,6 @@ | |||
5 | 49 | rightMargin: maxWidth - (maxWidth * root.progress) | 49 | rightMargin: maxWidth - (maxWidth * root.progress) |
6 | 50 | } | 50 | } |
7 | 51 | height: units.dp(2) | 51 | height: units.dp(2) |
9 | 52 | backgroundColor: UbuntuColors.orange | 52 | backgroundColor: theme.palette.normal.activity |
10 | 53 | } | 53 | } |
11 | 54 | } | 54 | } |
12 | 55 | 55 | ||
13 | === modified file 'plugins/Dash/ScopeStyle.qml' | |||
14 | --- plugins/Dash/ScopeStyle.qml 2016-03-29 03:47:39 +0000 | |||
15 | +++ plugins/Dash/ScopeStyle.qml 2016-05-16 14:05:53 +0000 | |||
16 | @@ -70,7 +70,7 @@ | |||
17 | 70 | readonly property url navigationBackground: "navigation-background" in d.headerStyle ? d.headerStyle["navigation-background"] : "color:///#f5f5f5" | 70 | readonly property url navigationBackground: "navigation-background" in d.headerStyle ? d.headerStyle["navigation-background"] : "color:///#f5f5f5" |
18 | 71 | 71 | ||
19 | 72 | /// Color of the primary preview button | 72 | /// Color of the primary preview button |
21 | 73 | readonly property color previewButtonColor: "preview-button-color" in style ? style["preview-button-color"] : UbuntuColors.orange | 73 | readonly property color previewButtonColor: "preview-button-color" in style ? style["preview-button-color"] : theme.palette.normal.positive |
22 | 74 | 74 | ||
23 | 75 | //! @cond | 75 | //! @cond |
24 | 76 | property var d: QtObject { | 76 | property var d: QtObject { |
25 | 77 | 77 | ||
26 | === modified file 'qml/Components/Dialogs.qml' | |||
27 | --- qml/Components/Dialogs.qml 2016-04-27 15:01:10 +0000 | |||
28 | +++ qml/Components/Dialogs.qml 2016-05-16 14:05:53 +0000 | |||
29 | @@ -185,7 +185,7 @@ | |||
30 | 185 | unitySessionService.reboot(); | 185 | unitySessionService.reboot(); |
31 | 186 | rebootDialog.hide(); | 186 | rebootDialog.hide(); |
32 | 187 | } | 187 | } |
34 | 188 | color: UbuntuColors.red | 188 | color: theme.palette.normal.negative |
35 | 189 | } | 189 | } |
36 | 190 | } | 190 | } |
37 | 191 | } | 191 | } |
38 | @@ -203,7 +203,7 @@ | |||
39 | 203 | powerDialog.hide(); | 203 | powerDialog.hide(); |
40 | 204 | root.powerOffClicked(); | 204 | root.powerOffClicked(); |
41 | 205 | } | 205 | } |
43 | 206 | color: UbuntuColors.red | 206 | color: theme.palette.normal.negative |
44 | 207 | } | 207 | } |
45 | 208 | Button { | 208 | Button { |
46 | 209 | text: i18n.ctr("Button: Restart the system", "Restart") | 209 | text: i18n.ctr("Button: Restart the system", "Restart") |
47 | 210 | 210 | ||
48 | === removed file 'qml/Components/Header.qml' | |||
49 | --- qml/Components/Header.qml 2016-01-25 10:12:48 +0000 | |||
50 | +++ qml/Components/Header.qml 1970-01-01 00:00:00 +0000 | |||
51 | @@ -1,28 +0,0 @@ | |||
52 | 1 | /* | ||
53 | 2 | * Copyright (C) 2014 Canonical, Ltd. | ||
54 | 3 | * | ||
55 | 4 | * This program is free software; you can redistribute it and/or modify | ||
56 | 5 | * it under the terms of the GNU General Public License as published by | ||
57 | 6 | * the Free Software Foundation; version 3. | ||
58 | 7 | * | ||
59 | 8 | * This program is distributed in the hope that it will be useful, | ||
60 | 9 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
61 | 10 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||
62 | 11 | * GNU General Public License for more details. | ||
63 | 12 | * | ||
64 | 13 | * You should have received a copy of the GNU General Public License | ||
65 | 14 | * along with this program. If not, see <http://www.gnu.org/licenses/>. | ||
66 | 15 | */ | ||
67 | 16 | |||
68 | 17 | import Ubuntu.Components 1.3 | ||
69 | 18 | import Ubuntu.Components.Themes.Ambiance 1.3 | ||
70 | 19 | |||
71 | 20 | PageHeadStyle { | ||
72 | 21 | // FIXME: Don't use this but PageHeader from Ubuntu.Components 1.3. | ||
73 | 22 | id: root | ||
74 | 23 | |||
75 | 24 | property var styledItem: root | ||
76 | 25 | property string title | ||
77 | 26 | property var config: PageHeadConfiguration {} | ||
78 | 27 | property var contents: null | ||
79 | 28 | } | ||
80 | 29 | 0 | ||
81 | === modified file 'qml/Components/ModeSwitchWarningDialog.qml' | |||
82 | --- qml/Components/ModeSwitchWarningDialog.qml 2016-03-08 13:57:52 +0000 | |||
83 | +++ qml/Components/ModeSwitchWarningDialog.qml 2016-05-16 14:05:53 +0000 | |||
84 | @@ -86,7 +86,7 @@ | |||
85 | 86 | Button { | 86 | Button { |
86 | 87 | objectName: "forceCloseButton" | 87 | objectName: "forceCloseButton" |
87 | 88 | text: i18n.tr("Close all") | 88 | text: i18n.tr("Close all") |
89 | 89 | color: UbuntuColors.red | 89 | color: theme.palette.normal.negative |
90 | 90 | onClicked: { | 90 | onClicked: { |
91 | 91 | root.forceClose(); | 91 | root.forceClose(); |
92 | 92 | } | 92 | } |
93 | 93 | 93 | ||
94 | === modified file 'qml/Components/WindowControlButtons.qml' | |||
95 | --- qml/Components/WindowControlButtons.qml 2016-04-27 14:57:19 +0000 | |||
96 | +++ qml/Components/WindowControlButtons.qml 2016-05-16 14:05:53 +0000 | |||
97 | @@ -43,7 +43,7 @@ | |||
98 | 43 | width: units.gu(2) | 43 | width: units.gu(2) |
99 | 44 | height: units.gu(2) | 44 | height: units.gu(2) |
100 | 45 | radius: height / 2 | 45 | radius: height / 2 |
102 | 46 | color: UbuntuColors.red | 46 | color: theme.palette.normal.negative |
103 | 47 | visible: parent.containsMouse | 47 | visible: parent.containsMouse |
104 | 48 | } | 48 | } |
105 | 49 | Icon { | 49 | Icon { |
106 | 50 | 50 | ||
107 | === modified file 'qml/Dash/DashPageHeader.qml' | |||
108 | --- qml/Dash/DashPageHeader.qml 2016-04-27 15:01:10 +0000 | |||
109 | +++ qml/Dash/DashPageHeader.qml 2016-05-16 14:05:53 +0000 | |||
110 | @@ -16,7 +16,7 @@ | |||
111 | 16 | 16 | ||
112 | 17 | import QtQuick 2.4 | 17 | import QtQuick 2.4 |
113 | 18 | import Ubuntu.Components 1.3 | 18 | import Ubuntu.Components 1.3 |
115 | 19 | import Ubuntu.Components.Themes.Ambiance 1.3 | 19 | import Ubuntu.Components.Popups 1.3 |
116 | 20 | import Ubuntu.Components.ListItems 1.3 | 20 | import Ubuntu.Components.ListItems 1.3 |
117 | 21 | import "../Components" | 21 | import "../Components" |
118 | 22 | 22 | ||
119 | @@ -78,13 +78,13 @@ | |||
120 | 78 | } | 78 | } |
121 | 79 | } | 79 | } |
122 | 80 | 80 | ||
124 | 81 | function closePopup(keepFocus) { | 81 | function closePopup(keepFocus, keepSearch) { |
125 | 82 | if (extraPanel.visible) { | 82 | if (extraPanel.visible) { |
126 | 83 | extraPanel.visible = false; | 83 | extraPanel.visible = false; |
127 | 84 | } else if (!keepFocus) { | 84 | } else if (!keepFocus) { |
129 | 85 | unfocus(); | 85 | unfocus(keepSearch); |
130 | 86 | } | 86 | } |
132 | 87 | if (!searchTextField.text && !root.navigationTag && searchHistory.count == 0) { | 87 | if (!keepSearch && !searchTextField.text && !root.navigationTag && searchHistory.count == 0) { |
133 | 88 | headerContainer.showSearch = false; | 88 | headerContainer.showSearch = false; |
134 | 89 | } | 89 | } |
135 | 90 | } | 90 | } |
136 | @@ -97,9 +97,9 @@ | |||
137 | 97 | closePopup(keepFocus); | 97 | closePopup(keepFocus); |
138 | 98 | } | 98 | } |
139 | 99 | 99 | ||
141 | 100 | function unfocus() { | 100 | function unfocus(keepSearch) { |
142 | 101 | searchTextField.focus = false; | 101 | searchTextField.focus = false; |
144 | 102 | if (!searchTextField.text && !root.navigationTag) { | 102 | if (!keepSearch && !searchTextField.text && !root.navigationTag) { |
145 | 103 | headerContainer.showSearch = false; | 103 | headerContainer.showSearch = false; |
146 | 104 | } | 104 | } |
147 | 105 | } | 105 | } |
148 | @@ -142,7 +142,7 @@ | |||
149 | 142 | objectName: "headerContainer" | 142 | objectName: "headerContainer" |
150 | 143 | clip: contentY < height | 143 | clip: contentY < height |
151 | 144 | anchors { left: parent.left; top: parent.top; right: parent.right } | 144 | anchors { left: parent.left; top: parent.top; right: parent.right } |
153 | 145 | height: header.contentHeight | 145 | height: header.__styleInstance.contentHeight |
154 | 146 | contentHeight: headersColumn.height | 146 | contentHeight: headersColumn.height |
155 | 147 | interactive: false | 147 | interactive: false |
156 | 148 | contentY: showSearch ? 0 : height | 148 | contentY: showSearch ? 0 : height |
157 | @@ -173,22 +173,18 @@ | |||
158 | 173 | id: headersColumn | 173 | id: headersColumn |
159 | 174 | anchors { left: parent.left; right: parent.right } | 174 | anchors { left: parent.left; right: parent.right } |
160 | 175 | 175 | ||
163 | 176 | PageHeadStyle { | 176 | PageHeader { |
162 | 177 | // FIXME: Replace PageHeadStyle from the Ambiance theme by the new PageHeader from Ubuntu.Components 1.3. | ||
164 | 178 | id: searchHeader | 177 | id: searchHeader |
165 | 179 | anchors { left: parent.left; right: parent.right } | 178 | anchors { left: parent.left; right: parent.right } |
166 | 180 | opacity: headerContainer.clip || headerContainer.showSearch ? 1 : 0 // setting visible false cause column to relayout | 179 | opacity: headerContainer.clip || headerContainer.showSearch ? 1 : 0 // setting visible false cause column to relayout |
175 | 181 | __separator_visible: false | 180 | |
176 | 182 | // Required to keep PageHeadStyle noise down as it expects the Page's properties around. | 181 | StyleHints { |
169 | 183 | property var styledItem: searchHeader | ||
170 | 184 | property color dividerColor: "transparent" // Doesn't matter as we don't have PageHeadSections | ||
171 | 185 | property color panelColor: background.topColor | ||
172 | 186 | panelForegroundColor: config.foregroundColor | ||
173 | 187 | backgroundColor: "transparent" | ||
174 | 188 | config: PageHeadConfiguration { | ||
177 | 189 | foregroundColor: root.scopeStyle ? root.scopeStyle.headerForeground : theme.palette.normal.baseText | 182 | foregroundColor: root.scopeStyle ? root.scopeStyle.headerForeground : theme.palette.normal.baseText |
178 | 183 | backgroundColor: "transparent" | ||
179 | 184 | dividerColor: "transparent" | ||
180 | 190 | } | 185 | } |
182 | 191 | property var contents: Item { | 186 | |
183 | 187 | contents: Item { | ||
184 | 192 | anchors.fill: parent | 188 | anchors.fill: parent |
185 | 193 | 189 | ||
186 | 194 | TextField { | 190 | TextField { |
187 | @@ -273,7 +269,7 @@ | |||
188 | 273 | anchors.fill: parent | 269 | anchors.fill: parent |
189 | 274 | anchors.margins: units.gu(2) | 270 | anchors.margins: units.gu(2) |
190 | 275 | name: "filters" | 271 | name: "filters" |
192 | 276 | color: root.activeFiltersCount > 0 ? UbuntuColors.orange : header.config.foregroundColor | 272 | color: root.activeFiltersCount > 0 ? theme.palette.normal.positive : header.__styleInstance.foregroundColor |
193 | 277 | } | 273 | } |
194 | 278 | 274 | ||
195 | 279 | onClicked: { | 275 | onClicked: { |
196 | @@ -283,6 +279,7 @@ | |||
197 | 283 | 279 | ||
198 | 284 | AbstractButton { | 280 | AbstractButton { |
199 | 285 | id: cancelButton | 281 | id: cancelButton |
200 | 282 | objectName: "cancelButton" | ||
201 | 286 | width: cancelLabel.width + cancelLabel.anchors.rightMargin + cancelLabel.anchors.leftMargin | 283 | width: cancelLabel.width + cancelLabel.anchors.rightMargin + cancelLabel.anchors.leftMargin |
202 | 287 | anchors { | 284 | anchors { |
203 | 288 | top: parent.top | 285 | top: parent.top |
204 | @@ -296,7 +293,7 @@ | |||
205 | 296 | Label { | 293 | Label { |
206 | 297 | id: cancelLabel | 294 | id: cancelLabel |
207 | 298 | text: i18n.tr("Cancel") | 295 | text: i18n.tr("Cancel") |
209 | 299 | color: header.panelForegroundColor | 296 | color: header.__styleInstance.foregroundColor |
210 | 300 | verticalAlignment: Text.AlignVCenter | 297 | verticalAlignment: Text.AlignVCenter |
211 | 301 | anchors { | 298 | anchors { |
212 | 302 | verticalCenter: parent.verticalCenter | 299 | verticalCenter: parent.verticalCenter |
213 | @@ -309,28 +306,27 @@ | |||
214 | 309 | } | 306 | } |
215 | 310 | } | 307 | } |
216 | 311 | 308 | ||
219 | 312 | PageHeadStyle { | 309 | PageHeader { |
218 | 313 | // FIXME: Replace PageHeadStyle from the Ambiance theme by the new PageHeader from Ubuntu.Components 1.3. | ||
220 | 314 | id: header | 310 | id: header |
221 | 315 | objectName: "innerPageHeader" | 311 | objectName: "innerPageHeader" |
222 | 316 | anchors { left: parent.left; right: parent.right } | 312 | anchors { left: parent.left; right: parent.right } |
223 | 317 | height: headerContainer.height | 313 | height: headerContainer.height |
224 | 318 | opacity: headerContainer.clip || !headerContainer.showSearch ? 1 : 0 // setting visible false cause column to relayout | 314 | opacity: headerContainer.clip || !headerContainer.showSearch ? 1 : 0 // setting visible false cause column to relayout |
233 | 319 | __separator_visible: false | 315 | title: root.title |
234 | 320 | property var styledItem: header | 316 | |
235 | 321 | property color dividerColor: "transparent" // Doesn't matter as we don't have PageHeadSections | 317 | StyleHints { |
228 | 322 | property color panelColor: background.topColor | ||
229 | 323 | panelForegroundColor: config.foregroundColor | ||
230 | 324 | backgroundColor: "transparent" | ||
231 | 325 | config: PageHeadConfiguration { | ||
232 | 326 | title: root.title | ||
236 | 327 | foregroundColor: root.scopeStyle ? root.scopeStyle.headerForeground : theme.palette.normal.baseText | 318 | foregroundColor: root.scopeStyle ? root.scopeStyle.headerForeground : theme.palette.normal.baseText |
243 | 328 | backAction: Action { | 319 | backgroundColor: "transparent" |
244 | 329 | iconName: backIsClose ? "close" : "back" | 320 | dividerColor: "transparent" |
245 | 330 | visible: root.showBackButton | 321 | } |
246 | 331 | onTriggered: root.backClicked() | 322 | |
247 | 332 | } | 323 | leadingActionBar.actions: Action { |
248 | 333 | 324 | iconName: backIsClose ? "close" : "back" | |
249 | 325 | visible: root.showBackButton | ||
250 | 326 | onTriggered: root.backClicked() | ||
251 | 327 | } | ||
252 | 328 | |||
253 | 329 | trailingActionBar { | ||
254 | 334 | actions: [ | 330 | actions: [ |
255 | 335 | Action { | 331 | Action { |
256 | 336 | objectName: "store" | 332 | objectName: "store" |
257 | @@ -366,7 +362,6 @@ | |||
258 | 366 | ] | 362 | ] |
259 | 367 | } | 363 | } |
260 | 368 | 364 | ||
261 | 369 | property var contents: null | ||
262 | 370 | Component.onCompleted: root.refreshLogo() | 365 | Component.onCompleted: root.refreshLogo() |
263 | 371 | 366 | ||
264 | 372 | Component { | 367 | Component { |
265 | 373 | 368 | ||
266 | === modified file 'qml/Dash/GenericScopeView.qml' | |||
267 | --- qml/Dash/GenericScopeView.qml 2016-05-04 18:08:35 +0000 | |||
268 | +++ qml/Dash/GenericScopeView.qml 2016-05-16 14:05:53 +0000 | |||
269 | @@ -662,6 +662,10 @@ | |||
270 | 662 | onShowFiltersPopup: { // item | 662 | onShowFiltersPopup: { // item |
271 | 663 | extraPanel.visible = false; | 663 | extraPanel.visible = false; |
272 | 664 | scopeView.filtersPopover = PopupUtils.open(Qt.resolvedUrl("FiltersPopover.qml"), item, { "contentWidth": scopeView.width - units.gu(2) } ); | 664 | scopeView.filtersPopover = PopupUtils.open(Qt.resolvedUrl("FiltersPopover.qml"), item, { "contentWidth": scopeView.width - units.gu(2) } ); |
273 | 665 | scopeView.filtersPopover.Component.onDestruction.connect(function () { | ||
274 | 666 | categoryView.pageHeader.closePopup(false, true); | ||
275 | 667 | categoryView.pageHeader.unfocus(true); // remove the focus from the search field | ||
276 | 668 | }) | ||
277 | 665 | } | 669 | } |
278 | 666 | } | 670 | } |
279 | 667 | 671 | ||
280 | @@ -683,6 +687,7 @@ | |||
281 | 683 | onHistoryItemClicked: { | 687 | onHistoryItemClicked: { |
282 | 684 | SearchHistoryModel.addQuery(text); | 688 | SearchHistoryModel.addQuery(text); |
283 | 685 | categoryView.pageHeader.searchQuery = text; | 689 | categoryView.pageHeader.searchQuery = text; |
284 | 690 | categoryView.pageHeader.unfocus(); | ||
285 | 686 | } | 691 | } |
286 | 687 | 692 | ||
287 | 688 | onDashNavigationLeafClicked: { | 693 | onDashNavigationLeafClicked: { |
288 | 689 | 694 | ||
289 | === modified file 'qml/Dash/Previews/PreviewActions.qml' | |||
290 | --- qml/Dash/Previews/PreviewActions.qml 2016-04-01 13:20:27 +0000 | |||
291 | +++ qml/Dash/Previews/PreviewActions.qml 2016-05-16 14:05:53 +0000 | |||
292 | @@ -66,7 +66,7 @@ | |||
293 | 66 | visible: root.actions && root.actions.length > 0 | 66 | visible: root.actions && root.actions.length > 0 |
294 | 67 | onTriggeredAction: root.triggered(root.widgetId, actionData.id, actionData) | 67 | onTriggeredAction: root.triggered(root.widgetId, actionData.id, actionData) |
295 | 68 | width: (root.width - units.gu(1)) / 2 | 68 | width: (root.width - units.gu(1)) / 2 |
297 | 69 | color: root.scopeStyle ? root.scopeStyle.previewButtonColor : UbuntuColors.orange | 69 | color: root.scopeStyle ? root.scopeStyle.previewButtonColor : theme.palette.normal.positive |
298 | 70 | } | 70 | } |
299 | 71 | } | 71 | } |
300 | 72 | } | 72 | } |
301 | 73 | 73 | ||
302 | === modified file 'qml/Dash/Previews/PreviewPayments.qml' | |||
303 | --- qml/Dash/Previews/PreviewPayments.qml 2016-02-16 20:00:47 +0000 | |||
304 | +++ qml/Dash/Previews/PreviewPayments.qml 2016-05-16 14:05:53 +0000 | |||
305 | @@ -35,7 +35,7 @@ | |||
306 | 35 | id: paymentButton | 35 | id: paymentButton |
307 | 36 | objectName: "paymentButton" | 36 | objectName: "paymentButton" |
308 | 37 | 37 | ||
310 | 38 | color: UbuntuColors.orange | 38 | color: theme.palette.normal.positive |
311 | 39 | text: paymentClient.formattedPrice | 39 | text: paymentClient.formattedPrice |
312 | 40 | onClicked: { | 40 | onClicked: { |
313 | 41 | paymentClient.start(); | 41 | paymentClient.start(); |
314 | 42 | 42 | ||
315 | === modified file 'qml/Launcher/LauncherDelegate.qml' | |||
316 | --- qml/Launcher/LauncherDelegate.qml 2016-03-29 13:51:56 +0000 | |||
317 | +++ qml/Launcher/LauncherDelegate.qml 2016-05-16 14:05:53 +0000 | |||
318 | @@ -164,7 +164,7 @@ | |||
319 | 164 | } | 164 | } |
320 | 165 | width: Math.min(root.itemWidth, Math.max(units.gu(2), countLabel.implicitWidth + units.gu(1))) | 165 | width: Math.min(root.itemWidth, Math.max(units.gu(2), countLabel.implicitWidth + units.gu(1))) |
321 | 166 | height: units.gu(2) | 166 | height: units.gu(2) |
323 | 167 | backgroundColor: UbuntuColors.green | 167 | backgroundColor: theme.palette.normal.positive |
324 | 168 | visible: root.countVisible | 168 | visible: root.countVisible |
325 | 169 | aspect: UbuntuShape.Flat | 169 | aspect: UbuntuShape.Flat |
326 | 170 | 170 | ||
327 | @@ -207,7 +207,7 @@ | |||
328 | 207 | top: parent.top | 207 | top: parent.top |
329 | 208 | bottom: parent.bottom | 208 | bottom: parent.bottom |
330 | 209 | } | 209 | } |
332 | 210 | backgroundColor: UbuntuColors.blue | 210 | backgroundColor: theme.palette.normal.activity |
333 | 211 | borderSource: "none" | 211 | borderSource: "none" |
334 | 212 | width: progressOverlay.width | 212 | width: progressOverlay.width |
335 | 213 | } | 213 | } |
336 | 214 | 214 | ||
337 | === modified file 'qml/Panel/Panel.qml' | |||
338 | --- qml/Panel/Panel.qml 2016-04-27 15:01:10 +0000 | |||
339 | +++ qml/Panel/Panel.qml 2016-05-16 14:05:53 +0000 | |||
340 | @@ -89,7 +89,7 @@ | |||
341 | 89 | 89 | ||
342 | 90 | Rectangle { | 90 | Rectangle { |
343 | 91 | id: indicatorAreaBackground | 91 | id: indicatorAreaBackground |
345 | 92 | color: callHint.visible ? UbuntuColors.green : theme.palette.normal.background | 92 | color: callHint.visible ? theme.palette.normal.positive : theme.palette.normal.background |
346 | 93 | anchors { | 93 | anchors { |
347 | 94 | top: parent.top | 94 | top: parent.top |
348 | 95 | left: parent.left | 95 | left: parent.left |
349 | 96 | 96 | ||
350 | === modified file 'qml/Stages/DecoratedWindow.qml' | |||
351 | --- qml/Stages/DecoratedWindow.qml 2016-04-27 15:01:10 +0000 | |||
352 | +++ qml/Stages/DecoratedWindow.qml 2016-05-16 14:05:53 +0000 | |||
353 | @@ -62,7 +62,7 @@ | |||
354 | 62 | Rectangle { | 62 | Rectangle { |
355 | 63 | anchors { left: selectionHighlight.left; right: selectionHighlight.right; bottom: selectionHighlight.bottom; } | 63 | anchors { left: selectionHighlight.left; right: selectionHighlight.right; bottom: selectionHighlight.bottom; } |
356 | 64 | height: units.dp(2) | 64 | height: units.dp(2) |
358 | 65 | color: UbuntuColors.orange | 65 | color: theme.palette.normal.focus |
359 | 66 | visible: highlightShown | 66 | visible: highlightShown |
360 | 67 | } | 67 | } |
361 | 68 | 68 | ||
362 | 69 | 69 | ||
363 | === modified file 'qml/Stages/DesktopSpread.qml' | |||
364 | --- qml/Stages/DesktopSpread.qml 2016-04-27 11:55:12 +0000 | |||
365 | +++ qml/Stages/DesktopSpread.qml 2016-05-16 14:05:53 +0000 | |||
366 | @@ -494,7 +494,7 @@ | |||
367 | 494 | // Rectangle { | 494 | // Rectangle { |
368 | 495 | // anchors { left: parent.left; right: parent.right; bottom: parent.bottom } | 495 | // anchors { left: parent.left; right: parent.right; bottom: parent.bottom } |
369 | 496 | // height: units.dp(2) | 496 | // height: units.dp(2) |
371 | 497 | // color: UbuntuColors.orange | 497 | // color: theme.palette.normal.focus |
372 | 498 | // visible: index == 0 // TODO: should be active workspace index | 498 | // visible: index == 0 // TODO: should be active workspace index |
373 | 499 | // } | 499 | // } |
374 | 500 | } | 500 | } |
375 | 501 | 501 | ||
376 | === modified file 'qml/Stages/DesktopSpreadDelegate.qml' | |||
377 | --- qml/Stages/DesktopSpreadDelegate.qml 2016-04-04 13:37:49 +0000 | |||
378 | +++ qml/Stages/DesktopSpreadDelegate.qml 2016-05-16 14:05:53 +0000 | |||
379 | @@ -83,7 +83,7 @@ | |||
380 | 83 | Rectangle { | 83 | Rectangle { |
381 | 84 | anchors { left: selectionHighlight.left; right: selectionHighlight.right; bottom: selectionHighlight.bottom; } | 84 | anchors { left: selectionHighlight.left; right: selectionHighlight.right; bottom: selectionHighlight.bottom; } |
382 | 85 | height: units.dp(2) | 85 | height: units.dp(2) |
384 | 86 | color: UbuntuColors.orange | 86 | color: theme.palette.normal.focus |
385 | 87 | visible: root.highlightShown | 87 | visible: root.highlightShown |
386 | 88 | antialiasing: true | 88 | antialiasing: true |
387 | 89 | } | 89 | } |
388 | 90 | 90 | ||
389 | === modified file 'qml/Stages/Splash.qml' | |||
390 | --- qml/Stages/Splash.qml 2016-03-08 20:59:22 +0000 | |||
391 | +++ qml/Stages/Splash.qml 2016-05-16 14:05:53 +0000 | |||
392 | @@ -29,7 +29,7 @@ | |||
393 | 29 | property color footerColor: d.undefinedColor | 29 | property color footerColor: d.undefinedColor |
394 | 30 | property alias imageSource: overlaidImage.source | 30 | property alias imageSource: overlaidImage.source |
395 | 31 | property url icon | 31 | property url icon |
397 | 32 | property alias title: headerConfig.title | 32 | property alias title: header.title |
398 | 33 | property alias showHeader: header.visible | 33 | property alias showHeader: header.visible |
399 | 34 | 34 | ||
400 | 35 | Ambiance.Palette { | 35 | Ambiance.Palette { |
401 | @@ -75,25 +75,13 @@ | |||
402 | 75 | style: Component { MainViewStyle {theme: styledItem.fakeTheme} } | 75 | style: Component { MainViewStyle {theme: styledItem.fakeTheme} } |
403 | 76 | } | 76 | } |
404 | 77 | 77 | ||
407 | 78 | Ambiance.PageHeadStyle { | 78 | PageHeader { |
406 | 79 | // FIXME: Replace PageHeadStyle by PageHeader from Ubuntu.Components 1.3. | ||
408 | 80 | id: header | 79 | id: header |
425 | 81 | anchors { | 80 | anchors { left: parent.left; right: parent.right } |
426 | 82 | left: parent.left; | 81 | StyleHints { |
427 | 83 | right: parent.right | 82 | foregroundColor: styledItem.fakeTheme.palette.selected.backgroundText |
428 | 84 | } | 83 | backgroundColor: "transparent" |
429 | 85 | property var styledItem: header | 84 | } |
414 | 86 | // FIXME Keep in sync with SDK's MainView.qml values of these two colors | ||
415 | 87 | property color dividerColor: Qt.darker(styledItem.backgroundColor, 1.1) | ||
416 | 88 | property color panelColor: Qt.lighter(styledItem.backgroundColor, 1.1) | ||
417 | 89 | panelForegroundColor: config.foregroundColor | ||
418 | 90 | backgroundColor: "transparent" | ||
419 | 91 | config: PageHeadConfiguration { | ||
420 | 92 | id: headerConfig | ||
421 | 93 | foregroundColor: styledItem.fakeTheme.palette.normal.backgroundText | ||
422 | 94 | } | ||
423 | 95 | |||
424 | 96 | property var contents: null | ||
430 | 97 | } | 85 | } |
431 | 98 | 86 | ||
432 | 99 | Image { | 87 | Image { |
433 | 100 | 88 | ||
434 | === modified file 'qml/Stages/SpreadDelegate.qml' | |||
435 | --- qml/Stages/SpreadDelegate.qml 2016-04-27 15:01:10 +0000 | |||
436 | +++ qml/Stages/SpreadDelegate.qml 2016-05-16 14:05:53 +0000 | |||
437 | @@ -286,7 +286,7 @@ | |||
438 | 286 | Rectangle { | 286 | Rectangle { |
439 | 287 | anchors { left: selectionHighlight.left; right: selectionHighlight.right; bottom: selectionHighlight.bottom; } | 287 | anchors { left: selectionHighlight.left; right: selectionHighlight.right; bottom: selectionHighlight.bottom; } |
440 | 288 | height: units.dp(2) | 288 | height: units.dp(2) |
442 | 289 | color: UbuntuColors.orange | 289 | color: theme.palette.normal.focus |
443 | 290 | visible: root.highlightShown | 290 | visible: root.highlightShown |
444 | 291 | antialiasing: true | 291 | antialiasing: true |
445 | 292 | } | 292 | } |
446 | 293 | 293 | ||
447 | === modified file 'tests/plugins/Dash/tst_ScopeStyle.qml' | |||
448 | --- tests/plugins/Dash/tst_ScopeStyle.qml 2016-03-29 03:47:39 +0000 | |||
449 | +++ tests/plugins/Dash/tst_ScopeStyle.qml 2016-05-16 14:05:53 +0000 | |||
450 | @@ -174,7 +174,7 @@ | |||
451 | 174 | 174 | ||
452 | 175 | function test_previewButtonColor_data() { | 175 | function test_previewButtonColor_data() { |
453 | 176 | return [ | 176 | return [ |
455 | 177 | { tag: "default", index: 0, previewButtonColor: UbuntuColors.orange }, | 177 | { tag: "default", index: 0, previewButtonColor: theme.palette.normal.positive }, |
456 | 178 | { tag: "red", index: 1, previewButtonColor: "red" }, | 178 | { tag: "red", index: 1, previewButtonColor: "red" }, |
457 | 179 | ]; | 179 | ]; |
458 | 180 | } | 180 | } |
459 | 181 | 181 | ||
460 | === modified file 'tests/qmltests/Dash/tst_Dash.qml' | |||
461 | --- tests/qmltests/Dash/tst_Dash.qml 2016-02-15 10:14:43 +0000 | |||
462 | +++ tests/qmltests/Dash/tst_Dash.qml 2016-05-16 14:05:53 +0000 | |||
463 | @@ -125,7 +125,7 @@ | |||
464 | 125 | // Go back | 125 | // Go back |
465 | 126 | var scopesList = findChild(dash, "scopesList"); | 126 | var scopesList = findChild(dash, "scopesList"); |
466 | 127 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); | 127 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); |
468 | 128 | var backButton = findChild(findChild(scopesListPageHeader, "innerPageHeader"), "customBackButton"); | 128 | var backButton = findChild(scopesListPageHeader, "innerPageHeader").leadingActionBar; |
469 | 129 | mouseClick(backButton); | 129 | mouseClick(backButton); |
470 | 130 | tryCompare(bottomEdgeController, "progress", 0); | 130 | tryCompare(bottomEdgeController, "progress", 0); |
471 | 131 | } | 131 | } |
472 | @@ -200,7 +200,7 @@ | |||
473 | 200 | 200 | ||
474 | 201 | // Go back | 201 | // Go back |
475 | 202 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); | 202 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); |
477 | 203 | var backButton = findChild(findChild(dashTempScopeItemHeader, "innerPageHeader"), "customBackButton"); | 203 | var backButton = findChild(dashTempScopeItemHeader, "innerPageHeader").leadingActionBar; |
478 | 204 | mouseClick(backButton); | 204 | mouseClick(backButton); |
479 | 205 | 205 | ||
480 | 206 | // Check temp scope is gone | 206 | // Check temp scope is gone |
481 | @@ -253,7 +253,7 @@ | |||
482 | 253 | 253 | ||
483 | 254 | // Go back | 254 | // Go back |
484 | 255 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); | 255 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); |
486 | 256 | var backButton = findChild(findChild(dashTempScopeItemHeader, "innerPageHeader"), "customBackButton"); | 256 | var backButton = findChild(dashTempScopeItemHeader, "innerPageHeader").leadingActionBar; |
487 | 257 | mouseClick(backButton); | 257 | mouseClick(backButton); |
488 | 258 | 258 | ||
489 | 259 | // Check temp scope is gone | 259 | // Check temp scope is gone |
490 | @@ -279,7 +279,7 @@ | |||
491 | 279 | // Go back | 279 | // Go back |
492 | 280 | var scopesList = findChild(dash, "scopesList"); | 280 | var scopesList = findChild(dash, "scopesList"); |
493 | 281 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); | 281 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); |
495 | 282 | var backButton = findChild(findChild(scopesListPageHeader, "innerPageHeader"), "customBackButton"); | 282 | var backButton = findChild(scopesListPageHeader, "innerPageHeader").leadingActionBar; |
496 | 283 | mouseClick(backButton); | 283 | mouseClick(backButton); |
497 | 284 | tryCompare(bottomEdgeController, "progress", 0); | 284 | tryCompare(bottomEdgeController, "progress", 0); |
498 | 285 | } | 285 | } |
499 | @@ -391,7 +391,7 @@ | |||
500 | 391 | // Exit edit mode and go back | 391 | // Exit edit mode and go back |
501 | 392 | var scopesList = findChild(dash, "scopesList"); | 392 | var scopesList = findChild(dash, "scopesList"); |
502 | 393 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); | 393 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); |
504 | 394 | var backButton = findChild(findChild(scopesListPageHeader, "innerPageHeader"), "customBackButton"); | 394 | var backButton = findChild(scopesListPageHeader, "innerPageHeader").leadingActionBar; |
505 | 395 | mouseClick(backButton); | 395 | mouseClick(backButton); |
506 | 396 | mouseClick(backButton); | 396 | mouseClick(backButton); |
507 | 397 | tryCompare(bottomEdgeController, "progress", 0); | 397 | tryCompare(bottomEdgeController, "progress", 0); |
508 | @@ -436,7 +436,7 @@ | |||
509 | 436 | // Exit edit mode | 436 | // Exit edit mode |
510 | 437 | var scopesList = findChild(dash, "scopesList"); | 437 | var scopesList = findChild(dash, "scopesList"); |
511 | 438 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); | 438 | var scopesListPageHeader = findChild(scopesList, "pageHeader"); |
513 | 439 | var backButton = findChild(findChild(scopesListPageHeader, "innerPageHeader"), "customBackButton"); | 439 | var backButton = findChild(scopesListPageHeader, "innerPageHeader").leadingActionBar; |
514 | 440 | mouseClick(backButton); | 440 | mouseClick(backButton); |
515 | 441 | 441 | ||
516 | 442 | // Click on third scope | 442 | // Click on third scope |
517 | @@ -546,7 +546,7 @@ | |||
518 | 546 | 546 | ||
519 | 547 | // Go back | 547 | // Go back |
520 | 548 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); | 548 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); |
522 | 549 | var backButton = findChild(findChild(dashTempScopeItemHeader, "innerPageHeader"), "customBackButton"); | 549 | var backButton = findChild(dashTempScopeItemHeader, "innerPageHeader").leadingActionBar; |
523 | 550 | mouseClick(backButton); | 550 | mouseClick(backButton); |
524 | 551 | 551 | ||
525 | 552 | // Check temp scope is gone | 552 | // Check temp scope is gone |
526 | @@ -614,7 +614,7 @@ | |||
527 | 614 | 614 | ||
528 | 615 | // Go back | 615 | // Go back |
529 | 616 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); | 616 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); |
531 | 617 | var backButton = findChild(findChild(dashTempScopeItemHeader, "innerPageHeader"), "customBackButton"); | 617 | var backButton = findChild(dashTempScopeItemHeader, "innerPageHeader").leadingActionBar; |
532 | 618 | mouseClick(backButton); | 618 | mouseClick(backButton); |
533 | 619 | 619 | ||
534 | 620 | // Check temp scope is gone | 620 | // Check temp scope is gone |
535 | @@ -646,7 +646,7 @@ | |||
536 | 646 | 646 | ||
537 | 647 | // Go back | 647 | // Go back |
538 | 648 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); | 648 | var dashTempScopeItemHeader = findChild(dashTempScopeItem, "scopePageHeader"); |
540 | 649 | var backButton = findChild(findChild(dashTempScopeItemHeader, "innerPageHeader"), "customBackButton"); | 649 | var backButton = findChild(dashTempScopeItemHeader, "innerPageHeader").leadingActionBar; |
541 | 650 | mouseClick(backButton); | 650 | mouseClick(backButton); |
542 | 651 | 651 | ||
543 | 652 | // Check temp scope is gone | 652 | // Check temp scope is gone |
544 | 653 | 653 | ||
545 | === modified file 'tests/qmltests/Dash/tst_DashContent.qml' | |||
546 | --- tests/qmltests/Dash/tst_DashContent.qml 2016-03-29 03:47:39 +0000 | |||
547 | +++ tests/qmltests/Dash/tst_DashContent.qml 2016-05-16 14:05:53 +0000 | |||
548 | @@ -215,7 +215,7 @@ | |||
549 | 215 | verify(pageHeader, "Could not find the scope page header."); | 215 | verify(pageHeader, "Could not find the scope page header."); |
550 | 216 | var innerHeader = findChild(pageHeader, "innerPageHeader"); | 216 | var innerHeader = findChild(pageHeader, "innerPageHeader"); |
551 | 217 | verify(innerHeader, "Could not find the scope page header."); | 217 | verify(innerHeader, "Could not find the scope page header."); |
553 | 218 | compare(innerHeader.config.title, scopesModel.getScope(data.index).name); | 218 | compare(innerHeader.title, scopesModel.getScope(data.index).name); |
554 | 219 | } | 219 | } |
555 | 220 | 220 | ||
556 | 221 | function test_is_active_data() { | 221 | function test_is_active_data() { |
557 | @@ -515,10 +515,18 @@ | |||
558 | 515 | } | 515 | } |
559 | 516 | 516 | ||
560 | 517 | function test_navigationShowFilterPopup() { | 517 | function test_navigationShowFilterPopup() { |
561 | 518 | goToSecondLevel(); | ||
562 | 519 | |||
563 | 520 | var dashContentList = findChild(dashContent, "dashContentList"); | 518 | var dashContentList = findChild(dashContent, "dashContentList"); |
564 | 519 | var pageHeader = findChild(dashContentList.currentItem, "scopePageHeader") | ||
565 | 521 | var settingsButton = findChild(dashContentList.currentItem, "settingsButton"); | 520 | var settingsButton = findChild(dashContentList.currentItem, "settingsButton"); |
566 | 521 | var searchButton = findChild(dashContentList.currentItem, "search_button"); | ||
567 | 522 | var cancelButton = findChild(dashContentList.currentItem, "cancelButton"); | ||
568 | 523 | var searchTextField = findChild(pageHeader, "searchTextField"); | ||
569 | 524 | var peExtraPanel = findChild(dashContentList.currentItem, "peExtraPanel"); | ||
570 | 525 | var headerContainer = findChild(pageHeader, "headerContainer"); | ||
571 | 526 | |||
572 | 527 | // test that closing the filters popover without a search unfocuses and removes the navigation | ||
573 | 528 | mouseClick(searchButton); | ||
574 | 529 | tryCompare(peExtraPanel, "visible", true); | ||
575 | 522 | 530 | ||
576 | 523 | var filtersPopover = findChild(shell, "filtersPopover") | 531 | var filtersPopover = findChild(shell, "filtersPopover") |
577 | 524 | verify(!filtersPopover); | 532 | verify(!filtersPopover); |
578 | @@ -526,8 +534,33 @@ | |||
579 | 526 | mouseClick(settingsButton); | 534 | mouseClick(settingsButton); |
580 | 527 | 535 | ||
581 | 528 | filtersPopover = findChild(shell, "filtersPopover") | 536 | filtersPopover = findChild(shell, "filtersPopover") |
584 | 529 | 537 | verify(filtersPopover); | |
585 | 530 | verify(filtersPopover); | 538 | |
586 | 539 | mouseClick(shell, shell.width - 1, shell.height - 1); | ||
587 | 540 | |||
588 | 541 | tryCompare(pageHeader.extraPanel, "visible", false); | ||
589 | 542 | tryCompare(headerContainer, "showSearch", true); | ||
590 | 543 | tryCompare(searchTextField, "focus", false); | ||
591 | 544 | |||
592 | 545 | mouseClick(cancelButton); | ||
593 | 546 | tryCompare(headerContainer, "showSearch", false); | ||
594 | 547 | tryCompare(headerContainer, "contentY", headerContainer.height); | ||
595 | 548 | |||
596 | 549 | // test within a navigation | ||
597 | 550 | goToSecondLevel(); | ||
598 | 551 | |||
599 | 552 | filtersPopover = findChild(shell, "filtersPopover") | ||
600 | 553 | verify(!filtersPopover); | ||
601 | 554 | |||
602 | 555 | mouseClick(settingsButton); | ||
603 | 556 | |||
604 | 557 | filtersPopover = findChild(shell, "filtersPopover") | ||
605 | 558 | verify(filtersPopover); | ||
606 | 559 | |||
607 | 560 | mouseClick(shell, shell.width - 1, shell.height - 1); | ||
608 | 561 | |||
609 | 562 | tryCompare(pageHeader.extraPanel, "visible", false); | ||
610 | 563 | tryCompare(headerContainer, "showSearch", true); | ||
611 | 531 | } | 564 | } |
612 | 532 | 565 | ||
613 | 533 | function test_primaryFilter() { | 566 | function test_primaryFilter() { |
614 | @@ -625,5 +658,31 @@ | |||
615 | 625 | tryCompareFunction(function() { return dashContentList.currentItem.item.scope.id == nextScopeId; }, true); | 658 | tryCompareFunction(function() { return dashContentList.currentItem.item.scope.id == nextScopeId; }, true); |
616 | 626 | compare(dashContentList.currentIndex, 0); | 659 | compare(dashContentList.currentIndex, 0); |
617 | 627 | } | 660 | } |
618 | 661 | |||
619 | 662 | function test_extraPanel() { | ||
620 | 663 | var dashContentList = findChild(dashContent, "dashContentList"); | ||
621 | 664 | var pageHeader = findChild(dashContentList.currentItem, "scopePageHeader") | ||
622 | 665 | pageHeader.searchEntryEnabled = true; | ||
623 | 666 | pageHeader.searchHistory.clear(); | ||
624 | 667 | |||
625 | 668 | pageHeader.searchHistory.addQuery("Search1"); | ||
626 | 669 | pageHeader.searchHistory.addQuery("Search2"); | ||
627 | 670 | |||
628 | 671 | pageHeader.triggerSearch(); | ||
629 | 672 | tryCompare(pageHeader.extraPanel, "visible", true); | ||
630 | 673 | |||
631 | 674 | var searchTextField = findChild(pageHeader, "searchTextField"); | ||
632 | 675 | compare(searchTextField.focus, true); | ||
633 | 676 | |||
634 | 677 | var recentSearches = findChild(pageHeader.extraPanel, "recentSearchesRepeater"); | ||
635 | 678 | verify(recentSearches, "Could not find recent searches"); | ||
636 | 679 | |||
637 | 680 | waitForRendering(recentSearches); | ||
638 | 681 | |||
639 | 682 | mouseClick(recentSearches.itemAt(0)); | ||
640 | 683 | compare(pageHeader.searchQuery, "Search2"); | ||
641 | 684 | tryCompare(pageHeader.extraPanel, "visible", false); | ||
642 | 685 | compare(searchTextField.focus, false); | ||
643 | 686 | } | ||
644 | 628 | } | 687 | } |
645 | 629 | } | 688 | } |
646 | 630 | 689 | ||
647 | === modified file 'tests/qmltests/Dash/tst_DashPageHeader.qml' | |||
648 | --- tests/qmltests/Dash/tst_DashPageHeader.qml 2016-03-29 03:47:39 +0000 | |||
649 | +++ tests/qmltests/Dash/tst_DashPageHeader.qml 2016-05-16 14:05:53 +0000 | |||
650 | @@ -147,30 +147,6 @@ | |||
651 | 147 | doResetSearch(); | 147 | doResetSearch(); |
652 | 148 | } | 148 | } |
653 | 149 | 149 | ||
654 | 150 | function test_extraPanel() { | ||
655 | 151 | searchEnabled = true; | ||
656 | 152 | pageHeader.searchHistory.clear(); | ||
657 | 153 | |||
658 | 154 | pageHeader.searchHistory.addQuery("Search1"); | ||
659 | 155 | pageHeader.searchHistory.addQuery("Search2"); | ||
660 | 156 | |||
661 | 157 | pageHeader.triggerSearch(); | ||
662 | 158 | |||
663 | 159 | tryCompare(pageHeader.extraPanel, "visible", true); | ||
664 | 160 | |||
665 | 161 | var searchTextField = findChild(pageHeader, "searchTextField"); | ||
666 | 162 | compare(searchTextField.focus, true); | ||
667 | 163 | |||
668 | 164 | var recentSearches = findChild(pageHeader.extraPanel, "recentSearchesRepeater"); | ||
669 | 165 | verify(recentSearches, "Could not find recent searches"); | ||
670 | 166 | waitForRendering(recentSearches); | ||
671 | 167 | mouseClick(recentSearches.itemAt(0)); | ||
672 | 168 | |||
673 | 169 | compare(pageHeader.searchQuery, "Search2"); | ||
674 | 170 | tryCompare(pageHeader.extraPanel, "visible", false); | ||
675 | 171 | compare(searchTextField.focus, false); | ||
676 | 172 | } | ||
677 | 173 | |||
678 | 174 | function test_popup_closing_data() { | 150 | function test_popup_closing_data() { |
679 | 175 | return [ | 151 | return [ |
680 | 176 | { tag: "with search text", searchText: "foobar", hideSearch: false }, | 152 | { tag: "with search text", searchText: "foobar", hideSearch: false }, |
681 | 177 | 153 | ||
682 | === modified file 'tests/qmltests/Dash/tst_GenericScopeView.qml' | |||
683 | --- tests/qmltests/Dash/tst_GenericScopeView.qml 2016-03-29 03:47:39 +0000 | |||
684 | +++ tests/qmltests/Dash/tst_GenericScopeView.qml 2016-05-16 14:05:53 +0000 | |||
685 | @@ -442,8 +442,8 @@ | |||
686 | 442 | 442 | ||
687 | 443 | var innerHeader = findChild(header, "innerPageHeader"); | 443 | var innerHeader = findChild(header, "innerPageHeader"); |
688 | 444 | verify(innerHeader, "Could not find the inner header"); | 444 | verify(innerHeader, "Could not find the inner header"); |
691 | 445 | verify(Qt.colorEqual(innerHeader.config.foregroundColor, data.foreground), | 445 | verify(Qt.colorEqual(innerHeader.__styleInstance.foregroundColor, data.foreground), |
692 | 446 | "Foreground color not equal: %1 != %2".arg(innerHeader.config.foregroundColor).arg(data.foreground)); | 446 | "Foreground color not equal: %1 != %2".arg(innerHeader.__styleInstance.foregroundColor).arg(data.foreground)); |
693 | 447 | 447 | ||
694 | 448 | var background = findChild(header, "headerBackground"); | 448 | var background = findChild(header, "headerBackground"); |
695 | 449 | verify(background, "Could not find the background"); | 449 | verify(background, "Could not find the background"); |
696 | 450 | 450 | ||
697 | === modified file 'tests/qmltests/Dash/tst_PreviewView.qml' | |||
698 | --- tests/qmltests/Dash/tst_PreviewView.qml 2016-03-29 03:47:39 +0000 | |||
699 | +++ tests/qmltests/Dash/tst_PreviewView.qml 2016-05-16 14:05:53 +0000 | |||
700 | @@ -98,7 +98,7 @@ | |||
701 | 98 | var header = findChild(view, "innerPageHeader"); | 98 | var header = findChild(view, "innerPageHeader"); |
702 | 99 | verify(header, "Could not find the preview header"); | 99 | verify(header, "Could not find the preview header"); |
703 | 100 | 100 | ||
705 | 101 | compare(header.config.title, "Mock Scope"); | 101 | compare(header.title, "Mock Scope"); |
706 | 102 | } | 102 | } |
707 | 103 | 103 | ||
708 | 104 | function test_header_style() { | 104 | function test_header_style() { |
709 | @@ -107,8 +107,8 @@ | |||
710 | 107 | 107 | ||
711 | 108 | var innerHeader = findChild(header, "innerPageHeader"); | 108 | var innerHeader = findChild(header, "innerPageHeader"); |
712 | 109 | verify(innerHeader, "Could not find the inner header"); | 109 | verify(innerHeader, "Could not find the inner header"); |
715 | 110 | verify(Qt.colorEqual(innerHeader.config.foregroundColor, UbuntuColors.darkGrey), | 110 | verify(Qt.colorEqual(innerHeader.__styleInstance.foregroundColor, UbuntuColors.darkGrey), |
716 | 111 | "Foreground color not equal: %1 != %2".arg(innerHeader.config.foregroundColor).arg(UbuntuColors.darkGrey)); | 111 | "Foreground color not equal: %1 != %2".arg(innerHeader.__styleInstance.foregroundColor).arg(UbuntuColors.darkGrey)); |
717 | 112 | 112 | ||
718 | 113 | var background = findChild(header, "headerBackground"); | 113 | var background = findChild(header, "headerBackground"); |
719 | 114 | verify(background, "Could not find the background"); | 114 | verify(background, "Could not find the background"); |
FAILED: Continuous integration, rev:2396 /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/1203/ /unity8- jenkins. ubuntu. com/job/ test-0- autopkgtest/ label=amd64, release= vivid+overlay, testname= qmluitests. sh/740 /unity8- jenkins. ubuntu. com/job/ test-0- autopkgtest/ label=amd64, release= xenial+ overlay, testname= qmluitests. sh/740 /unity8- jenkins. ubuntu. com/job/ build-0- fetch/1612 /unity8- jenkins. ubuntu. com/job/ build-1- sourcepkg/ release= vivid+overlay/ 1567 /unity8- jenkins. ubuntu. com/job/ build-1- sourcepkg/ release= xenial+ overlay/ 1567 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 1560 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 1560/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 1560 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 1560/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 1560 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 1560/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 1560 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 1560/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 1560 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 1560/artifact/ output/ *zip*/output. zip /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 1560 /unity8- jenkins. ubuntu. com/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 1560/artifact/ output/ *zip*/output. zip
https:/
Executed test runs:
UNSTABLE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild: /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/1203/ rebuild
https:/