Skip to content

Conversation

clapierre
Copy link
Collaborator

No description provided.

@clapierre
Copy link
Collaborator Author

@mattgarrish All your improvements have been included as well as adding two missing "annotations" and "unlocked"
let me know what you think now.
Thanks

<h4>Structure and navigation terms</h4>

<section id="metadata-a11y-features-annotations">
<h5>Annotations</h5>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weren't we just discussing deprecating this value, though? I thought the argument against it was that the annotations are generally for all readers and rarely address a specific accessibility need.

@clapierre
Copy link
Collaborator Author

clapierre commented Jun 20, 2025 via email

@mattgarrish
Copy link
Member

No, we talked about it but I think it was left to us to take up in the schema.org vocabulary document. I just want to make sure I'm remembering that right.

@madeleinerothberg @avneeshsingh does that match your thinking on this value?

It might be good to remove the section and I'll open a pull request in the other repository to mark the property as deprecated.

@clapierre
Copy link
Collaborator Author

@mattgarrish Ok I think we are ready for the final review here before merging all the features.

@clapierre clapierre requested a review from mattgarrish July 11, 2025 15:03
@clapierre
Copy link
Collaborator Author

Hi @mattgarrish @gregoriopellegrino @GeorgeKerscher
I think this is ready to be merged which includes all accessibilityFeatures, if you can add your review.

Here is a link to the rendered view of the PR.[1]

[1] https://raw.githack.com/w3c/publ-a11y/pull/731/head/epub-a11y-meta-guide/1.0/draft/index.html

@GeorgeKerscher
Copy link
Collaborator

I did not go through each line of added code, but what I went through is looking good!

@clapierre clapierre merged commit 636438d into main Jul 23, 2025
@clapierre clapierre deleted the EPUB-metadata---Features branch July 23, 2025 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants