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

a11y:exemption is not recognized by epubcheck #1566

Open
jhswift opened this issue Jul 8, 2024 · 2 comments
Open

a11y:exemption is not recognized by epubcheck #1566

jhswift opened this issue Jul 8, 2024 · 2 comments
Assignees
Labels
status: has PR The issue is being processed in a pull request type: false-positive This issue is about valid content being incorrectly rejected

Comments

@jhswift
Copy link

jhswift commented Jul 8, 2024

The relatively new property a11y:exemption is not recognised by the 5.1 version of epubcheck.
If that property is used in the opf file, epubcheck give this error:
Col: 63: ERROR(OPF-027): Undefined property: ""a11y:exemption"".

@mattgarrish
Copy link
Member

Ya, I don't think it's helpful for epubcheck to enforce what properties are allowed in the a11y vocabulary. It's not like the package document vocabularies in the core spec that have elements that require their use.

It could maybe be a usage message to help people make sure they aren't misspelling property names, but we always expected to extend the vocabulary and this is the second time in the last year alone where we've done it.

@rdeltour rdeltour added this to the Next maintenance release milestone Dec 26, 2024
@rdeltour rdeltour self-assigned this Dec 26, 2024
@rdeltour rdeltour added status: in progress The issue is being implemented by the development team type: false-positive This issue is about valid content being incorrectly rejected labels Dec 26, 2024
@rdeltour
Copy link
Member

I don't think it's helpful for epubcheck to enforce what properties are allowed in the a11y vocabulary. It's not like the package document vocabularies in the core spec that have elements that require their use.

It could maybe be a usage message to help people make sure they aren't misspelling property names, but we always expected to extend the vocabulary and this is the second time in the last year alone where we've done it.

Yeah, good point.

There is currently no way in the EPUBCheck vocabulary declaration to say "these values are ok, and all other values are ok too but should be reported as usage". It's either a fully-checked vocabulary or a fully permissive one.

For now, I'm just adding the a11y:exemption to the list of known properties, but will consider adding a new vocabulary type to implement what you suggest 👍

rdeltour added a commit that referenced this issue Dec 27, 2024
@rdeltour rdeltour added status: has PR The issue is being processed in a pull request and removed status: in progress The issue is being implemented by the development team labels Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: has PR The issue is being processed in a pull request type: false-positive This issue is about valid content being incorrectly rejected
Projects
None yet
Development

No branches or pull requests

3 participants