Skip to content

Help texts, improve contrast and better UX writing for a11y#8052

Merged
Wagner3UB merged 14 commits intomainfrom
sharing-page-a11y
Apr 7, 2026
Merged

Help texts, improve contrast and better UX writing for a11y#8052
Wagner3UB merged 14 commits intomainfrom
sharing-page-a11y

Conversation

@Wagner3UB
Copy link
Copy Markdown
Contributor

@Wagner3UB Wagner3UB commented Mar 25, 2026

UI and Accessibility Fixes

  • Better Contrast - The current contrast between @secondaryBackground and @secondaryColor does not meet WCAG accessibility standards. We are increasing the color depth of the font to ensure it reaches the minimum contrast ratio required for better readability and compliance.

  • Accessibility First - The text needs to work for everyone, including people with vision problems. Visual icons alone are not enough for accessibility. For this reason, the text was updated to include all necessary information to assist users with visual impairments.

  • Safe Style Changes - Even though many of these parts use the same help class, some use their own special settings that override the default. Changing the base color to a darker shade is safe and won't break anything, even when those custom styles are active.

Technical Assessment of Affected Components

The following components implement className="help" and are being reviewed for accessibility and contrast:

Aliases.jsx
FormFieldWrapper.jsx
QueryWidget.jsx
RecurrenceWidget.jsx
Login.jsx

Issue related: #5127

@Wagner3UB Wagner3UB self-assigned this Mar 25, 2026
@Wagner3UB Wagner3UB requested a review from stevepiercy March 26, 2026 09:26
Copy link
Copy Markdown
Collaborator

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Looks good, except unit tests fail because you need to update the snapshots to include that trailing period.

@Wagner3UB Wagner3UB marked this pull request as ready for review March 27, 2026 08:08
@Wagner3UB Wagner3UB requested review from pnicolli and wesleybl March 27, 2026 08:08
@Wagner3UB Wagner3UB requested a review from wesleybl April 2, 2026 09:26
@davisagli davisagli added the 99 tag: UX Accessibility Accessibility issues label Apr 4, 2026
Copy link
Copy Markdown
Member

@wesleybl wesleybl left a comment

Choose a reason for hiding this comment

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

The contrast is much better now.

@Wagner3UB Wagner3UB merged commit 863b75f into main Apr 7, 2026
80 checks passed
@Wagner3UB Wagner3UB deleted the sharing-page-a11y branch April 7, 2026 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

99 tag: UX Accessibility Accessibility issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants