Removing radiogroup role from radios component to solve accessibility issues (#131)

sarahrose-ferris1-NHSD · web-flow · commit f5339f8811a6 · 2022-12-13T16:50:38.000Z
diff --git a/src/components/radios/Radios.tsx b/src/components/radios/Radios.tsx
@@ -28,9 +28,6 @@ class Radios extends PureComponent<RadiosProps, RadiosState> {
 
   static Radio = Radio;
 
-  static defaultProps = {
-    role: 'radiogroup',
-  };
 
   constructor(props: RadiosProps) {
     super(props);
diff --git a/src/components/radios/__tests__/__snapshots__/Radios.test.tsx.snap b/src/components/radios/__tests__/__snapshots__/Radios.test.tsx.snap
@@ -4,21 +4,18 @@ exports[`Radios matches snapshot 1`] = `
 <Radios
   id="example"
   name="example"
-  role="radiogroup"
 >
   <FormGroup
     id="example"
     inputType="radios"
     name="example"
-    role="radiogroup"
   >
     <div
       className="nhsuk-form-group"
     >
       <div
         className="nhsuk-radios"
         id="example"
-        role="radiogroup"
       >
         <Radio
           id="example-1"

Original file line number	Diff line number	Diff line change
@@ -4,21 +4,18 @@ exports[`Radios matches snapshot 1`] = `
`4`	`4`	`<Radios`
`5`	`5`	`id="example"`
`6`	`6`	`name="example"`
`7`		`- role="radiogroup"`
`8`	`7`	`>`
`9`	`8`	`<FormGroup`
`10`	`9`	`id="example"`
`11`	`10`	`inputType="radios"`
`12`	`11`	`name="example"`
`13`		`- role="radiogroup"`
`14`	`12`	`>`
`15`	`13`	`<div`
`16`	`14`	`className="nhsuk-form-group"`
`17`	`15`	`>`
`18`	`16`	`<div`
`19`	`17`	`className="nhsuk-radios"`
`20`	`18`	`id="example"`
`21`		`- role="radiogroup"`
`22`	`19`	`>`
`23`	`20`	`<Radio`
`24`	`21`	`id="example-1"`