Update HTML attributes to use quotes

[InstyleVII]

Added quotes to all HTML attributes to improve editor tooling.

---

Preview | Diff

[InstyleVII]

This is an updated version of my previous PR with just the addition of quotes to all HTML attributes as well as one minor fix to an incorrect class name.

[andreastt]

I don’t see the value of this PR at all. This is not incorrect usage of attributes.

[InstyleVII]

Two mains reasons:
Editor tooling - Some editors won't read links as links allowing you to nav to them from the editor if they aren't in quotes, VS Code is one such example
Consistency - We had some attributes using quotes and others not using quotes, now the document is consistently using quotes

[andreastt]

The document is pretty much consistent in not using quotes where it is unnecessary already, so I wouldn’t go as far as calling that an inconsistency. I would also consider it a bug that your editor doesn’t recognise valid HTML attributes.

Now I’m not going to block a patch that adds quotes everywhere, but I would like to reiterate that they are completely pointless.

[InstyleVII]

@shs96c mind assigning a reviewer?

[InstyleVII]

Going to close as consensus is this isn't needed.