** Description changed: 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.
-- You received this bug notification because you are a member of Ubuntu Touch seeded packages, which is subscribed to ubuntu-ui-toolkit in Ubuntu. https://bugs.launchpad.net/bugs/1566231 Title: When changing Page.header, set previous header.parent to null. Status in ubuntu-ui-toolkit package in Ubuntu: New 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. To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1566231/+subscriptions -- Mailing list: https://launchpad.net/~touch-packages Post to : touch-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~touch-packages More help : https://help.launchpad.net/ListHelp
