Merge lp:~aacid/unity8/scopeListPageHeaderScopeStyle into lp:unity8

Proposed by Albert Astals Cid
Status: Superseded
Proposed branch: lp:~aacid/unity8/scopeListPageHeaderScopeStyle
Merge into: lp:unity8
Diff against target: 29 lines (+5/-1)
1 file modified
qml/Dash/ScopesList.qml (+5/-1)
To merge this branch: bzr merge lp:~aacid/unity8/scopeListPageHeaderScopeStyle
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Michael Zanetti (community) Approve
Review via email: mp+244737@code.launchpad.net

This proposal has been superseded by a proposal from 2014-12-15.

Commit message

ScopesList: Use the default scope style for the header

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, headers are dark grey now

 * 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?
N/A

To post a comment you must log in.
Revision history for this message
Michael Zanetti (mzanetti) 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.

not yet

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

yes

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Unmerged revisions

1506. By Albert Astals Cid

Use the default scope style for the header

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qml/Dash/ScopesList.qml'
2--- qml/Dash/ScopesList.qml 2014-12-02 12:04:11 +0000
3+++ qml/Dash/ScopesList.qml 2014-12-15 14:25:10 +0000
4@@ -34,6 +34,9 @@
5
6 state: "browse"
7
8+ property var scopeStyle: ScopeStyle {
9+ }
10+
11 onStateChanged: {
12 if (state == "edit") {
13 // As per design entering edit mode clears the possible existing search
14@@ -66,6 +69,7 @@
15 backIsClose: root.state == "edit"
16 storeEntryEnabled: root.state == "browse"
17 searchEntryEnabled: false // Disable search for now
18+ scopeStyle: root.scopeStyle
19 onBackClicked: {
20 if (backIsClose) {
21 root.state = "browse"
22@@ -111,7 +115,7 @@
23
24 editMode: root.state == "edit"
25
26- scopeStyle: root.scope.scopeStyle
27+ scopeStyle: root.scopeStyle
28 isFavoritesFeed: categoryId == "favorites"
29 isAlsoInstalled: categoryId == "other"
30

Subscribers

People subscribed via source and target branches