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

[select] feat(Select2, Suggest2, MultiSelect2): menuProps allows forwarding props to Menu #5435

Merged
merged 7 commits into from
Jul 20, 2022

Conversation

bvandercar-vt
Copy link
Contributor

Fixes #0000

Checklist

  • [N/A] Includes tests
  • [N/A] Update documentation

in order to pass aXe a11y testing, we need to apply an aria-label to the Menu that is created by MultiSelect2/Select2/Suggest2

@bvandercar-vt bvandercar-vt changed the title allow for passing an aria-label to the menu that gets created in MultiSelect2/Select2/Suggest2 allow for passing menuProps to the menu that gets created in MultiSelect2/Select2/Suggest2 Jul 12, 2022
@bvandercar-vt
Copy link
Contributor Author

I am also open to renaming this prop ‘listProps’ or ‘listboxProps’ instead of ‘menuProps’, whatever the devs prefer

@adidahiya adidahiya self-requested a review July 12, 2022 18:40
Copy link
Contributor

@adidahiya adidahiya 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! just some minor docs edit suggestions

packages/select/src/components/select/select2.tsx Outdated Show resolved Hide resolved
packages/select/src/components/suggest/suggest2.tsx Outdated Show resolved Hide resolved
@adidahiya adidahiya changed the title allow for passing menuProps to the menu that gets created in MultiSelect2/Select2/Suggest2 [select] feat(Select2, Suggest2, MultiSelect2): menuProps allows forwarding props to Menu Jul 20, 2022
bvandercar-vt and others added 3 commits July 19, 2022 19:34
Co-authored-by: Adi Dahiya <adi.dahiya14@gmail.com>
Co-authored-by: Adi Dahiya <adi.dahiya14@gmail.com>
Co-authored-by: Adi Dahiya <adi.dahiya14@gmail.com>
@adidahiya adidahiya merged commit c328210 into palantir:develop Jul 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants