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

Added .is-sr and .is-sr-focusable #153

Closed
wants to merge 1 commit into from

Conversation

jbrooksuk
Copy link

Closes #150

@jbrooksuk jbrooksuk mentioned this pull request Apr 1, 2016
@jbrooksuk
Copy link
Author

Ping @jgthms is this something we can get merged in? :)

@jgthms
Copy link
Owner

jgthms commented Apr 9, 2016

This has just been copied from Bootstrap.

The idea behind Bulma is to come with more modern and up-to-date solutions, rather than simply copying features by copying code. There must be new way to show content only to text readers.

@jgthms jgthms closed this Apr 9, 2016
@jbrooksuk
Copy link
Author

Yeah but every bit of research I did suggested that it's the way of doing it.

@thinksaydo
Copy link

thinksaydo commented Oct 22, 2017

https://webaim.org/techniques/css/invisiblecontent/

This is straight from WebAIM: use CSS. My experience as an accessibility developer is that JAWS in particular is not modern. Bootstrap uses a technique that works, even if a bit old school.

@Muhnad
Copy link
Contributor

Muhnad commented Aug 25, 2018

@jgthms @jbrooksuk Sorry I don't see any reason to close this PR!

@jbrooksuk I just added this PR lately #1764 you can check it. and you can add .is-sr-only-focusable as I missed to add it

@JohnAherne
Copy link

Just checking. I don't see is-sr-only-focusable any where in the docs.
Has it not been added or has it been decided not to include it for some reason

Any ideas?

@Muhnad
Copy link
Contributor

Muhnad commented May 11, 2019

@JohnAherne no I just missed adding it, you can open PR with it.

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.

Screenreader Support
5 participants