diff options
author | Alberto Mardegan <alberto.mardegan@canonical.com> | 2015-11-10 15:42:23 +0200 |
---|---|---|
committer | Alberto Mardegan <mardy@users.sourceforge.net> | 2015-11-13 07:25:02 +0000 |
commit | 2b779fbd25aaf09897ee2cdc4edffd12a980420b (patch) | |
tree | b3d5664e2e2303acb13cdcb900161edd75e6ead6 /tests/auto/quick/qquicklistview/data/snapOneItemCurrentIndexRemoveAnimation.qml | |
parent | 4ff385e8cd509ab735e3dc5149273fe75d2de105 (diff) |
Don't make currentIndex skip an extra item on removal
Improve the logic for determining the desired viewport position, which
got partially broken with 134d980a7fcf61c5440019bcfb3fdfc39c3f5f3c.
Let's not alter topItem and bottomItem if their index appears to be
correct.
Task-number: QTBUG-49330
Change-Id: Ib1c88de51be28cbb0afb1741440adb03ae8ebd87
Reviewed-by: Albert Astals Cid <albert.astals@canonical.com>
Reviewed-by: J-P Nurmi <jpnurmi@theqtcompany.com>
Diffstat (limited to 'tests/auto/quick/qquicklistview/data/snapOneItemCurrentIndexRemoveAnimation.qml')
-rw-r--r-- | tests/auto/quick/qquicklistview/data/snapOneItemCurrentIndexRemoveAnimation.qml | 39 |
1 files changed, 39 insertions, 0 deletions
diff --git a/tests/auto/quick/qquicklistview/data/snapOneItemCurrentIndexRemoveAnimation.qml b/tests/auto/quick/qquicklistview/data/snapOneItemCurrentIndexRemoveAnimation.qml new file mode 100644 index 0000000000..215467f0cc --- /dev/null +++ b/tests/auto/quick/qquicklistview/data/snapOneItemCurrentIndexRemoveAnimation.qml @@ -0,0 +1,39 @@ +import QtQuick 2.4 + +ListView { + id: root + height: 400 + width: height + model: ListModel { + id: lmodel + ListElement { dummy: 0 } + ListElement { dummy: 0 } + ListElement { dummy: 0 } + ListElement { dummy: 0 } + ListElement { dummy: 0 } + ListElement { dummy: 0 } + } + + function removeItemZero() + { + lmodel.remove(0); + } + + orientation: ListView.Horizontal + snapMode: ListView.SnapOneItem + highlightRangeMode: ListView.StrictlyEnforceRange + + property int transitionsRun: 0 + + removeDisplaced: Transition { + id: transition + PropertyAnimation { property: "x"; duration: 500 } + onRunningChanged: if (!running) transitionsRun++; + } + + delegate: Text { + text: index + " of " + lmodel.count + width: root.width + height: root.height + } +}
\ No newline at end of file |