Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Combobox multi-select improvements #881

Open
gretanausedaite opened this issue Dec 8, 2022 · 5 comments
Open

Combobox multi-select improvements #881

gretanausedaite opened this issue Dec 8, 2022 · 5 comments
Labels
a11y Accessibility issues (keyboard navigation, color contrast, assistive technologies, semantics, etc) bug Something isn't working css Needs change in css package react Needs change in react package

Comments

@gretanausedaite
Copy link
Contributor

Combobox multi-select functionality has some concerns that could be improved:

  • researched the accessibility concerns that come with this markup
  • fix focus when multi-select is enabled
    • Scroll to one of the selected items on menu open
  • test performance - we have some foreach loops that could cause problems with bigger datasets
@gretanausedaite gretanausedaite added the bug Something isn't working label Dec 8, 2022
@r100-stack r100-stack added the maintenance Related to development or technical debt or chores label Dec 8, 2022
@mayank99 mayank99 transferred this issue from iTwin/iTwinUI-react Dec 22, 2022
@FlyersPh9 FlyersPh9 added css Needs change in css package react Needs change in react package labels Jan 4, 2023
@mayank99
Copy link
Contributor

mayank99 commented May 2, 2023

Adding some comments here as I've been working a lot with ComboBox multiselect lately.

In response to the original issue:

One new issue I noticed: after typing in a combobox and selecting an option (using Enter), I would have expected the text field to be cleared but it doesn't.

Also, when testing combobox in VO/NVDA (in general, not just multiselect), I was noticing weird behaviors and having some difficulty opening the menu. Will need to test and research some more. It might be better UX if we add inline autocomplete (#862).

@mayank99 mayank99 added a11y Accessibility issues (keyboard navigation, color contrast, assistive technologies, semantics, etc) bug Something isn't working and removed bug Something isn't working maintenance Related to development or technical debt or chores labels May 2, 2023
@jfisher-bentley

This comment has been minimized.

@r100-stack

This comment has been minimized.

@jfisher-bentley

This comment has been minimized.

@mayank99

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility issues (keyboard navigation, color contrast, assistive technologies, semantics, etc) bug Something isn't working css Needs change in css package react Needs change in react package
Projects
None yet
Development

No branches or pull requests

5 participants