Merge lp:~tpeeters/ubuntu-ui-toolkit/apl-dont-change-page-geometry into lp:ubuntu-ui-toolkit/staging

Proposed by Tim Peeters
Status: Merged
Approved by: Timo Jyrinki
Approved revision: 1894
Merged at revision: 1897
Proposed branch: lp:~tpeeters/ubuntu-ui-toolkit/apl-dont-change-page-geometry
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 32 lines (+14/-1)
1 file modified
src/Ubuntu/Components/1.3/AdaptivePageLayout.qml (+14/-1)
To merge this branch: bzr merge lp:~tpeeters/ubuntu-ui-toolkit/apl-dont-change-page-geometry
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+289054@code.launchpad.net

Commit message

Prevent Page geometry changes in APL when changing Pages in a column.

To post a comment you must log in.
Revision history for this message
Zsombor Egri (zsombi) wrote :

Good to go, thanks!!!

review: Approve
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-autolanding/201/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-land-mp/206/console

review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Ubuntu/Components/1.3/AdaptivePageLayout.qml'
2--- src/Ubuntu/Components/1.3/AdaptivePageLayout.qml 2016-03-03 10:42:53 +0000
3+++ src/Ubuntu/Components/1.3/AdaptivePageLayout.qml 2016-03-15 14:41:12 +0000
4@@ -754,6 +754,19 @@
5 property var page: pageWrapper ? pageWrapper.object : null
6 property bool customHeader: page && page.hasOwnProperty("header") &&
7 page.header
8+ onCustomHeaderChanged: {
9+ // do not change the holderBody anchors until the new page
10+ // for the column has been set to prevent geometry changes in
11+ // the current/previous page.
12+ if (page) {
13+ if (customHeader) {
14+ holderBody.anchors.top = holderBody.parent.top;
15+ } else {
16+ holderBody.anchors.top = subHeader.bottom;
17+ }
18+ }
19+ }
20+
21 onPageChanged: body.updateHeaderHeight(0)
22 Connections {
23 target: page
24@@ -770,7 +783,7 @@
25 id: holderBody
26 objectName: parent.objectName + "Body"
27 anchors {
28- top: customHeader ? parent.top : subHeader.bottom
29+ top: subHeader.bottom // updated in onCustomHeaderChanged
30 bottom: parent.bottom
31 left: parent.left
32 right: parent.right

Subscribers

People subscribed via source and target branches