Merge lp:~zsombi/ubuntu-ui-toolkit/fixAplColumnDivider into lp:ubuntu-ui-toolkit/staging
- fixAplColumnDivider
- Merge into staging
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Tim Peeters | ||||
Approved revision: | 1964 | ||||
Merged at revision: | 1975 | ||||
Proposed branch: | lp:~zsombi/ubuntu-ui-toolkit/fixAplColumnDivider | ||||
Merge into: | lp:ubuntu-ui-toolkit/staging | ||||
Diff against target: |
51 lines (+4/-6) 2 files modified
src/Ubuntu/Components/1.3/AdaptivePageLayout.qml (+4/-3) src/Ubuntu/Components/plugin/uclistitem.cpp (+0/-3) |
||||
To merge this branch: | bzr merge lp:~zsombi/ubuntu-ui-toolkit/fixAplColumnDivider | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
ubuntu-sdk-build-bot | continuous-integration | Approve | |
Tim Peeters | Approve | ||
Review via email: mp+293058@code.launchpad.net |
Commit message
AdaptivePageLayout vertical divider occludes with ListItem's visuals.
Description of the change
AdaptivePageLayout vertical divider occludes with ListItem's visuals.
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1961
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:1961
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:1961
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:1961
https:/
Executed test runs:
SUCCESS: https:/
deb: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
None: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild:
https:/
Tim Peeters (tpeeters) wrote : | # |
CI has failures on tst_adaptivepag
http://
- 1962. By Zsombor Egri
-
divider hides based on the column count
- 1963. By Zsombor Egri
-
staging sync
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
FAILED: Continuous integration, rev:1963
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:1963
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:1963
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:1963
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:1963
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
None: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild:
https:/
- 1964. By Zsombor Egri
-
staging sync
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1964
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:1964
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:1964
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:1964
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:1964
https:/
Executed test runs:
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
None: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
Click here to trigger a rebuild:
https:/
Tim Peeters (tpeeters) wrote : | # |
Looks good. Thanks.
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : | # |
PASSED: Continuous integration, rev:1964
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:1964
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:1964
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:1964
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-16 09:15:39 +0000 | |||
3 | +++ src/Ubuntu/Components/1.3/AdaptivePageLayout.qml 2016-05-10 07:10:08 +0000 | |||
4 | @@ -788,7 +788,7 @@ | |||
5 | 788 | bottom: parent.bottom | 788 | bottom: parent.bottom |
6 | 789 | left: parent.left | 789 | left: parent.left |
7 | 790 | right: parent.right | 790 | right: parent.right |
9 | 791 | rightMargin: dividerThickness | 791 | rightMargin: dividerThickness + verticalDivider.width |
10 | 792 | } | 792 | } |
11 | 793 | Item { | 793 | Item { |
12 | 794 | id: hiddenItem | 794 | id: hiddenItem |
13 | @@ -806,6 +806,7 @@ | |||
14 | 806 | left: parent.left | 806 | left: parent.left |
15 | 807 | top: parent.top | 807 | top: parent.top |
16 | 808 | right: parent.right | 808 | right: parent.right |
17 | 809 | rightMargin: dividerThickness + verticalDivider.width | ||
18 | 809 | } | 810 | } |
19 | 810 | height: body.headerHeight | 811 | height: body.headerHeight |
20 | 811 | 812 | ||
21 | @@ -856,7 +857,7 @@ | |||
22 | 856 | right: parent.right | 857 | right: parent.right |
23 | 857 | rightMargin: dividerThickness | 858 | rightMargin: dividerThickness |
24 | 858 | } | 859 | } |
26 | 859 | width: (column == (d.columns - 1)) || !pageWrapper ? 0 : units.dp(1) | 860 | width: (column == (d.columns - 1)) ? 0 : units.dp(1) |
27 | 860 | color: theme.palette.normal.base | 861 | color: theme.palette.normal.base |
28 | 861 | MouseArea { | 862 | MouseArea { |
29 | 862 | id: resizerSensing | 863 | id: resizerSensing |
30 | @@ -923,7 +924,7 @@ | |||
31 | 923 | wrapper.active = false; | 924 | wrapper.active = false; |
32 | 924 | subHeader.config = null; | 925 | subHeader.config = null; |
33 | 925 | pageWrapper = null; | 926 | pageWrapper = null; |
35 | 926 | wrapper.parent = hiddenPool; | 927 | wrapper.parent = hiddenItem; |
36 | 927 | wrapper.pageHolder = null; | 928 | wrapper.pageHolder = null; |
37 | 928 | return wrapper; | 929 | return wrapper; |
38 | 929 | } | 930 | } |
39 | 930 | 931 | ||
40 | === modified file 'src/Ubuntu/Components/plugin/uclistitem.cpp' | |||
41 | --- src/Ubuntu/Components/plugin/uclistitem.cpp 2016-04-26 10:21:32 +0000 | |||
42 | +++ src/Ubuntu/Components/plugin/uclistitem.cpp 2016-05-10 07:10:08 +0000 | |||
43 | @@ -1140,9 +1140,6 @@ | |||
44 | 1140 | updateNode = true; | 1140 | updateNode = true; |
45 | 1141 | } | 1141 | } |
46 | 1142 | QRectF rect(boundingRect()); | 1142 | QRectF rect(boundingRect()); |
47 | 1143 | // FIXME: The 1dp margin is here so that part of the focus frame | ||
48 | 1144 | // is not hidden by the APL divider. See bug #1575060. | ||
49 | 1145 | rect -= QMarginsF(0, 0, UCUnits::instance()->dp(1), 0); | ||
50 | 1146 | d->divider->setOpacity(paintFocus ? 0.0 : 1.0); | 1143 | d->divider->setOpacity(paintFocus ? 0.0 : 1.0); |
51 | 1147 | rectNode->setRect(rect); | 1144 | rectNode->setRect(rect); |
52 | 1148 | 1145 |
FAILED: Continuous integration, rev:1961 /jenkins. ubuntu. com/ubuntu- sdk/job/ ubuntu- ui-toolkit- ci-amd64- devel/540/ /jenkins. ubuntu. com/ubuntu- sdk/job/ generic- update- mp/2981/ console
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild: /jenkins. ubuntu. com/ubuntu- sdk/job/ ubuntu- ui-toolkit- ci-amd64- devel/540/ rebuild
https:/