vuestic-ui icon indicating copy to clipboard operation
vuestic-ui copied to clipboard

Feat/tree view rework

Open rustem-nasyrov opened this issue 2 years ago • 4 comments

Related issues

Close #1693, close #1567, close #1693

Description

demos

Added:

  • Demos
  • Selection
  • Expanding/collapsing tree nodes
  • Filter
  • Docs
  • Model value
  • Template
  • Colors

Types of changes

  • [x] New feature (non-breaking change which adds functionality)
  • [x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [x] Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

rustem-nasyrov avatar May 05 '22 09:05 rustem-nasyrov

@aluarius, can you later check how is implemented the accessibility in the component?

rustem-nasyrov avatar Jul 25 '22 08:07 rustem-nasyrov

@aluarius, can you later check how is implemented the accessibility in the component?

  • every input element should have aria-label (for example, every checkbox for selectable state - check *node label value* for them or append aria-hidden="true" if you don't like this variant). See also VaCheckbox components for additional aria-attributes (for example, aria-checked);
  • missing any interaction options with focused node (how can i expand it via keyboard?). Check this: https://miro.medium.com/max/1400/1*f3t8bHWDOktffVIchD_lTg.png
  • according to https://www.w3.org/WAI/ARIA/apg/example-index/treeview/treeview-1/treeview-1a.html here should be also aria-selected attribute.

aluarius avatar Jul 25 '22 10:07 aluarius

Looks close to finish, but reactivity isn't there.

asvae avatar Aug 30 '22 23:08 asvae

We need to fix demos before continuing forward with that. Too much untested code.

asvae avatar Sep 08 '22 21:09 asvae