Skip to content

Conversation

@yafred
Copy link
Contributor

@yafred yafred commented Apr 12, 2025

addresses part of #17234

I made some videos to illustrate the same scenario visual vs non visual

In this video, the screen reader keeps reading from the start of the page ... I added an autofocus on the selection that triggers the new page. I think it make the experience better

@yafred yafred marked this pull request as ready for review April 12, 2025 01:38
@yafred yafred marked this pull request as draft April 12, 2025 04:35
@yafred yafred marked this pull request as ready for review April 12, 2025 18:12
const target = e.target as HTMLSelectElement;
const selectedOption = target.options[target.selectedIndex];
const url = selectedOption.getAttribute('url');
if (url) window.location.href = url;
Copy link
Collaborator

@ornicar ornicar Apr 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of a <select> with an event listener, shouldn't this just be a list of plain links in a <menu>?

Copy link
Contributor Author

@yafred yafred Apr 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In short, I think select is better

Here is what the user hear using NVDA (I put cmd: to show the commands I used)

cmd: Cntl Home, d, d, d (go to the top, scan the landmarks)
main landmark
Current group: combo box Girls collapsed
cmd: Alt arrow up and down (list the possible options in the select group)
combo box expanded
Girls list
Girls 2 of 2
Open 1 of 2
cmd: Enter to choose 'Open
document busy
main landmark
combo box collapsed
Lola Ana broadcast | Open • Round 1 • lichess.dev — Mozilla Firefox
Lola Ana broadcast | Open • Round 1 • lichess.dev busy
button Accessibility: Disable blind mode
main landmark
Current group: combo box Open collapsed

I am not sure, a menu would give such a result

Copy link
Contributor Author

@yafred yafred Apr 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There a bit of pollution that may be explained by the fact we are using snabdom (the reader starts too soon) ?

Lola Ana broadcast | Open • Round 1 • lichess.dev — Mozilla Firefox
Lola Ana broadcast | Open • Round 1 • lichess.dev busy
button Accessibility: Disable blind mode
main landmark

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, the additional text is only visible in NVDA speech viewer ... not in what you actually hear .. https://www.youtube.com/watch?v=Q6HykFcLOgw (less rushed video waiting for the vocalization to happen)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had the confirmation in #17231 that combo box is better than menu

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, then could you please refactor the duplicated event listener code?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done ... hopefully as you expected

@ornicar ornicar merged commit a191bca into lichess-org:master Apr 13, 2025
3 checks passed
@yafred yafred deleted the 17234-nvui-broadcast-round-selection branch April 14, 2025 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants