Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

New rule: title or aria-label required on page break markers #71

Closed
rdeltour opened this issue Oct 13, 2017 · 2 comments
Closed

New rule: title or aria-label required on page break markers #71

rdeltour opened this issue Oct 13, 2017 · 2 comments
Assignees
@rdeltour
Labels
feature rules
Milestone
v0.6.0 “Beta 2”

Comments

@rdeltour
Copy link
Member

As recommended in EPUB Accessibility Technique PAGE-001
@rdeltour rdeltour self-assigned this Oct 20, 2017
@rdeltour rdeltour added this to the Beta 2 milestone Oct 20, 2017
rdeltour added a commit that referenced this issue Oct 29, 2017
@rdeltour
feat(rules): ensure that page break markers have an accessible label
a078739 
Configure a new aXe rule to ensure that page break markers (identified with the ARIA
role `doc-pagebreak` or with `epub:type="pagebreak"`) have an accessible label defined
in a `title` attribute or an `aria-label` attribute.

Closes #71
@rdeltour rdeltour mentioned this issue Oct 29, 2017
Merged
@clapierre
Copy link

clapierre commented Jun 16, 2021 •
edited
Loading

I think this may be too strict?
If the author exposes the Page numbers to everyone, there would be no need to require an aria-label as well.

IE: The following should be valid, but currently causes an ACE Serious Issue.

<span epub:type=“pagebreak” role=“doc-pagebreak” id=“page_1”>Page 1</span>

@rdeltour rdeltour mentioned this issue Jun 17, 2021
Open
@rdeltour
Copy link
Member Author

Thanks for the report Charles. I created a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdeltour rdeltour

Labels
feature rules
Projects
None yet
Milestone
v0.6.0 “Beta 2”
Development

No branches or pull requests
2 participants
@rdeltour @clapierre