mirror of
https://github.com/serrebidev/BlindRSS.git
synced 2026-08-13 13:29:27 -07:00
Add screen reader announcements for adding/removing articles to/from favorites via keyboard shortcut #57
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
serrebi/BlindRSS#57
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Add screen reader announcements when a user adds or removes an article to/from favorites using the Ctrl+D keyboard shortcut.
Checklist:
Done in v1.107.0 — all three checklist items.
Two details worth knowing:
The announcement is emitted before the row is removed in the Favorites view. Un-favoriting there drops the article from the list, so announcing afterwards would have given you silence plus a vanished row.
You were right that this needed fixing rather than being a nice-to-have: Ctrl+D changes no visible text in the row, so previously a successful toggle and a no-op were indistinguishable to a screen reader.
One thing I did not do: the Russian translations for "Added to favorites"/"Removed from favorites" are intentionally left untranslated for now.
msgmergeoffered "Добавить в избранное" / "Удалить из избранного" — but those are the imperative menu commands ("Add to favorites"), and announcing a command after the fact reads wrong; it should be something like "Добавлено в избранное". Rather than ship that, the strings fall back to English until a translator supplies the right form. Same for "Favorites change", which msgmerge guessed as "Изменение фильтра" ("Filter change").