Merge lp:~tpeeters/ubuntu-ui-toolkit/apl-dont-change-page-geometry into lp:ubuntu-ui-toolkit/staging
- apl-dont-change-page-geometry
- Merge into staging
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 |
Related bugs: |
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.
Description of the change
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
SUCCESS: https:/
FAILURE: https:/
SUCCESS: https:/
FAILURE: https:/
None: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
None: https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
None: https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
None: https:/
SUCCESS: https:/
SUCCESS: https:/
None: https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1894
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild:
https:/
Preview Diff
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 |
Good to go, thanks!!!