Merge lp:~paulliu/unity8/divider20140729 into lp:unity8

Proposed by Ying-Chun Liu
Status: Merged
Approved by: Michał Sawicz
Approved revision: 1092
Merged at revision: 1119
Proposed branch: lp:~paulliu/unity8/divider20140729
Merge into: lp:unity8
Diff against target: 100 lines (+46/-0)
4 files modified
qml/Components/PageHeader.qml (+21/-0)
qml/Dash/DashContent.qml (+2/-0)
qml/Dash/GenericScopeView.qml (+2/-0)
tests/qmltests/Components/tst_PageHeader.qml (+21/-0)
To merge this branch: bzr merge lp:~paulliu/unity8/divider20140729
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Michał Sawicz Approve
Albert Astals Cid (community) Abstain
Review via email: mp+228708@code.launchpad.net

Commit message

Add divider dots.

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?
Need someone to review.

To post a comment you must log in.
Revision history for this message
Ying-Chun Liu (paulliu) wrote :
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

See inline.

review: Needs Fixing
lp:~paulliu/unity8/divider20140729 updated
1089. By Ying-Chun Liu

Rename dividerDots* to pagination*.
Remove some unused id.

1090. By Ying-Chun Liu

Merge trunk

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
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
Albert Astals Cid (aacid) wrote :

Text conflict in tests/qmltests/Components/tst_PageHeader.qml
1 conflicts encountered.

review: Needs Fixing
lp:~paulliu/unity8/divider20140729 updated
1091. By Ying-Chun Liu

merge trunk

1092. By Ying-Chun Liu

rename png

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
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
Albert Astals Cid (aacid) wrote :

Code merges now

review: Abstain
Revision history for this message
Michał Sawicz (saviq) 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, restarted.

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'qml/Components/PageHeader.qml'
--- qml/Components/PageHeader.qml 2014-07-31 16:49:50 +0000
+++ qml/Components/PageHeader.qml 2014-08-01 11:23:19 +0000
@@ -35,6 +35,8 @@
35 property bool searchInProgress: false35 property bool searchInProgress: false
3636
37 property alias bottomItem: bottomContainer.children37 property alias bottomItem: bottomContainer.children
38 property int paginationCount: 0
39 property int paginationIndex: -1
3840
39 // TODO We should use foreground for the icons41 // TODO We should use foreground for the icons
40 // of the toolbar but unfortunately Action does not have42 // of the toolbar but unfortunately Action does not have
@@ -294,6 +296,25 @@
294 }296 }
295 }297 }
296298
299 Row {
300 spacing: units.gu(.5)
301 Repeater {
302 objectName: "paginationRepeater"
303 model: root.paginationCount
304 Image {
305 objectName: "paginationDots_" + index
306 height: units.gu(1)
307 width: height
308 source: (index == root.paginationIndex) ? "graphics/pagination_dot_on.png" : "graphics/pagination_dot_off.png"
309 }
310 }
311 anchors {
312 top: headerContainer.bottom
313 horizontalCenter: headerContainer.horizontalCenter
314 topMargin: units.gu(.5)
315 }
316 }
317
297 Component {318 Component {
298 id: popoverComponent319 id: popoverComponent
299 Popover {320 Popover {
300321
=== added file 'qml/Components/graphics/pagination_dot_off@30.png'
301Binary files qml/Components/graphics/pagination_dot_off@30.png 1970-01-01 00:00:00 +0000 and qml/Components/graphics/pagination_dot_off@30.png 2014-08-01 11:23:19 +0000 differ322Binary files qml/Components/graphics/pagination_dot_off@30.png 1970-01-01 00:00:00 +0000 and qml/Components/graphics/pagination_dot_off@30.png 2014-08-01 11:23:19 +0000 differ
=== added file 'qml/Components/graphics/pagination_dot_on@30.png'
302Binary files qml/Components/graphics/pagination_dot_on@30.png 1970-01-01 00:00:00 +0000 and qml/Components/graphics/pagination_dot_on@30.png 2014-08-01 11:23:19 +0000 differ323Binary files qml/Components/graphics/pagination_dot_on@30.png 1970-01-01 00:00:00 +0000 and qml/Components/graphics/pagination_dot_on@30.png 2014-08-01 11:23:19 +0000 differ
=== modified file 'qml/Dash/DashContent.qml'
--- qml/Dash/DashContent.qml 2014-07-24 23:31:44 +0000
+++ qml/Dash/DashContent.qml 2014-08-01 11:23:19 +0000
@@ -143,6 +143,8 @@
143 item.scope = Qt.binding(function() { return scope })143 item.scope = Qt.binding(function() { return scope })
144 item.isCurrent = Qt.binding(function() { return visible && ListView.isCurrentItem })144 item.isCurrent = Qt.binding(function() { return visible && ListView.isCurrentItem })
145 dashContent.scopeLoaded(item.scope.id)145 dashContent.scopeLoaded(item.scope.id)
146 item.paginationCount = Qt.binding(function() { return dashContentList.count } )
147 item.paginationIndex = Qt.binding(function() { return dashContentList.currentIndex } )
146 }148 }
147 Connections {149 Connections {
148 target: isCurrent ? scope : null150 target: isCurrent ? scope : null
149151
=== modified file 'qml/Dash/GenericScopeView.qml'
--- qml/Dash/GenericScopeView.qml 2014-07-25 11:49:28 +0000
+++ qml/Dash/GenericScopeView.qml 2014-08-01 11:23:19 +0000
@@ -32,6 +32,8 @@
32 property bool hasBackAction: false32 property bool hasBackAction: false
33 property bool enableHeightBehaviorOnNextCreation: false33 property bool enableHeightBehaviorOnNextCreation: false
34 property var categoryView: categoryView34 property var categoryView: categoryView
35 property alias paginationCount: pageHeader.paginationCount
36 property alias paginationIndex: pageHeader.paginationIndex
3537
36 property var scopeStyle: ScopeStyle {38 property var scopeStyle: ScopeStyle {
37 style: scope ? scope.customizations : {}39 style: scope ? scope.customizations : {}
3840
=== 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-01 11:23:19 +0000
@@ -192,6 +192,27 @@
192 ];192 ];
193 }193 }
194194
195 function test_pagination() {
196 var paginationRepeater = findChild(pageHeader, "paginationRepeater");
197 tryCompare(paginationRepeater, "count", 0);
198 pageHeader.paginationCount = 5;
199 tryCompare(paginationRepeater, "count", 5);
200 for (var i=0; i<pageHeader.paginationCount; i++) {
201 pageHeader.paginationIndex = i;
202 for (var j=0; j<paginationRepeater.count; j++) {
203 var paginationDot = findChild(pageHeader, "paginationDots_"+j);
204 if (i==j) {
205 compare(paginationDot.source.toString().indexOf("pagination_dot_on") > -1, true);
206 } else {
207 compare(paginationDot.source.toString().indexOf("pagination_dot_off") > -1, true);
208 }
209 }
210 }
211 pageHeader.paginationIndex = -1;
212 pageHeader.paginationCount = 0;
213 tryCompare(paginationRepeater, "count", 0);
214 }
215
195 function test_popup_closing(data) {216 function test_popup_closing(data) {
196 searchEnabled = true;217 searchEnabled = true;
197 pageHeader.searchHistory.clear();218 pageHeader.searchHistory.clear();

Subscribers

People subscribed via source and target branches