Spec icon indicating copy to clipboard operation
Spec copied to clipboard

selectAll is missing in SpAbstractTreePresenter

Open hernanmd opened this issue 1 year ago • 1 comments

Just noticed there is a unselectAll method but no selectAll which could be very handy for lists where items are not using a checkbox.

Screenshot 2024-01-09 at 10 19 33

hernanmd avatar Jan 09 '24 09:01 hernanmd

thing is, there is no logical interpretation of what "selectAll" would mean in the context of a tree.

  • expand all and select any element?
  • select all the sibling nodes of current selection?
  • select all the hierarchy and siblings of current selection?

hence, selectAll was never implemented, letting the user to cover that behavior (a bit more cumbersome, but more appropriate IMO).

estebanlm avatar Jan 29 '24 10:01 estebanlm