Skip to content

Base aria-live message on tabSelectsValue prop #4136

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

Merged
merged 7 commits into from
Jan 13, 2021

Conversation

jkjustjoshing
Copy link
Contributor

If the tabSelectsValue prop is set to false, the aria-live message is inaccurate. This should fix that.

Addresses #3367

Aria message mentions "press Tab to select the option and exit the menu", but that's only true if tabSelectsValue is true
@changeset-bot
Copy link

changeset-bot bot commented Jul 24, 2020

🦋 Changeset detected

Latest commit: b27d01b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
react-select Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 24, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b27d01b:

Sandbox Source
react-codesandboxer-example Configuration

@bladey
Copy link
Contributor

bladey commented Aug 24, 2020

Thanks for the PR @jkjustjoshing!

@bladey bladey added pr/bug-fix PRs that are specifically addressing a bug pr/needs-review PRs that need to be reviewed to determine outcome labels Aug 24, 2020
@Methuselah96 Methuselah96 added this to the 3.2 milestone Dec 18, 2020
Copy link
Collaborator

@Methuselah96 Methuselah96 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, this looks good!

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@Methuselah96
Copy link
Collaborator

@jkjustjoshing Can you add a changeset?

Copy link
Collaborator

@ebonow ebonow 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!

Methuselah96 and others added 5 commits January 13, 2021 03:42

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@JedWatson JedWatson merged commit ef2d357 into JedWatson:master Jan 13, 2021
@github-actions github-actions bot mentioned this pull request Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/bug-fix PRs that are specifically addressing a bug pr/needs-review PRs that need to be reviewed to determine outcome
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Screen reader still reads tabbing instructions when tabSelectsValue prop is false
5 participants