Closed jessegreenberg closed 6 years ago
This code looks very old
110 selectValueProperty.lazyLink( function( newValue ) {
111 switch( newValue ) {
112: case ( centerOption.accessibleId ):
113 self.setFocusHighlight( electronOuterFocusHighlight );
114 break;
115: case ( innerRing.accessibleId ):
116 self.setFocusHighlight( electronInnerFocusHighlight );
117 break;
118: case ( outerRing.accessibleId ):
119 self.setFocusHighlight( nucleusFocusHighlight );
120 break;
...
137 }
138
139: var nextElementId = optionNodes[ currentIndex ].accessibleId;
140 self.setAccessibleAttribute( 'aria-activedescendant', nextElementId );
141 selectValueProperty.set( nextElementId );
This code is using aria-activedescendant, which there currently isn't support for in scenery. I made https://github.com/phetsims/scenery/issues/873 to track the implementation, but since it isn't needed for anything, I am not sure we will get to this soon.
Ill remove the current usage of self.setAccessibleAttribute( 'aria-activedescendant', nextElementId );, and we need to remove the references of accessibleId. We should use a map instead that maps from active option to focus highlight.
Removing aria-activedescendant showed another problem with the way that focus highlights are updating. Ill make an issue in scenery.
This is no longer occurring with fuzzBoard.
While testing fuzzBoard, I encountered this error in build-an-atom.