Valid HTML Improvement: ul tag for language switch shouldn't be a child of a span

[drohen]

I tried validating the HTML of my website, and it failed on the Site-level language switch. The ul tag used to list the available languages is a block-level element, and this is nested within various span elements. As span is a phrasing element, it also should only contain phrasing elements. Either the ul should be replaced with something else, or the parent tags should be changed to something like a div.

If you would prefer me to make a PR for this I can do so.

[adityatelange]

Is HTML validation everything? Like a <span> here works the same as using 2 or more CSS rules. It gets the job done.

[louisfoster]

I figured as it's a blog theme, blogs generally contain valid HTML to help with parsers for search engine indexing and accessibility. I was merely pointing this out in case you weren't aware. It's your theme, If you're not concerned, then feel free to close the issue.

[Rotzbua]

I observed the same problems.

W3 validator

Current demo

https://validator.w3.org/nu/?doc=https%3A%2F%2Fadityatelange.github.io%2Fhugo-PaperMod%2F

Error output

Warning output

Solution

Created two PRs which solved the problems for my setup. Not sure if they solve it for every configuration.