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 menu - "faked" current selection text separately announced by screen readers #8256

Closed
patrickhlauke opened this Issue Aug 5, 2015 · 2 comments

Comments

Projects
None yet
3 participants
@patrickhlauke
Contributor

patrickhlauke commented Aug 5, 2015

Taking the first example from http://demos.jquerymobile.com/1.4.5/selectmenu/, based on the following simplified markup

<div ... class="...">
  <span>The 1st Option</span>
  <select name="..." ...>
    <option value="1">The 1st Option</option>
    ...
  </select>
</div>

when using TalkBack/Android or VoiceOver/iOS, the <span> is read/announced separately, before the actual <select>. This results in redundant/inappropriate output (as the <span> is not announced as being actionable, and the current option is already announced once the user moves their screenreader focus to the actual <select>).

Solution would be to apply an aria-hidden="true" to that <span>

<div ... class="...">
  <span aria-hidden="true">The 1st Option</span>
  <select name="..." ...>
    <option value="1">The 1st Option</option>
    ...
  </select>
</div>

which suppresses that element completely from being seen/announced by SRs

@arschmitz arschmitz added this to the 1.5.0 milestone Aug 5, 2015

@arschmitz

This comment has been minimized.

Show comment
Hide comment
@arschmitz

arschmitz Aug 5, 2015

Member

@patrickhlauke thank you for reporting this. PR very welcome if you have time. Either way we will make sure this is fixed for the beta coming shortly.

Member

arschmitz commented Aug 5, 2015

@patrickhlauke thank you for reporting this. PR very welcome if you have time. Either way we will make sure this is fixed for the beta coming shortly.

@patrickhlauke

This comment has been minimized.

Show comment
Hide comment
@patrickhlauke

patrickhlauke Aug 5, 2015

Contributor

No worries, I'll see if I can find some time to properly PR this in the next few days :)

Contributor

patrickhlauke commented Aug 5, 2015

No worries, I'll see if I can find some time to properly PR this in the next few days :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment