oui icon indicating copy to clipboard operation
oui copied to clipboard

Add `count` prop to OuiTreeView

Open KrooshalUX opened this issue 1 year ago • 6 comments

  • [ ] Count prop should reflect the number of OuiTreeNode children, and should be optional to include
Screen Shot 2023-07-28 at 12 35 57 AM
  • [ ] Reference count styling from OuiFacet and OuiFilterGroup to build styling while retaining hover styling of OuiTreeView - spacing between text & count = $ouiSizeS Examples: - Default: Screen Shot 2023-07-28 at 12 04 09 AM
Screen Shot 2023-07-28 at 12 08 03 AM

- Disabled: Screen Shot 2023-07-28 at 12 04 14 AM

Discuss: What do we think about using the styling of selected that is present in OuiFacet and OuiFilterGroup for the isExpanded state of OuiTreeGroup ? Do we think that the expansion itself is visual communication or not? IMO, the expansion itself is sufficient, but for the sake of complete exploration here are examples of count changing color when a user is interacting with an element that contains a count:

Screen Shot 2023-07-28 at 12 08 55 AM Screen Shot 2023-07-28 at 12 09 03 AM

KrooshalUX avatar Jul 28 '23 07:07 KrooshalUX