Without further investigation, the following patch fixes 3 of the 4
failures (I haven’t checked whether it doesn’t break other things in the
UITK though):
=== modified file 'src/Ubuntu/Components/plugin/privates/listviewextensions.cpp'
--- src/Ubuntu/Components/plugin/privates/listviewextensions.cpp
2016-03-01 15:08:08 +0000
+++ src/Ubuntu/Components/plugin/privates/listviewextensions.cpp
2016-03-08 09:55:06 +0000
@@ -141,11 +141,12 @@
int currentIndex = this->currentIndex();
int count = this->count();
+ int oldIndex = currentIndex;
if (currentIndex >= 0 && count > 0) {
currentIndex = qBound<int>(0, forwards ? currentIndex - 1 :
currentIndex + 1, count - 1);
setCurrentIndex(currentIndex);
setKeyNavigationForListView(true);
}
- return true;
+ return (oldIndex != currentIndex);
}
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1554447
Title:
4 webbrowser-app unit test failures with staging UITK
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1554447/+subscriptions
--
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
