git.iko.soy/unity8
on xenial help sign in
remove snapping workaround as we have a proper fix for ListView in the queue
commit4249c625a194787bd718313d0109baac160f09e0 authorMichael Zanetti <michael.zanetti@canonical.com> dateFri 5 Jul 2013, 10:56 (13 years ago) parent5939f9d
2 files changed+12−21
compare with xenial
unifiedsplit
Launcher/LauncherPanel.qml−19 tests/qmltests/Launcher/tst_Launcher.qml+12−2
Launcher/LauncherPanel.qml−19
@@ -119,25 +119,6 @@ Item {
119119 property int itemSize: width
120120 property int clickFlickSpeed: units.gu(60)
121121
122− // Setting snapMode delayed to make sure the ListView stays positioned
123− // at the beginning. If the SnapMode is already set when the model
124− // delivers the items, it's going to snap immediately. Depending on
125− // the height of the list, this might cause the first item to be half
126− // folded at the beginning. Once the list is populated we can set the
127− // snapMode and actual snapping will only happen when the user
128− // interacts with the list.
129− Timer {
130− interval: 1000
131− running: true
132− repeat: true
133− onTriggered: {
134− topFoldingArea.clicked(undefined)
135− //launcherListView.snapMode = ListView.SnapToItem
136−// launcherListView.flick(0, -units.gu(2000000));
137−// launcherListView.contentY = -launcherListView.topMargin;
138− }
139− }
140−
141122 delegate: LauncherDelegate {
142123 id: launcherDelegate
143124 objectName: "launcherDelegate" + index
tests/qmltests/Launcher/tst_Launcher.qml+12−2
@@ -114,9 +114,11 @@ Item {
114114
115115 revealer.dragLauncherIntoView()
116116
117+ var listView = findChild(launcher, "launcherListView");
118+ listView.flick(0, units.gu(500));
119+ tryCompare(listView, "flicking", false);
120+
117121 var appIcon = findChild(launcher, "launcherDelegate0")
118− print("clicking now!", appIcon.angle);
119− wait(3000);
120122
121123 verify(appIcon != undefined)
122124
@@ -225,7 +227,14 @@ Item {
225227 name: "LauncherInit"
226228 when: windowShown
227229
230+ /*
231+ * FIXME: There is a bug in ListView which makes it snap to an item
232+ * instead of the edge at startup. Enable this test once our patch for
233+ * ListView has landed upstream.
234+ * https://bugreports.qt-project.org/browse/QTBUG-32251
235+
228236 function test_initFirstUnfolded() {
237+
229238 // Make sure noone changed the height of the window. The issue this test case
230239 // is verifying only happens on certain heights of the Launcher
231240 compare(root.height, units.gu(55));
@@ -238,5 +247,6 @@ Item {
238247 // Now do check that snapping is in fact enabled
239248 compare(listView.snapMode, ListView.SnapToItem, "Snapping is not enabled");
240249 }
250+ */
241251 }
242252}