elixirschool/school_house

HTML issue: some sections lack heading

jaimeiniesta opened this issue · 3 comments

jaimeiniesta commented

Our next biggest issue is this one:

https://rocketvalidator.com/s/0d798b32-e799-4867-8f8c-45698197f2e0/i/115600579

You can read the guide about "Section lacks heading".

It usually means that in web pages like https://elixirschool.com/bg/lessons/intermediate there's a <section> that doesn't have a heading (like h1, h2) etc. to give a title to that section.

The HeadingsMap Chrome Extension may help here.

jaimeiniesta commented

For example, this p could be converted to an h1 and then adjust CSS accordingly:

https://github.com/elixirschool/school_house/blob/master/lib/school_house_web/templates/lesson/_intermediate.html.heex#L4

This would give this section a heading.

Alternatively, as this page has only one section then maybe a better fix would be converting this section in a div:

https://github.com/elixirschool/school_house/blob/master/lib/school_house_web/templates/lesson/index.html.heex#L1

I mean, why declare a section if there's only a section? It makes sense to have sections when we have many within a document, and that's why each section needs a heading :)

sujinleeme commented

Hello. @jaimeiniesta. I would like to take this ticket. Could I grab it?

jaimeiniesta commented

Sure @sujinleeme - also ping me at jaime AT rocketvalidator.com if you'd like a free account on the tool so you can run your own site validations :)

  • 👍1