The bug was not introduced by the MR linked in the bug description. I
reproduced the bug with this code, and pulling down the listview and
releasing it with a vertical velocity:

import QtQuick 2.2
import QtQuick.XmlListModel 2.0
import Ubuntu.Components 1.3
import Ubuntu.Components.ListItems 1.3
MainView {
    width: units.gu(40)
    height: units.gu(71)
    Page {
        id: page0
        title: "Reuters"
        ListView {
            id: view
            anchors.fill: parent
            model: 15
            delegate: Standard {
                width: ListView.view.width
                height: units.gu(5)
                text: index//title
            }
            onVerticalVelocityChanged: print("vv = "+verticalVelocity)
            PullToRefresh {
                id: pullToRefresh
                onRefreshingChanged: print("refreshing = "+refreshing)
                onRefresh: {
                    refreshing = true;
                    refreshing = false;
                    //                        refreshTimer.start();
                }
            }
        }
    }

//    Timer {
//        id: refreshTimer
//        onTriggered: pullToRefresh.refreshing = false
//    }
}

and with UITK r1930 (before the mentioned MR) and r1918.

I think the bug was always there, the topMargin gets messed up by the
PullToRefresh. However, pre-r1931, the topMargin was (incorrectly) reset
to 0 when a new Page was pushed on the PageStack (which happens in
onRefresh for the example linked to th ebug). Now that this no longer
happens, the bug becomes visible when the Page is popped from the stack.

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