Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Publisher: Multiselection in card view #3993

Merged
merged 9 commits into from
Oct 19, 2022

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Oct 17, 2022

Brief description

Card view now supports multiselection of items.

Description

Card view have multiselection and selectetion between views is consistent (or should be). Card view support Control + click to add/remove selection of single item and Shift+click to add selection from last selected to just selected item.

Screenshot

image

Testing notes:

  1. Open TrayPublisher
  2. Create some instances with different families so there is more groups
  3. Play with the selection
  4. Switch of view should keep the selection between

Depends on #3972

@ynbot
Copy link
Contributor

ynbot commented Oct 17, 2022

Task linked: OP-4239 Multiselection in card view

@iLLiCiTiT iLLiCiTiT self-assigned this Oct 17, 2022
@iLLiCiTiT iLLiCiTiT added the type: enhancement Enhancements to existing functionality label Oct 17, 2022
self._explicitly_selected_groups.append(group_name)
new_widget.set_selected(True)

def _select_item_extend_to(self, instance_id, group_name, new_widget):
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'InstanceCardView._select_item_extend_to' is too complex (35)

Copy link
Member

@mkolar mkolar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works nicely. thanks

Copy link
Member

@jakubjezek001 jakubjezek001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well!

@iLLiCiTiT iLLiCiTiT merged commit 1567800 into develop Oct 19, 2022
@iLLiCiTiT iLLiCiTiT deleted the feature/OP-4239_Multiselection-in-card-view branch October 19, 2022 10:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants