Skip to content

Conversation

pangolingo
Copy link
Contributor

@pangolingo pangolingo commented May 2, 2025

The "actions" columns in the collection table did not have any text in the column header cells. This was fine visually because it's a common pattern to use the final cells for action buttons. But people using a screen readers may not have this visual context. By adding a title of "Actions" to the column, this title is now announced by screen readers when moving into that column.

This addresses on of the accessibility issues listed in #794

Screenshot 2025-06-06 at 12 10 09 PM

@pangolingo
Copy link
Contributor Author

I'm marking this as a draft PR because I used an LLM to generate translations for the new column. These should be manually checked.

The "actions" columns in the collection table did not have any text in the column header cells. This was fine visually because it's a common pattern to use the final cells for action buttons. But people using a screen readers may not have this visual context. By adding a title of "Actions" to the column, this title is now announced by screen readers when moving into that column.
@pangolingo pangolingo force-pushed the di-accessibility-fix-1 branch from 3f94f4b to 3e6c6a6 Compare May 2, 2025 21:21
@pangolingo pangolingo mentioned this pull request May 2, 2025
4 tasks
@pablobm
Copy link
Collaborator

pablobm commented Jun 6, 2025

Any idea on how we can check those translations? 😅

@pangolingo
Copy link
Contributor Author

I will see if I can crowdsource the translations from my thoughtbot coworkers. We can probably get 70% of them.

@pangolingo
Copy link
Contributor Author

pangolingo commented Jun 6, 2025

Running list of validated translations:

ar/Arabic
bs/Bosnian
ca/Catalan
da/Danish
de/German: Aktionen
en/English: Actions
es/Spanish: Acciones
fi/Finnish
fr/French: Actions
id/Indonesian
it/Italian: Azioni
ja/Japanese
ko/Korean: 작업
nl/Dutch
pl/Polish
pt-BR/Portuguese (Brazil): Ações
pt/Portuguese: Ações
ru/Russian
sl/Slovenian
sq/Albanian
sv/Swedish: Åtgärder
tr/Turkish: İşlemler
uk/Ukrainian
vi/Vietnamese
zh-CN/Chinese (Simplified): 操作
zh-TW/Chinese (Traditional): 操作

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.

2 participants