I'm not a fan of those HTML documents, they're so cluttered and hard to read
. I mean, can't we use some basic guidelines for naming conventions? Like, what's with `gv-standfirst` vs `gvStandfirst`, is it just one or the other?
And don't even get me started on the code organization, it's like they just threw everything together and expected us to clean up after them. A good coder would use some basic CSS files to style the elements instead of mixing everything in one big file.
I also noticed that there are some redundant attributes like `style="display:none"` which is just unnecessary, why not just use `<div style="display:none">`? And what about accessibility, where's the `tabindex` attribute?
The example improvements they provided are a good start, but I think we could take it further. For instance, using semantic HTML elements like `<header>`, `<nav>`, etc would make the code way more readable.
And lastly, why not minify and compress those files? It's just common sense to reduce file size for faster page loads.
All in all, I'd give that document a solid 2/10 for readability and maintainability
.
