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

Improve the screen-reader only classes #13

Closed
wants to merge 5 commits into from
Closed

Improve the screen-reader only classes #13

wants to merge 5 commits into from

Conversation

ffoodd
Copy link
Contributor

@ffoodd ffoodd commented Oct 13, 2016

I also wrote a post about this.

It's supposed to be much more robust and future proof than the actual version, but it needs to be tested against various configurations (browsers and ATs).

I'd be pleased to discuss this.

I also wrote [a post about this](http://www.ffoodd.fr/cache-cache-css/).

It's supposed to be much more robust and future proof than the actual version, but it needs to be tested against various configurations (browsers and ATs).

I'd be pleased to discuss this.
Thanks to @thierryk for this correction.
Since clip only apply if position isn't static, resetting clip for focusable things is unnecessary.
@ffoodd
Copy link
Contributor Author

ffoodd commented Oct 18, 2016

A bug has been found with iOS + VoiceOver and Android + TalkBack, issue opened in Bootstrap.

My additions don't help.

As [described in a Bootstrap issue](twbs/bootstrap#20732) and [reported by Johan Ramon](https://twitter.com/johan_ramon/status/788372720224526336) from @atalan.

Fix found by @PigeardSylvain
@ffoodd
Copy link
Contributor Author

ffoodd commented Oct 19, 2016

For your information, bugs have been filled in almost every mobile / tactile screen readers:

It appears that screen readers on tactile devices move the accessibility focus without firing the JavaScript focus event nor applying the CSS :focus pseudo-class. This is the short version.

It's meant to prevent user from accidentally activate something (like text input that opens the keyboard and so on), but causes skip links to be useless — and maybe painfull.

However VoiceOver does restrict this behavior to form inputs: links and buttons do trigger focus as accessibility focus. But it still has bug when linking to a container using tabindex, which is pretty common (and recommended by Bootstrap).

@ffoodd
Copy link
Contributor Author

ffoodd commented Mar 20, 2017

Hi there,

I close this because it's been merged into Bootstrap and should be shipped in Beta 1:
twbs/bootstrap#22154

If you find any issue with this, we shall report it on Bootstrap directly :)

@ffoodd ffoodd closed this Mar 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Boosted
Help wanted
Development

Successfully merging this pull request may close these issues.

None yet

1 participant