Skip to content

Conversation

@nilloq
Copy link
Member

@nilloq nilloq commented Dec 15, 2025

Types of change

  • Non-breaking change

Context & Motivation

Update input (text, area and select) accessibility for helper text, helper link and errors

Description

  • .error-text are now referenced in aria-describedby for better vocalisation order. aria-describedby value should be dynamically updated replacing helper text reference with error-text reference when the input becomes invalid
  • .helper-link are now "standalone" and must have their own aria-describedby combining their own id with the one of the related input
  • additional .visually-hidden span is recommended to clarify the link purpose

Checklists

  • I have read the contributing guidelines
  • My change follows the developer guide
  • My change pass all tests
  • My change is compatible with a responsive display
  • I have added tests (Javascript unit test or visual) to cover my changes
  • My change introduces changes to the documentation that I have updated accordingly
    • Title and DOM structure is correct
    • Links have been updated (title changes impact links)
    • CSS for the documentation
  • I have checked all states and combinations of the component with my change
  • I have checked all the impacts for the other components and core behavior (grid, reboot, utilities)

Checklist (for Core Team only)

  • The changes need to be in the migration guide
  • The changes are well displayed in Storybook (be careful if example order has changed for DSM)
  • The changes are compatible with RTL
  • Manually test browser compatibility with BrowserStack (Chrome 120, Firefox 121, Edge 120, Safari 15.6, iOS Safari, Chrome & Firefox on Android)

Progression (for Core Team only)

Live previews

@netlify
Copy link

netlify bot commented Dec 15, 2025

Deploy Preview for boosted ready!

Name Link
🔨 Latest commit 712b172
🔍 Latest deploy log https://app.netlify.com/projects/boosted/deploys/694509a407e1e6000817b02c
😎 Deploy Preview https://deploy-preview-3268--boosted.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nilloq nilloq changed the title fix: input errors accessibility fix: input accessibility improvements Dec 15, 2025
@boosted-bot boosted-bot moved this from In Progress / Draft to Need Lead Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Dec 18, 2025
@louismaximepiton louismaximepiton marked this pull request as ready for review December 18, 2025 15:45
@boosted-bot boosted-bot moved this from Need Lead Dev Review to Need Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Dec 18, 2025
@boosted-bot boosted-bot moved this from Need Dev Review to Need Lead Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Dec 19, 2025
@hannahiss hannahiss closed this Dec 19, 2025
@hannahiss hannahiss deleted the ouds/main-sco-input-accessibility branch December 19, 2025 08:54
@github-project-automation github-project-automation bot moved this from Need Lead Dev Review to Done in 🟣 [Orange-Boosted-Bootstrap] PRs Board Dec 19, 2025
@vprothais vprothais restored the ouds/main-sco-input-accessibility branch December 19, 2025 08:56
@vprothais vprothais reopened this Dec 19, 2025
@boosted-bot boosted-bot moved this from Done to Need Dev Review in 🟣 [Orange-Boosted-Bootstrap] PRs Board Dec 19, 2025
@vprothais vprothais merged commit 0b0c1d8 into ouds/main Dec 19, 2025
21 checks passed
@vprothais vprothais deleted the ouds/main-sco-input-accessibility branch December 19, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📖 documentation Improvements or additions to documentation

Projects

Development

Successfully merging this pull request may close these issues.

5 participants