Skip to content

Toast alert a11y v19#8105

Open
Wagner3UB wants to merge 5 commits intomainfrom
toast-alert-a11y-v19
Open

Toast alert a11y v19#8105
Wagner3UB wants to merge 5 commits intomainfrom
toast-alert-a11y-v19

Conversation

@Wagner3UB
Copy link
Copy Markdown
Contributor

@Wagner3UB Wagner3UB commented Apr 10, 2026

Accessibility improvements for Toast notifications

This PR improves the accessibility of the Toast component for screen readers and general UX.

Changes

Toast component (Toast.jsx)

  • Icon is now hidden from screen readers (ariaHidden) since it is purely decorative
  • Added a visually hidden label announcing the notification type (Error, Warning, Success, Information) so screen readers can identify the context before reading the content
  • Labels are fully internationalized via react-intl (defineMessages + useIntl)

Styles (main.less)

  • Added .toast-visually-hidden scoped inside .toast-inner-content to hide the type label visually while keeping it accessible to screen readers

Date formatting in event validation (validators.ts)

  • Date values in event range error messages were being displayed as raw ISO strings (e.g. 2026-04-08T15:00:00+00:00)
  • Now formatted using Intl.DateTimeFormat respecting the user's browser locale (e.g. Apr 8, 2026, 3:00 PM)

@Wagner3UB
Copy link
Copy Markdown
Contributor Author

@sneridagh Hey Victor, could you take a look at the 'validations.js' file? I changed how the dates are displayed, but I'm not sure if it might cause issues since this file is used so frequently.

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.

LGTM, except for the local settings file.

@github-project-automation github-project-automation bot moved this from In progress to Review in progress in Accessibility Apr 10, 2026
@github-project-automation github-project-automation bot moved this from In progress to Review in progress in RedTurtle priorities Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Review in progress
Status: Review in progress

Development

Successfully merging this pull request may close these issues.

2 participants