tidalcycles / strudel

Web-based environment for live coding algorithmic patterns, incorporating a faithful port of TidalCycles to JavaScript
https://strudel.cc/
GNU Affero General Public License v3.0
675 stars 119 forks source link

Improve Screen Reader Accessibility #1156

Open nicopn opened 3 months ago

nicopn commented 3 months ago

I'm a blind Strudel user who uses a screen reader to navigate, and though accessibility is pretty good in my opinion, it can still be improved in some ways

yaxu commented 3 months ago

Thanks @nicopn ! Glad to hear it's working to some extent.

Looking at the audio output device combo box and friends, I can see it does actually have a label, but it's not associated with the form element - they're just side-by-side in the html. I don't know why this is, as in the jsx code the audio device selector is inside a formitem with the label. Anyone know what's going wrong?

        <FormItem label="Audio Output Device">
          <AudioDeviceSelector
            isDisabled={started}
            audioDeviceName={audioDeviceName}
            onChange={(audioDeviceName) => settingsMap.setKey('audioDeviceName', audioDeviceName)}
          />
        </FormItem>