head sections on the left side of an AdaptivePageLayout 'leak' onto the right side instead of scrolling
Bug #1489591 reported by
Andrew Hayzen
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
High
|
Zoltan Balogh | ||
ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
High
|
Tim Peeters |
Bug Description
When using many head sections on a page in the left side of an AdaptivePageLayout they 'leak' into the right side instead of scrolling, an example can be found here [0]
Related branches
lp:~tpeeters/ubuntu-ui-toolkit/60-scectionScrolling
- ubuntu-sdk-build-bot: Approve (continuous-integration)
- Cris Dywan: Approve
- PS Jenkins bot: Pending (continuous-integration) requested
-
Diff: 1299 lines (+717/-150)15 files modifiedexamples/ubuntu-ui-toolkit-gallery/PageHeaders.qml (+27/-0)
examples/ubuntu-ui-toolkit-gallery/Sections.qml (+62/-14)
src/Ubuntu/Components/1.3/Sections.qml (+0/-4)
src/Ubuntu/Components/Themes/Ambiance/1.3/SectionsForPageHead.qml (+2/-3)
src/Ubuntu/Components/Themes/Ambiance/1.3/SectionsForPageHeadStyle.qml (+122/-0)
src/Ubuntu/Components/Themes/Ambiance/1.3/SectionsStyle.qml (+307/-36)
src/Ubuntu/Components/Themes/Ambiance/Ambiance.pro (+1/-0)
src/Ubuntu/Components/Themes/Ambiance/qmldir (+1/-0)
tests/Gallery/gallery (+0/-6)
tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_sections.py (+11/-12)
tests/autopilot/ubuntuuitoolkit/tests/custom_proxy_objects/test_sections.SectionsTestCase.qml (+36/-4)
tests/autopilot/ubuntuuitoolkit/tests/custom_proxy_objects/test_sections.py (+25/-1)
tests/unit_x11/tst_components/tst_pagehead_sections.qml (+1/-1)
tests/unit_x11/tst_components/tst_pagehead_sections_bug1511839.qml (+1/-1)
tests/unit_x11/tst_components/tst_sections.qml (+121/-68)
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | New → Confirmed |
importance: | Undecided → High |
assignee: | nobody → Tim Peeters (tpeeters) |
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | Confirmed → In Progress |
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | New → Fix Committed |
importance: | Undecided → High |
assignee: | nobody → Zoltan Balogh (bzoltan) |
milestone: | none → ww08-2016 |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
I bumped into this bug some days ago.
Actually the issue seems more generic: all the pages should be clipped when pushed inside the AdaptivePageLayout.
I wrote an "interactive" example (attached to this comment):
Swipe the ListItem to left, swipe it back to the original position, hit the button (push a page in the second column) and swipe the new ListItem to right.
Each page will occupy the space that in theory is reserved to the other page.
This is easily workaround-able by adding 'clip: true' in the QML document that contains a page. (pageWrapper) " function[1].
However I think that AdaptivePageLayout should do it by its own, in its "addWrappedPage
[1] Since the issue is due to the particular layout applied to the Pages; and AdaptivePageLayout already changes the settings of the Page when it appends the 'Back' action to the header.