On 20/04/2016 16:29, Andrea Bernabei wrote:
> because topMargin should be available for the dev to use it for whatever
> he needs it to. The developer should not have to mind about setting it
> to accomodate a behaviour that is handled by PageHeader.
> 
> So, on one side, you have to make space for the header, on the other
> side, you should (must, I'd say) still let the developer decide a custom
> topMargin to implement whatever usecase he needs it for

But the documentation clearly states (and it has always been like this):
"The topMargin of the flickable will automatically be updated to always
match the height of the header."

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