survey-library icon indicating copy to clipboard operation
survey-library copied to clipboard

A11Y: radiogroup redundant labelling (VoiceOver)

Open saltmade opened this issue 3 years ago • 5 comments

Are you requesting a feature, reporting a bug or asking a question?

Reporting a bug.

What is the current behavior?

Screenreaders read redundant information when selecting radio inputs. For example,

What is the expected behavior?

Each choice would be read once. If the label contains text content there's no need for an aria-label on the <label>

How would you reproduce the current behavior (if this is a bug)?

  1. Open Voiceover
  2. Navigate to the radio sample page
  3. tab to a choice and see it read twice

Provide the test code and the tested page URL (if applicable)

Tested page URL: https://surveyjs.io/Examples/LibrarySinglePage?id=questiontype-radiogroup&platform=Reactjs&theme=modern

Specify your

  • browser: Firefox
  • browser version: 99.0b3
  • surveyjs platform (angular or react or jquery or knockout or vue): React
  • surveyjs version: 1.9.19

saltmade avatar Mar 15 '22 17:03 saltmade

I've checked the example with the nvda reader and it looks ok:

image

Probably there is dome difference between nvda, narrator, Voiceover etc

Here is the etalon example: https://www.w3.org/TR/wai-aria-practices-1.1/examples/radio/radio-1/radio-1.html could you please check it with the Voiceover? Does it sound well?

dmitry-kurmanov avatar Mar 15 '22 18:03 dmitry-kurmanov

The ARIA example works fine in Voiceover, with no issues of duplication. It also does not duplicate the text content of the radio in an aria-label, as described in the issue.

saltmade avatar Mar 15 '22 19:03 saltmade

@saltdotac in that case it looks like a bug I will investigate the issue and write back asap

dmitry-kurmanov avatar Mar 16 '22 06:03 dmitry-kurmanov

We've done some investigations and can't find fast way to fix the problem. We decided to add this ticket to our backlog for a while and we will return to it a bit later.

dmitry-kurmanov avatar May 26 '22 13:05 dmitry-kurmanov

@dmitrykurmanov Thanks for the update! I appreciate y'all still working on it.

saltmade avatar May 26 '22 15:05 saltmade

closed in favor of https://github.com/surveyjs/survey-library/issues/6859

dmitry-kurmanov avatar Sep 11 '23 10:09 dmitry-kurmanov