Skip to content

test(radio-button): amend validation stories#7751

Open
paulrobinson88 wants to merge 1 commit intomasterfrom
FE-7582
Open

test(radio-button): amend validation stories#7751
paulrobinson88 wants to merge 1 commit intomasterfrom
FE-7582

Conversation

@paulrobinson88
Copy link
Contributor

Proposed behaviour

Stories New Validation and New Validation Inline Radio Button Groups have unique ID's so have the correct validation message read out, either Error or Warning when using screen readers
Screenshot 2026-02-04 at 12 08 02

Current behaviour

Stories New Validation and New Validation Inline Radio Button Groups do not have unique ID's so all groups are read out as being Error validation.
Screenshot 2026-02-04 at 12 08 57

Checklist

  • Commits follow our style guide
  • Related issues linked in commit messages if required
  • Screenshots are included in the PR if useful
  • All themes are supported if required
  • Unit tests added or updated if required
  • Playwright automation tests added or updated if required
  • Storybook added or updated if required
  • Translations added or updated (including creating or amending translation keys table in storybook) if required
  • Typescript d.ts file added or updated if required
  • Related docs have been updated if required

QA

  • Tested in provided StackBlitz sandbox/Storybook
  • Add new Playwright test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Testing instructions

Check test stories New Validation and New Validation Inline have the correct validation of either Error or Warning ready out by a screen reader using VO+Arrowkeys to navigate through to the RadioButtonGroup as shown in images above.

@paulrobinson88 paulrobinson88 marked this pull request as ready for review February 4, 2026 15:42
@paulrobinson88 paulrobinson88 requested a review from a team as a code owner February 4, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants