Skip to content

Conversation

@PosikBoy
Copy link
Contributor

Closes #919

@PosikBoy PosikBoy changed the title feat(a11y):Add aria labels to buttons feat(a11y): Add aria labels to buttons Jan 12, 2026
@PosikBoy
Copy link
Contributor Author

That was fast)

@gravity-ui
Copy link

gravity-ui bot commented Jan 12, 2026

🎭 Playwright Report

Comment on lines 11 to 12
"preview_hint": "Preview",
"settings_label": "Settings"
Copy link
Member

Choose a reason for hiding this comment

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

I think it would be better to name both keys similarly: preview_label, settings_label

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed key name

Copy link
Member

Choose a reason for hiding this comment

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

Something went wrong - checks failed

Copy link
Member

Choose a reason for hiding this comment

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

"preview_label" in en.json
but "preview_hint" in ru.json

@makhnatkin makhnatkin changed the title feat(a11y): Add aria labels to buttons feat: improve button a11y with aria labels Jan 13, 2026
@makhnatkin makhnatkin merged commit 4bde8ba into gravity-ui:main Jan 13, 2026
5 of 8 checks passed
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.

feat(a11y): Add aria-label to settings and preview buttons

3 participants