Move no-js class to html element

a11yproject / a11yproject.com

The A11Y Project is a community-driven effort to make digital accessibility easier.

https://a11yproject.com

Apache License 2.0

3.7k stars 551 forks source link

Move no-js class to html element #1566

Closed szepeviktor closed 1 year ago

szepeviktor commented 1 year ago

Just that.

From https://darekkay.com/evaluatory/demo/6.html

Cc @adamlaki

davatron5000 commented 1 year ago

Thanks for catching this. In general, please provide more context around your fixes than "Just that." The link to Dare Kay's took was provided without context.

[x] class cannot be applied to the <!doctype html> declaration
[x] I looked through the project and there's no use of .no-js in script or CSS

I almost rejected this PR for lack of context

szepeviktor commented 1 year ago

Thank you for merging it!