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
=== modified file 'qml/Dash/ScopesList.qml'
--- qml/Dash/ScopesList.qml 2014-12-02 12:04:11 +0000
+++ qml/Dash/ScopesList.qml 2014-12-15 14:25:10 +0000
@@ -34,6 +34,9 @@
3434
35 state: "browse"35 state: "browse"
3636
37 property var scopeStyle: ScopeStyle {
38 }
39
37 onStateChanged: {40 onStateChanged: {
38 if (state == "edit") {41 if (state == "edit") {
39 // As per design entering edit mode clears the possible existing search42 // As per design entering edit mode clears the possible existing search
@@ -66,6 +69,7 @@
66 backIsClose: root.state == "edit"69 backIsClose: root.state == "edit"
67 storeEntryEnabled: root.state == "browse"70 storeEntryEnabled: root.state == "browse"
68 searchEntryEnabled: false // Disable search for now71 searchEntryEnabled: false // Disable search for now
72 scopeStyle: root.scopeStyle
69 onBackClicked: {73 onBackClicked: {
70 if (backIsClose) {74 if (backIsClose) {
71 root.state = "browse"75 root.state = "browse"
@@ -111,7 +115,7 @@
111115
112 editMode: root.state == "edit"116 editMode: root.state == "edit"
113117
114 scopeStyle: root.scope.scopeStyle118 scopeStyle: root.scopeStyle
115 isFavoritesFeed: categoryId == "favorites"119 isFavoritesFeed: categoryId == "favorites"
116 isAlsoInstalled: categoryId == "other"120 isAlsoInstalled: categoryId == "other"
117121

Subscribers

People subscribed via source and target branches