When changing Page.header, set previous header.parent to null.
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical System Image |
Fix Released
|
High
|
Zoltan Balogh | ||
ubuntu-ui-toolkit (Ubuntu) |
Fix Released
|
High
|
Tim Peeters | ||
ubuntu-ui-toolkit (Ubuntu RTM) |
Fix Released
|
High
|
Tim Peeters |
Bug Description
Currently when Page.header = header1, and then you set Page.header = header2, header1.parent will be reverted to the parent that header1 had before it was assigned to Page.header (and thus re-parented to that page).
Motivation: You want only one header at the same time to touch the topMargins of a flickable. We will not set the flickable margins for a header that is not visible or has parent null. So unsetting the parent when the header is no longer active will revert the flickable margins. Here we choose to change header.parent because header.visible may have other bindings that we do not need to break this way.
Note: There will be no ownership transfer. We are changing the parentItem, not the QObject parent.
Related branches
- ubuntu-sdk-build-bot: Approve (continuous-integration)
- Zsombor Egri: Approve
-
Diff: 410 lines (+112/-58)6 files modifiedsrc/Ubuntu/Components/1.3/Page.qml (+1/-4)
src/Ubuntu/Components/plugin/ucheader.cpp (+30/-22)
src/Ubuntu/Components/plugin/ucheader.h (+2/-2)
tests/unit_x11/tst_components/tst_header.qml (+63/-4)
tests/unit_x11/tst_components/tst_page_with_header.qml (+11/-13)
tests/unit_x11/tst_components/tst_scrollbar_header.qml (+5/-13)
Changed in ubuntu-ui-toolkit (Ubuntu): | |
importance: | Undecided → High |
assignee: | nobody → Tim Peeters (tpeeters) |
description: | updated |
description: | updated |
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | New → In Progress |
Changed in canonical-devices-system-image: | |
assignee: | nobody → Zoltan Balogh (bzoltan) |
importance: | Undecided → High |
milestone: | none → 11 |
Changed in ubuntu-ui-toolkit (Ubuntu RTM): | |
status: | New → Fix Committed |
importance: | Undecided → High |
Changed in ubuntu-ui-toolkit (Ubuntu): | |
status: | In Progress → Fix Committed |
Changed in canonical-devices-system-image: | |
status: | New → In Progress |
Changed in canonical-devices-system-image: | |
status: | In Progress → Fix Committed |
Changed in ubuntu-ui-toolkit (Ubuntu RTM): | |
assignee: | nobody → Tim Peeters (tpeeters) |
Changed in canonical-devices-system-image: | |
status: | Fix Committed → Fix Released |
This bug was fixed in the package ubuntu-ui-toolkit 1.3.1938+ 15.04.20160412 in https:/ /launchpad. net/~ci- train-ppa- service/ +archive/ ubuntu/ stable- phone-overlay
---------------
ubuntu-ui-toolkit (1.3.1938+ 15.04.20160412) vivid; urgency=medium
[ Tim Peeters ]
* Hide the PageStack back button when depth == 1. Fixes LP: #1565811
* Add header subtitle. Fixes LP: #1399289
* Reveal the header in gallery when changing the layout to two columns.
Fixes LP: #1556860
* Set correct colors for disabled actions in the header.
Fixes LP: #1393485
* Disable tst_datepicker.qml to unblock the staging.Fixes LP: #1567840
* Prevent invisible header from setting the flickable topMargin.
Fixes LP: #1560419, LP: #1560458, LP: #1566231.
[ Zsombor Egri ]
* Move MouseTouchAdaptor into UbuntuToolkit library. Fix adaptor code for
Xenial. Fixes LP: #1561436
[ Christian Dywan ]
* Don't use a separate argument to distinguish touch events. If it's touch,
it has our overloaded methods. Fixes LP: #1530802
* Use export_qml_dir.sh in qmlapicheck and runtest Also add Usage to runtest.
Fixes LP: #1567286.
* Summary style error results with sections. Fixes LP: #1568804
[ Andrea Bernabei ]
* Mouse filter: check if mouse is inside the area on mouse moves and
setHovered accordingly. Fixes LP: #1566378.
[ Timo Jyrinki ] ubuntu- ui-toolkit- plugin to resolve convert_ to_gles: sort and add module- ubuntu- performancemetr ics-gles. install. Fixes LP: #1569217
* Add latest changes for GLES.
* Add back Provides: qtdeclarative5-
upgrade issue. Fixes LP: #1568817
* Fix wrapper script auto-generated by Qt that incorrectly tries to execute
itself. Fixes LP: #1560000
* bileto_
qml-
* Add additional Breaks as requested by archive admin.
* Add latest changes for GLES.
[ Zolán Balogh ]
* Fix the UITK test plan script.
[ CI Train Bot ] ubuntu- ui-toolkit- gallery/ po/nb.po
* Resync trunk. added: examples/
[ Robert Park ] bileto_ convert_ to_gles control. gles debian/rules.gles
* Inline GLES packaging. added: debian/
debian/
-- Zoltan Balogh <email address hidden> Tue, 12 Apr 2016 11:12:44 +0000