Aria live region has been added and hooked up to mw.notify. If a string or mw.message gets passed into mw.notify it will be added to the live region and thus read to screen readers. If instead an object gets passed into mw.notify we can pass in an options.ariaText to specify what to add to the live region.
The reading list notifications are unnecessarily passing in an object instead of an mw.message. Remove this so that the notification gets added to the live region.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Make notification message available for ARIA live region | mediawiki/extensions/ReadingLists | master | +1 -10 |
Change #1201715 had a related patch set uploaded (by LorenMora; author: LorenMora):
[mediawiki/extensions/ReadingLists@master] Make notification message availble for live region
Change #1201715 merged by jenkins-bot:
[mediawiki/extensions/ReadingLists@master] Make notification message available for ARIA live region
Per @LMora-WMF this is validated by this task T393510#11368160, once the open questions are addressed there, this can be moved to sign off.
I tried screen reader mode and the audio for notification mentions the article twice.
Please add codebase project tags to tasks, so other people who don't know or don't care about WMF team names can also find this task when searching via projects or looking at workboards. Thanks.
Checked on testwiki wmf.4 - no issues found. Adding Verified tag and boldly sign off.
I tried VoiceOver (MacOS) and the issue was not present. Which screen recorder did you do the testing?
@Etonkovidova I tried Mac voiceover too. It only happens in certain cases. The first time you save it does not happen but it happens if you unsave and save without tabbing to other options. I have attached the video here