use list for a11y in RadioButtonGroup, see #361

phetsims · May 17, 2018 · f404165 · f404165
1 parent 26d015d
commit f404165
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/js/buttons/RadioButtonGroup.js b/js/buttons/RadioButtonGroup.js
@@ -52,7 +52,7 @@ define( function( require ) {
       tandem: Tandem.required,
 
       // a11y
-      tagName: 'fieldset',
+      tagName: 'ul',
       groupFocusHighlight: true
     }, options );
 

diff --git a/js/buttons/RadioButtonGroupMember.js b/js/buttons/RadioButtonGroupMember.js
@@ -65,6 +65,7 @@ define( function( require ) {
       tagName: 'input',
       inputType: 'radio',
       labelTagName: 'label',
+      containerTagName: 'li',
       appendDescription: true,
       appendLabel: true,