Skip to content

Textinput: Added tabindex="-1" on clearbuttons #7155

Closed
wants to merge 1 commit into from

6 participants

@frequent

Fixes: gh-6467

@arschmitz
jQuery Foundation member

i want to wait on this one to check with an accessibility expert here

@scottgonzalez
jQuery Foundation member

+1

@gabrielschulhof gabrielschulhof modified the milestone: 1.5.0, 1.4.3 Mar 18, 2014
@jaspermdegroot jaspermdegroot modified the milestone: 1.4.4, 1.4.3 Jun 2, 2014
@arschmitz
jQuery Foundation member

@dylanb do you think you might have a second to take a look at this?

@dylanb
dylanb commented Aug 26, 2014

@arschmitz commented on the original ticket, have not looked at the code, will do once my comments are resolved

@gabrielschulhof gabrielschulhof added a commit that referenced this pull request Aug 29, 2014
@frequent frequent Textinput: added tabindex -1 on clearbuttons
Closes gh-7155
Fixes gh-6567
013c111
@gabrielschulhof gabrielschulhof removed this from the 1.4.4 milestone Aug 29, 2014
@gabrielschulhof

@dylanb I've updated the PR, adding aria-hidden="true" to the clear button markup: #7647

@gabrielschulhof gabrielschulhof added a commit that referenced this pull request Sep 4, 2014
@frequent frequent Textinput: added tabindex -1 on clearbuttons
(cherry picked from commit 826ada0)

Closes gh-7155
Fixes gh-6467
5c27fa0
@agcolom agcolom added a commit to agcolom/jquery-mobile that referenced this pull request Nov 26, 2014
@frequent frequent Textinput: added tabindex -1 on clearbuttons
Closes gh-7155
Fixes gh-6467
1de0e50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.