Fix comments heading order#8876
Conversation
|
I converted this to draft for now because of some inconsistencies I found with the heading orders right now in Decidim. E.g. in debates, the debate title is a In order to merge this, we need to solve first the issue with the process subtitle I raised at the related discussion: |
|
The process/participatory space subtitle issue will be solved by #8901. After that PR is merged, I'll merge with |
|
So since this keeps the build green, we should first merge this before the other fixes. Both #8901 and #8877 require this to be merged first. @andreslucena Can you take a look at this PR before those other PRs, so that we can keep the build green while merging these? |
|
Can you please backport to v0.26-stable 🙏🏽? Thanks |
|
Backport at #8950. |
🎩 What? Why?
The comments section heading order is illogical.
Also, the comments main heading should be a
<h3>in the Decidim default layout for it to be logical compared to most of the contexts where the comment section is displayed in the Decidim default layout.Currently e.g. on this page:
https://meta.decidim.org/processes/RedesignDecidim/f/1662/posts/235
The current
<h4>level indicates that the comments section is a subsection of the last text section titled "Methodology: Open design / participated design". The correct top-level context of the comments section should be the blog article's title which is<h2>and therefore<h3>makes more sense for the comments section.Many screen reader users users glance the page quickly by going through its headings in which case this can be confusing.
WCAG 2.2 / 1.3.1 Info and Relationships (Level A)
https://www.w3.org/TR/WCAG22/#info-and-relationships
https://www.w3.org/WAI/WCAG22/Understanding/info-and-relationships.html
📌 Related Issues
Testing
Give the linked blog page to a visually impaired user and ask them to read it.
📋 Checklist
docs/.