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 What Input? to the list of scripts to remove outlines #383

Merged
merged 1 commit into from
Jan 5, 2016

Conversation

FezVrasta
Copy link
Contributor

I think "What Input?" is the best solution if a front-end developer needs to remove outlines from the elements of the page.

It allows you to style your elements in a specific manner only if the user is using a specific input method (mouse, touch, keyboard) and does not interfere with the click events.
Additionally, it's much more lightweight.

In the startup I work for we used it successfully and now most of the app is keyboard accessible.

davatron5000 added a commit that referenced this pull request Jan 5, 2016
added What Input? to the list of scripts to remove outlines
@davatron5000 davatron5000 merged commit d663286 into a11yproject:gh-pages Jan 5, 2016
@davatron5000
Copy link
Contributor

Nice find! I like it.

@joe-watkins
Copy link

Hmm... this is a cool idea but makes me a little nervous.. spidey senses tingling. In a quick test you could see here how a user could be disoriented if this plugin is used how prescribed. http://www.screencast.com/t/VqUzNs3nZz

In this screencast I'm using both a mouse and a keyboard for navigating the interface found on this Pen. You'll see the plugin remove the outline as I move the mouse outside the viewport as it bounces between keyboard and mouse assumption.

I still need to test on a laptop at home that has both touch and mouse... but perhaps we should make a note about using with extreme caution? Thoughts?

@FezVrasta
Copy link
Contributor Author

Didn't noticed this behavior, I don't think it's a big problem but I'll open an issue to see if it can be fixed

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.

3 participants