To summarize, there were 2 bugs:

1. PullToRefresh messes up the topMargin
2. On pop(), the topMargin of page0 was set to header.height (instead of 
keeping the current topMargin) (Fixed)

Bug 2 was hiding bug 1 in the example from the report, and it was
exposed when bug 2 got fixed. We still need to fix bug 1, but a
workaround for now is to reset the incorrect topMargin when popping a
page.

Note that the example in my previous comment shows bug 1 independently
from bug 2.

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

Title:
  [regression] Extra flickable margin added when using PullToRefresh

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

Bug description:
  The attached test works fine under OTA-10, but fails in rc-proposed.
  This is most likely caused by https://code.launchpad.net/~tpeeters
  /ubuntu-ui-toolkit/invisible-header-topmargin/+merge/290659 which also
  causes bug 1572525 and I asked to be reverted.

  To reproduce, run the attached test case with qmlscene:

  1) Wait for the window to be populated (network connectivity is needed)
  2) Swipe down, to trigger the pull to refresh action
  3) View automatically jumps to the second page
  4) navigate back to the first page
  5) watch the flickable topMargin: the pulltorefresh item is now visible

  If time you repeat steps 2-4, the topMargin increases.

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