For modal use :target instead of :focus

[dominikwilkowski]

Hi Una,

Love this effort. Thanks for putting some work into this.
My suggestion for the modal is to use :target instead of :focus as it also moves the focus to the open modal (important for a11y) and is arguably more stable.
A trick here is to use the close button to target the original open button as well so focus is also returned to the original element.
Happy to do a PR if you're interested.

Cheers

[r3nya]

Hi @una and @dominikwilkowski!

I totally agree with @dominikwilkowski.
Btw, I made a simple example for you http://codepen.io/r3nya/pen/pEKzyK
Please look.

PS: Thanks for you idea. I love it. ❤️

[Haroenv]

related to #10, the version by @r3nya does work on Safari.

[r3nya]

Hey @Haroenv!
It works on Safari 10 (macOS).

[Haroenv]

Yep, that's what I was saying. The version you posted does work, while the one on the site doesn't yet.

[r3nya]

@Haroenv oops, sorry …