Cross-slide and multiple-selection are slated for the new proposed tilegroup widget. In order to unblock the implementation of the contextual appbar and the views & tiles actions that depend on it, we need a stop-gap measure that adapts the existing richgrid with enough to test and work through contextual-action flows
Created attachment 708080 [details] [diff] [review] Implements multiple selection behavior on richgrid/richgriditem, triggerd via contextmenu (stop-gap) This is just the richgrid/richgrid item changes which allow us to fire a selectionchange event on the grid and the user to toggle tile selection (via contextmenu/right-click/long-press until cross-slide is implemented). I'm still fighting getting tests to run in the xul file so I'll land that seperately if/when its ready.
Created attachment 708087 [details] [diff] [review] Implements multiple selection behavior on richgrid/richgriditem, triggerd via contextmenu (stop-gap) Addressed feedback to use selectedItems property instead of QSA in clearSelection. I left the selectionchange event behavior as-is, rather than calling some method on the controller. This seems more like the expected, DOM-ish behavior and we can stop events bubbling to avoid them being global if it becomes a perf. problem.
Attachment #708087 - Flags: review?(mbrubeck) → review+
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.