search

dillonkearns / elm-pages-starter

Starter blog for elm-pages
https://elm-pages-starter.netlify.com
95 stars 41 forks source link

Please include focus styles by default #2

Open fpapado opened 5 years ago

fpapado commented 5 years ago

Hi!

I've been reading through the source and the blog post on elm-pages. I noticed while doing so that focus styles are missing. I feel a bit frustrated by that, because I would like this project to have accessible defaults, and that it is important to have a solid basis if the community centralises on it.

If I understand correctly, elm-ui has layoutWith and FocusStyle to include focus styles.

Would it be possible to include the focus by default? Please let me know if I'm missing something, or you would like any more information.

You can find more about focus styles on the Web Content Accessibility Guidelines. There is also a post by Hidde de Vries on Mozilla Hacks about focus.

dillonkearns commented 5 years ago

Hi @fpapado, thank you so much for pointing that out! As we discussed on Twitter, if you get around to it I would very much welcome a PR here!