Skip to content

dbeaver/pro#5241 [CB] Screen reader does not read some fields in the application#3585

Merged
Wroud merged 12 commits intodevelfrom
5241-cb-the-screen-reader-does-not-read-some-fields-in-the-application
Jul 29, 2025
Merged

dbeaver/pro#5241 [CB] Screen reader does not read some fields in the application#3585
Wroud merged 12 commits intodevelfrom
5241-cb-the-screen-reader-does-not-read-some-fields-in-the-application

Conversation

@SychevAndrey
Copy link
Copy Markdown
Contributor

Fixed input component, linked controls with labels in field via for attribute.
This approach doesn't work for 100% in all assistive software, that's why I also defined aria-label for inputs as well.

…nd add aria-label for accessibility

even with linked label, voiceOver doesn't pronounce label when focusing textarea, that's why aria-label is used there
aria-live polite works with aria regions which initially empty and then dynamically populated with some data. If we want assistive technologies read initial content, we need to use alert role.

also hide icon from assistive technologies (no additional information from that)
sergeyteleshev
sergeyteleshev previously approved these changes Jul 15, 2025
Comment thread webapp/packages/core-blocks/src/FormControls/Textarea.tsx Outdated
Comment thread webapp/packages/core-blocks/src/FormControls/InputField/InputFieldBase.tsx Outdated
Wroud
Wroud previously approved these changes Jul 16, 2025
Wroud
Wroud previously approved these changes Jul 17, 2025
@SychevAndrey SychevAndrey force-pushed the 5241-cb-the-screen-reader-does-not-read-some-fields-in-the-application branch from 82da2e6 to f37c559 Compare July 18, 2025 08:34
@SychevAndrey SychevAndrey requested a review from Wroud July 18, 2025 08:41
@Wroud Wroud merged commit 603b94c into devel Jul 29, 2025
10 checks passed
@Wroud Wroud deleted the 5241-cb-the-screen-reader-does-not-read-some-fields-in-the-application branch July 29, 2025 15:09
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.

6 participants