Add headings (h1, h2, h3 and other semantic headers)

[giggio]

Problem or enhancement idea

Right now the theme does not have semantic headers. It would be a good idea to add them to menus, articles etc. This will help blind people and on SEO.

Proposed solution

Update the theme to add semantic headers.

Alternatives

Not really any available.

Additional context

I'd also review the rest of the theme for semantic elements. There is a nav and other elements, but an overall check would be useful.

[markdumay]

Thanks for the suggestion @giggio. It's worth looking into. I made a list of things to check/revise, feel free to extend the list.

• Page header (single pages) https://github.com/gethinode/hinode/blob/0a6dbccb12073b7862b9ed9a213decbb6aae70bd/layouts/header.html#L24 Consider enclosing the entire page header with <header>...</header>
• Section titles (as part of a bookshop component) https://github.com/gethinode/hinode/blob/0a6dbccb12073b7862b9ed9a213decbb6aae70bd/layouts/_partials/assets/section-title.html#L38-L53
• Nav items (replace ul/li with nav/a, see Bootstrap example https://github.com/gethinode/hinode/blob/0a6dbccb12073b7862b9ed9a213decbb6aae70bd/layouts/_partials/assets/nav.html#L70-L101
• Card title https://github.com/gethinode/hinode/blob/0a6dbccb12073b7862b9ed9a213decbb6aae70bd/layouts/_partials/assets/card.html#L17-L31