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

fix(ListBox): enable drawer animation #5810

Merged

Conversation

asudoh
Copy link
Contributor

@asudoh asudoh commented Apr 6, 2020

(Please use gear icon - "Hide whitespace changes" for reviewing)

This change adds was-missing animation to list boxes' drawers.

Fixes #2697.

Changelog

New

  • Animation to list of drawers.

Testing / Reviewing

Testing should make sure <ComboBox>, <Dropdown>, <MultiSelect> are not broken.

This change adds was-missing animation to list boxes' drawers.

Fixes carbon-design-system#2697.
@asudoh asudoh requested review from emyarod, a team and jeanservaas and removed request for a team April 6, 2020 08:05
@ghost ghost requested review from dakahn and removed request for a team April 6, 2020 08:05
@netlify
Copy link

netlify bot commented Apr 6, 2020

Deploy preview for carbon-elements ready!

Built with commit c63c08d

https://deploy-preview-5810--carbon-elements.netlify.com

@netlify
Copy link

netlify bot commented Apr 6, 2020

Deploy preview for carbon-components-react ready!

Built with commit c63c08d

https://deploy-preview-5810--carbon-components-react.netlify.com

Copy link
Member

@emyarod emyarod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the drawer open and close animations are now visible in all listbox components. looks good to me pending visual review

@tw15egan tw15egan requested a review from a team as a code owner April 6, 2020 18:38
@ghost ghost requested a review from joshblack April 6, 2020 18:38
Copy link
Contributor

@jeanservaas jeanservaas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@tw15egan tw15egan merged commit 60cd06c into carbon-design-system:master Apr 8, 2020
@asudoh asudoh deleted the dropdown-drawer-animation branch April 8, 2020 23:52
joshblack pushed a commit to joshblack/carbon that referenced this pull request Apr 16, 2020
This change adds was-missing animation to list boxes' drawers.

Fixes carbon-design-system#2697.

Co-authored-by: TJ Egan <tw15egan@gmail.com>
joshblack pushed a commit that referenced this pull request Apr 21, 2020
This change adds was-missing animation to list boxes' drawers.

Fixes #2697.

Co-authored-by: TJ Egan <tw15egan@gmail.com>
This was referenced Apr 23, 2020
@nertzy
Copy link

nertzy commented Apr 27, 2020

Should we be setting aria-hidden on the listbox when it is "closed"? We are no longer removing it from the DOM so it will still get picked up by screen readers, right?

@nertzy
Copy link

nertzy commented Apr 27, 2020

See #5949

@asudoh asudoh mentioned this pull request Apr 29, 2020
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Dropdown, react] Missing drawer animation
5 participants