Skip to content

Add new failure technique for non-semantic ruby annotation#4829

Open
adampage wants to merge 11 commits intow3c:mainfrom
adampage:ruby-new-failure
Open

Add new failure technique for non-semantic ruby annotation#4829
adampage wants to merge 11 commits intow3c:mainfrom
adampage:ruby-new-failure

Conversation

@adampage
Copy link
Member

@adampage adampage commented Dec 27, 2025

@netlify
Copy link

netlify bot commented Dec 27, 2025

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 4e35c22
🔍 Latest deploy log https://app.netlify.com/projects/wcag2/deploys/6966ba61484c90000810b536
😎 Deploy Preview https://deploy-preview-4829--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@giacomo-petri
Copy link
Contributor

@adampage, love this failing technique! I would also suggest adding a reference to 1.3.2 (at the top as well as describing it in description/examples), because expecially in example 3, the meaning changes when the text is read in sequence.

@adampage
Copy link
Member Author

adampage commented Jan 9, 2026

I would also suggest adding a reference to 1.3.2 (at the top as well as describing it in description/examples), because expecially in example 3, the meaning changes when the text is read in sequence.

Oh yes, excellent, I’ll add that. Thanks, @giacomo-petri. 🙏🏻

@adampage adampage marked this pull request as ready for review January 9, 2026 15:57
@adampage
Copy link
Member Author

Hi @kfranqueiro, I temporarily authored this one as F999.html and used that for the assorted file references. Is there a right way for me to choose/reserve a new failure number?

kfranqueiro pushed a commit that referenced this pull request Jan 13, 2026
…ang]` element (#4852)

In #4829, cspell gave me [“unknown word” failures for “_tou_” and
“_shi_”](https://github.com/w3c/wcag/actions/runs/20857561983), so I
resolved them by [adding a file-specific
override](https://github.com/w3c/wcag/pull/4829/files#diff-911db262cf96b6fa444f9cf0fc8fe00952325cda6b66c53952ea15d552ef42adR58-R59).

A little further down in `cspell.yml`, I noticed a regex for ignoring
any elements with the `lang` attribute — which is exactly where my
unknown words had occurred in #4829. However, my `lang` content
contained multiple lines, whereas the regex seems to be looking for
single line matches.
@kfranqueiro
Copy link
Contributor

There isn't really a way to "reserve" per se - they're primarily first-come first-serve.

We used to wait until the last minute and use temporary slug filenames until we were ready to merge, but more recently we resolved to take the next unoccupied ID, and if that shows a conflict by the time we're ready to merge it, increment it again.

@adampage
Copy link
Member Author

Thanks, @kfranqueiro. I’ve snagged the next unoccupied ID and will be ready to bump it once more if necessary.

@patrickhlauke
Copy link
Member

discussed on the TF call. need further research/testing/checking with native Japanese SR users to work out accessibility support of the "correct" way to mark these things up.

@adampage
Copy link
Member Author

Hello, @murata2makoto and @MakotoUeki. It’s a pleasure to meet you both.

If you have time, I’d be very grateful for any feedback you might be willing to share on this pull request. Your perspectives and recommendations would be very valuable.

Here are preview links for the new failure technique and three supporting examples:

I’ve also authored a companion PR — #4830 — which mainly updates the existing H62 technique to remove usage of the deprecated <rb> element, but also introduces a new supporting example.

Here are preview links for those:

Thank you very much for your time and consideration.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

We need a failure technique for Ruby

4 participants