Skip to content

Improve accessibility (CFM-491) #384

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

arlen
Copy link
Contributor

@arlen arlen commented Mar 30, 2026

This PR improves a number of accessibility issues, adding aria attributes where appropriate and ensuring that labels are present for all input fields.

@arlen arlen requested review from Ioannis and benno March 30, 2026 18:15
Comment on lines 33 to 50
msgid "aria.label.actions"
msgstr "Actions menu"

msgid "aria.label.lightbox"
msgstr "Registry lightbox"

msgid "aria.label.lightbox.main"
msgstr "Registry lightbox content"

msgid "aria.label.search"
msgstr "Search and user menu"

msgid "aria.label.subnavigation"
msgstr "Subnavigation"

msgid "aria.label.subnavigation.links"
msgstr "Subnavigation secondary links"

Copy link
Contributor

Choose a reason for hiding this comment

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

information.po doesn't quite feel right as the place to put the aria labels. My first thought was to add a new accessibility.po file, but that doesn't feel right either, especially when most accessibility translations should probably be closer to their context (eg: field.po:foo.alttext).

Some of these maybe go in menu.po?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've moved them into menu.po.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

...and rebased against the latest develop

@arlen arlen force-pushed the feature-cfm491-accessibilityImprovements branch from 06180be to 85c332c Compare April 7, 2026 16:14
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants