As Andrea said, this is the expected behavior.

The PageHeader will update the topMargin of the flickable that you set
in order to ensure enough space for the header when it is visible. In
addition to that, you set another topMargin inside the Flickable. We had
to make a choice to either let the app developer always set the
topMargin in the apps, or do it automatically when you set
PageHeader.flickable. We chose the latter. There were some bugs related
to that which were fixed in the MR you linked which probably caused the
issues for you, but changing the behavior back will re-introduce those
bugs.

I'm not sure what the use case of this construction is, but perhaps this would 
work as a solution for you: http://paste.ubuntu.com/15948932/
There I add the PageHeader inside the ListView delegate so that each of the 
Items in the ListView has its own header instead of sharing one header that 
shows/hides depending on what is the currently selected item in the ListView.

-- 
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:
  New

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     : [email protected]
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to