daisy / ace

Ace by DAISY, an Accessibility Checker for EPUB
MIT License
74 stars 22 forks source link

Page numbers are displayed in outlines tab #415

Open mkraetke opened 2 months ago

mkraetke commented 2 months ago

ACE shows the page numbers in the "Headings Outline" and the "HTML Outline" section. This is confusing for users especially if you have numbered headlines. For example, we have tagged the page number but it rather looks like a heading number in the "Headings Outline".

<h2 id="d335992e274" class="_notoc U3 notoc" title="2. Perspektiven der Soziologie auf Arbeit im Kapitalismus">
  <span class="pagenum" title="12" id="page_12" epub:type="pagebreak" role="doc-pagebreak">12</span>
  2. Perspektiven der Soziologie auf Arbeit im Kapitalismus
</h2>

I would like to suggest omitting the page numbers in these views as they do not provide important information. Furthermore, they can be easily identified by their role or epub:type attribute.

mattgarrish commented 2 months ago

If you put the page number inside the heading tag, then it is part of the heading and is computed in the accessible name (e.g., it would be read out in the landmarks). That's not what end users are going to want to hear.

This should really be raised as a violation of SC 1.3.1 since the page number is not heading text.

gregoriopellegrino commented 2 months ago

The fact is that there should not be pagebreaks within a heading, but should be before or after. It violates WCAG Success Criterion Info and Relationships.

mkraetke commented 2 months ago

Thank you very much for the quick reply! Then we'll change this on our side. An additional check in ACE would be appreciated as well.