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.

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

New rule: report absence of ARIA roles along side epub:type #70

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

New rule: report absence of ARIA roles along side epub:type #70

rdeltour opened this issue Oct 13, 2017 · 8 comments

Comments

@rdeltour
Copy link
Member

The DPUB ARIA roles have well-defined accessibility API mappings, and should be used in conjunction to epub:type.

Ace could include a rule to check that, and report as a "moderate" violation.

@mattgarrish thoughts welcome

@mattgarrish
Copy link

That would be useful, IMO.

We want people to stop thinking of epub:type as an accessibility mechanism, and the guidance is to prefer ARIA for accessibility but use both if you need some specific reading system functionality only supported by epub:type.

@rdeltour
Copy link
Member Author

ok thanks, that's what I needed to know!

@laudrain
Copy link

Hi,
to start with, I don't see epub:type being reported in ace. Is it missing?

@rdeltour
Copy link
Member Author

I don't see epub:type being reported in ace. Is it missing?

You're right, it's not reported.

To clarify, Ace will:

  • report violations of some a11y guidelines (think WCAG + some EPUB-specific rules).
  • extract some useful data from Content Documents, for later use in manual inspection.

@laudrain, are you saying some epub:types should be extracted as useful data? which and why?

@laudrain
Copy link

laudrain commented Oct 18, 2017

Some epub:type yes:

  1. high level for logical structure:
    level 1: frontmatter
    and inside, level 2: cover, tiltepage, copyright-page, ...
    level 1: bodymatter
    and inside, level 2: part, chapter, subchapter, division
    level 1: backmatter
    and inside, level 2: appendix, credits, index

  2. specifics:

  • pagebreak for page navigation checks
  • footnote, endnote for notes in HTML aside

@rdeltour
Copy link
Member Author

Some epub:type yes

to be followed in #87...

@rdeltour rdeltour self-assigned this Oct 20, 2017
@rdeltour rdeltour added this to the Beta 2 milestone Oct 20, 2017
@rdeltour
Copy link
Member Author

rdeltour commented Oct 29, 2017

Moving out of Beta 2

This rule can be quite disturbing at first for many people who've been told epub:type was good for accessibility, and will create many violation reports. I think it would be better to wait until we have an option to finely configure the rules run by Ace, so that people can easily disable this rule if it pollutes their report.

Waiting on #77

@mattgarrish
Copy link

Sorry, looks like I missed some notifications. I wouldn't make this an error even when implemented. WCAG only requires that user interface components use ARIA roles. This is more like a best practice for a best practice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants