ListItems with trailing but no leading actions, after swiping right break selectMode
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | ubuntu-ui-toolkit (Ubuntu) |
High
|
Zsombor Egri | ||
| | Vivid |
Undecided
|
Unassigned | ||
Bug Description
If you have a listitem with trailing actions but no leading actions it is possible to break the selectMode as the press and hold signal is never fired.
Steps with the example QML app [0]
1) Swipe one of the listitems to the right (to try and reveal a leading action [which there aren't any])
2) Press and hold on the swiped listitem
3) Notice nothing happens (also notice nothing in the log for the press and hold appears)
4) Press and hold on another listitem
5) Notice that all but the swiped listitem change to selectMode=True
What was expected:
3) For all of the items to change to selectMode=True
5) For all of the items to change to selectMode=True
Note that you can 'fix' the listitem by swiping to the left and clicking on an action or dismissing the swipe, then you are able to press and hold it to activate selectMode correctly again.
Related branches
- PS Jenkins bot: Approve (continuous-integration) on 2015-07-14
- Ubuntu SDK team: Pending requested 2015-07-13
-
Diff: 179 lines (+157/-3)2 files modifiedmodules/Ubuntu/Components/plugin/uclistitem.cpp (+6/-3)
tests/unit_x11/tst_components/tst_listitem_actions_breaks_selectmode.qml (+151/-0)
| description: | updated |
| summary: |
- ListItems with trailing but no leading, after swiping right break - selectMode + ListItems with trailing but no leading actions, after swiping right + break selectMode |
| Launchpad Janitor (janitor) wrote : | #1 |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | New → Confirmed |
| Victor Thompson (vthompson) wrote : | #2 |
I can confirm this bug. I wonder if the ListItem's selectMode property is becoming disconnected from the view's property. Perhaps a work around would be to individual set each ListItem's selectMode property [0].
0 - https:/
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| assignee: | nobody → Zsombor Egri (zsombi) |
| tags: | added: new-listitem |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| importance: | Undecided → High |
| status: | Confirmed → In Progress |
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | In Progress → Fix Committed |
| Changed in ubuntu-ui-toolkit (Ubuntu Vivid): | |
| status: | New → Fix Committed |
| Launchpad Janitor (janitor) wrote : | #3 |
This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1584+
---------------
ubuntu-ui-toolkit (1.3.1584+
[ Florian Boucault ]
* New BottomEdgeHint component to represent extra features available from the
bottom edge of an application.
[ Loïc Molinari ]
* [UbuntuShape] Added a big radius.
* [UbuntuShape] Added relative radius support. Fixes LP: #1478124.
* Ensured components, styles, examples and tests use the new UbuntuShape
properties (not deprecated). Fixes LP: #1437412.
[ Benjamin Zeller ]
* Make use of the official qt build macros to blend into the Qt buildprocess.
* Fix debug builds, optimization is always enabled by the system qt build.
[ Richard Huddie ]
* Fix for autopilot bug lp:1476715. Don't throw an exception if maliit-server
is not found. Fixes LP: #1476715.
[ Tim Peeters ]
* Clean up the MainView docs.
* Set theme version for Sections component.
* Implement AdaptivePageLayout.
[ Zsombor Egri ]
* Fix width for trailing and leading actions of a ListItem. Fixes LP: #1465582.
* Button and Haptics import wrong toolkit versions, thus they break style
versioning. Moving Icon and ProgressBar to 1.0 and 1.1 version source folder.
* SuruDark theme for ListItem style. Fixes LP: #1451225.
* Swiping ListItem when no actions are defined for the gesture breaks
selectMode. Fixes LP: #1468100.
* Fixing selected connection with the ListItem's select mode checkbox state.
Fixes LP: #1461501, LP: #1469471.
[ Timo Jyrinki ]
* Fix ucstylehints.cpp compilation with Qt 5.5. Fixes LP: #1473873.
* Add PageHeadStyle 1.3 reference to fix install_
Qt 5.5. Fixes LP: #1466484.
[ Christian Dywan ]
* Avoid hard-coded skipping of members by name "type" can be a property name
regardless of also being a field in the JSON description of a property.
* Add apicheck unit test for QML and Javascript.
* Remove "do cleanup" comments. Fixes LP: #1369874.
* Initialize defaultTypes later to avoid bogus types.
* Implement Action.shortcut property. Fixes LP: #1202464.
* Update text handler to 3gu assert.
* Add a deprecated note to ListItems.
* Don't include overridden properties in API.
* Clean-up API check wrapper scripts.
* Track version members were introduced.
* Implement ListItemPopover on right-click. Fixes LP: #1452676.
* Move delegate's chevron into the row and size it explicitly.
Fixes LP: #1474418.
* Enable (Shift)Tab via activeFocusOnTab. Fixes LP: #1276797.
* Only swipe with left button and block timer otherwise.
Fixes LP: #1476300, LP: #1476310.
* Include Javascript libraries in QML documentation. Fixes LP: #1466058.
[ Albert Astals Cid ]
* Fix warning if there's no __propagated
* TypeError: Cannot call method 'hasOwnProperty' of null.
-- CI Train Bot <email address hidden> Thu, 30 Jul 2015 13:04:18 +0000
| Changed in ubuntu-ui-toolkit (Ubuntu): | |
| status: | Fix Committed → Fix Released |


Status changed to 'Confirmed' because the bug affects multiple users.