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

MultiSelect cannot be closed via keyboard #7197

Closed
jrpomeroy opened this issue Nov 2, 2020 · 14 comments
Closed

MultiSelect cannot be closed via keyboard #7197

jrpomeroy opened this issue Nov 2, 2020 · 14 comments
Labels
component: multiselect package: react carbon-components-react severity: 3 https://ibm.biz/carbon-severity type: a11y ♿

Comments

@jrpomeroy
Copy link
Contributor

Environment

Operating system

OSX

Browser

Latest Chrome

Automated testing tool and ruleset

IBM CI 162 Checkpoint Violation: IBM 2.1.1 Keyboard
IBM Guidance: https://www.ibm.com/able/guidelines/ci162/keyboard.html
Test Tool Type: Manual test

Detailed description

Unable to close the MultiSelect menu with keyboard. If you use the Escape key the menu will close but also appears to remove selection.

What version of the Carbon Design System are you using?

[email protected]

What did you expect to happen?

Menu can close while also maintaining selected items.

What happened instead?

Menu closes with Escape key but selection removed (or at least appears that way).

What WCAG 2.1 checkpoint does the issue violate?

IBM CI 162 Checkpoint Violation: IBM 2.1.1 Keyboard

Steps to reproduce the issue

  1. Go to https://react.carbondesignsystem.com/?path=/story/multiselect--filterable
  2. Use keyboard to navigate into MultiSelect dropdown and select an item
  3. Press escape key, or otherwise try to navigate out and keep the selection

Please create a reduced test case in CodeSandbox

Hopefully the storybook example is enough to reproduce.

Additional information

Original issue here:
https://github.ibm.com/alchemy-containers/armada-ui/issues/5904

@mashenka123
Copy link

@joshblack Could you please confirm when it could be triaged? We need the fix this month, could you please help kindly support? Thank you very much!

@tw15egan
Copy link
Collaborator

tw15egan commented Dec 9, 2020

Would the intended behavior be that the menu can be closed via the esc key, and not remove the selected items?

@tw15egan
Copy link
Collaborator

tw15egan commented Dec 9, 2020

Our downshift dependency may need to be updated to 6.0.0

downshift-js/downshift#719

@xiepingp
Copy link

Would the intended behavior be that the menu can be closed via the esc key, and not remove the selected items?

@tw15egan, yes, I think it is acceptable.
Current behaviour is that if user presses ESC, the selection will be removed along with the dropdown menu.

Do you have any plan when to update your downshift dependency? We need to fix the issue this month, would you please kindly take a look? Thanks a lot!!

@dakahn
Copy link
Contributor

dakahn commented Jan 13, 2021

just jumping in to confirm this is Downshift functionality. On close menu/focus move text input is replace with empty string. We're discussing this here

@mashenka123
Copy link

@dakahn So what's the idea for this bug now?

@dakahn
Copy link
Contributor

dakahn commented Jan 21, 2021

The bug has been prioritized and moved to our back log. Watch this space for updates. Thanks @mashenka123

@mashenka123
Copy link

@dakahn Do you know when this would be fixed?

@dakahn
Copy link
Contributor

dakahn commented Jan 25, 2021

Can't give you an ETA, sorry

@dakahn
Copy link
Contributor

dakahn commented Feb 10, 2021

This issues appears to be resolved testing on Windows 10 in both Chrome and Firefox against our sandbox.

@joshblack
Copy link
Contributor

Going to close this out since the behavior seems to have been fixed. Feel free to comment and I can re-open!

@xiepingp
Copy link

xiepingp commented Feb 18, 2021

Thanks for the fixing! Now the multiselect can be closed by pressing esc and the selections are kept well in https://react.carbondesignsystem.com/?path=/story/multiselect--default.
But I tried in https://react.carbondesignsystem.com/iframe.html?id=multiselect--filterable, press esc key will remove all the selections:
press up/down arrow and enter to select options:
7197 - retest 1
press esc, all the selections are removed
7197 - retest 2
If press Tab, the multiselect will not be closed which is same as before....
@joshblack , would you please reopen and have a look?

@joshblack
Copy link
Contributor

@xiepingp I went ahead and opened up an issue for this (and some other issues) over in: #7861

@xiepingp
Copy link

Thanks for the information 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: multiselect package: react carbon-components-react severity: 3 https://ibm.biz/carbon-severity type: a11y ♿
Projects
None yet
Development

No branches or pull requests

6 participants