Form - support for role="group" and role="radiogroup"

[mcoker]

patternfly/patternfly#4240

A form group with a single label and multiple form elements (text inputs, checkboxes, etc), should have role="group" with aria-labelledby the id of the <label> in that form group.

• Exception to this is if the elements are a grouping of radio buttons where only one can be selected at a time, use role="radiogroup"

cc @jessiehuff

[jessiehuff]

I think the question I'd have here is which things should be following this pattern? How is this different from the form sections? Our pattern should probably be consistent across. Are we updating our existing examples to use this same pattern? I think it just requires a little clarity. 🙂

[mcoker]

How is this different from the form sections? Our pattern should probably be consistent across.

Opened this issue to add to form sections - patternfly/patternfly#4276. Feel free to add any additional comments.

Are we updating our existing examples to use this same pattern?

We updated the examples in patternfly/patternfly#4240, so I would expect the form examples to be updated here, too. WDYT?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[mcoker]

not stale

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[nicolethoen]

The react examples need to be updated to match the core examples with the use of role="group" as was implemented in this PR patternfly/patternfly#4240.

@mcarrano can we add this to a milestone?