Skip to content

Conversation

gitdallas
Copy link
Contributor

Closes #9695

@patternfly-build
Copy link
Contributor

patternfly-build commented Oct 4, 2025

Signed-off-by: gitdallas <[email protected]>
@gitdallas gitdallas force-pushed the feat/9695-checkbox-radio-aria-describedby branch from a15e207 to 1eba459 Compare October 5, 2025 14:55
Copy link
Contributor

@mcoker mcoker left a comment

Choose a reason for hiding this comment

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

LGTM though since there is no shiny CSS/UI stuff here, I'm just testing that it works like expected.

  • Passing aria-describedby='' both with and without a description prevented aria-describedby from rendering ✅
  • Passing aria-describedby="foo" rendered aria-describedby="foo" both with and without a description ✅
  • Not passing aria-describedby did nothing if there is no description ✅
  • Not passing aria-describedby with a description properly added aria-description="id-of-description"

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.

Checkbox/Radio - add support for aria-describedby

4 participants