Skip to content

Add section on success criteria filtering#3022

Merged
mattgarrish merged 9 commits into
mainfrom
understand/sc-filter
Jun 18, 2026
Merged

Add section on success criteria filtering#3022
mattgarrish merged 9 commits into
mainfrom
understand/sc-filter

Conversation

@mattgarrish

@mattgarrish mattgarrish commented Jun 16, 2026

Copy link
Copy Markdown
Member

Okay, new attempt. Instead of subsections for each content type and having to re-explain the limitations, I've restructured it so there is now an intro to explain once what we're trying to achieve. I then put all the guidance on identifying the content types into a separate section. And there's a final section with a table to cover the content types and what can be excluded.

The non-text content still doesn't fit with the table since it's additive. To put it in the table we'd have to re-list all the success criteria for the other content types. If we assume that CSS animations are almost never used, we can also add a few other success criteria. So, I've listed these additional SC after the table to try and make this easier to follow.

A direct link to the preview for the new section is here:
https://raw.githack.com/w3c/epub-specs/understand/sc-filter/epub34/a11y-understand/index.html#eval-filter

Edit: I also removed the script to fix the errata links from this document and the techniques because notes don't get errata.


Preview | Diff

Comment thread epub34/a11y-understand/index.html Outdated
Comment thread epub34/a11y-understand/index.html Outdated
Comment thread epub34/a11y-understand/index.html Outdated
Comment thread epub34/a11y-understand/index.html Outdated
Comment thread epub34/a11y-understand/index.html Outdated
Comment thread epub34/a11y-understand/index.html Outdated
@sueneu

sueneu commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This is making so much more sense to me now. Thank you!

Can we rename the mapping table? Since it is only for filtering success criteria, maybe if we name it something that indicates that people won't be tempted to misuse it.

Something like
"Inapplicable Success Criteria"
"When Success Criteria Can Be Skipped"

Comment thread epub34/a11y-understand/index.html Outdated
</section>

<section id="eval-filter-table">
<h4>Content-specific success criteria</h4>

@sueneu sueneu Jun 16, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
<h4>Content-specific success criteria</h4>
<h4>Filtering guide</h4>

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Changing it to "Inapplicable success criteria" makes it sound like the section defines SC that never apply to EPUB. Which ones are inapplicable is a mix and match.

At one point I was using the title "Filtering table" for the section (hence the id). I'd lean to something more like that to keep it connected with the main section title. "Filtering guide" might also work.

Co-authored-by: Susan Neuhaus <susan@susanneuhaus.com>
@murata2makoto

murata2makoto commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

If the sentence "The content does not pass because the requirements have been met" still remaints, it is a bit hard to understand for non-native speakers.

Addition after seeing comments from @sueneu and @mattgarrish: Sorry about my mistake. I thought that 4.2 was also added as part of this PR.

@sueneu

sueneu commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

@murata2makoto
I gave this a try. Please see CR
Update accessibility reporting explanation #3024

@mattgarrish

Copy link
Copy Markdown
Member Author

I believe it would be best to make a new PR for this

Yes, it confused me at first because I didn't find it in the text for this new section so I thought it was something from the closed PR. Let's not start a side discussion here.

Comment thread epub34/a11y-understand/index.html Outdated
Comment thread epub34/a11y-understand/index.html Outdated
@mattgarrish mattgarrish merged commit 75359dc into main Jun 18, 2026
2 checks passed
@mattgarrish mattgarrish deleted the understand/sc-filter branch June 18, 2026 14:52
@github-project-automation github-project-automation Bot moved this from In review to Done in PM/EPUB issues Jun 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants