Hi Tim, I attached a branch which should give you an idea of how I think
the problem can be solved. Beware, I didn't test it and it's not
complete (it doesn't adjust contentY yet), it's just to be used as a
starting point.

In that branch I'm attaching some extra data to the flickable (using 
setProperty; but other methods, such a static 
QHash<QQuickFlickable*,HeaderData> are also possible -- I used setProperty 
because then I don't have to care about memory management):
- The original value of topMargin, before any header was bound to the flickable
- The number of headers currently bound to the flickable

This makes it easy to restore the flickable's margin once the last
header detaches from it.


Please let me know if you'd like me to continue working on this branch, or if 
you'd like to take it over yourself, or if you completely disagree with this 
solution. I don't see any reason why something like this wouldn't work but, as 
I wrote, I didn't test it, so maybe I'm missing some big point.

-- 
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/1572525

Title:
  [regression] Double header height is set as flickable topMargin

Status in Canonical System Image:
  Incomplete
Status in ubuntu-ui-toolkit package in Ubuntu:
  In Progress

Bug description:
  Th attached test application works fine under qml-module-ubuntu-
  components 1.3.1918+16.04.20160404-0ubuntu3 but breaks with the latest
  1.3.1938+16.04.20160416. At a first examination, I believe that the
  change which cause the regression is this one:

  https://code.launchpad.net/~tpeeters/ubuntu-ui-toolkit/invisible-
  header-topmargin/+merge/290659

  Run the attached test case with QML scene. You can scroll the view
  horizontally to see the other model items. Under the old version of
  the toolkit, all items have their page headers correctly aligned; with
  the new version, an extra spacing is added below the header.

  I found this bug while testing my ttrss app in rc-proposed; please
  don't let this UITK version reach our users, as more apps might be
  affected.

To manage notifications about this bug go to:
https://bugs.launchpad.net/canonical-devices-system-image/+bug/1572525/+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

Reply via email to