-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Place cursor at highlighted item name in construction and crafting menus as expected by screen readers #39588
Conversation
construction menu: works like a charm. |
weird. the cursor is always on the second item from the bottom in the visible part of the list. |
…ted by screen readers
Perhaps crafting menu didn't work because the window was not refreshed. Please test the new patch and see if it works. |
yay! now all is working. @Qrox great job. |
Thanks a lot for tremendous work, @Qrox , screen reader and general accessibility, broadening this great game's audience by such means so that players from various communities can play it and enjoy it is very deeply appreciated by me, maybe i'll be able to work on this too someday, for now i'm just a JSON fixer :) |
@alex19EP @Kenan2000 glad to help! :) |
@Qrox are you able to record what you learned in the docs somewhere for future reference? |
if you want i can create issue describing an interface that needs cursor treatment. |
@jbytheway I've added documentation in #39609. @alex19EP No worries, just open an issue and I believe people will be willing to help. Cursor fixes seem pretty straightforward so we can probably gradually add it to every UI. |
@alex19EP if you really are the only blind player, then maybe accessibility fixes are exactly what we need to encourage more, so I also encourage you to open issues. |
ok @jbytheway . I received a carte blanche. ☺ so get ready. 😀 I will deal with this soon. |
@alex19EP thank you so much for providing feedback! there is no substitute for getting info from the folks who need a11y support the most :) I've come across least one other blind CDDA user on GitHub, @BlindGuyNW from way back on #15864. In my experience, that means there are many more lurkers out there! hope ya stick around 🚀 |
You rang? :) I haven't been poking at CDDA for a while, mostly because my Linux machine isn't a constant presence like my Windows and Macs. However, I'm always happy to see new accessibility fixes. I'll keep an eye on this at the very least |
Summary
SUMMARY: Bugfixes "Place cursor at highlighted item name in construction and crafting menus as expected by screen readers"
Purpose of change
May fix #39499.
Describe the solution
Place the cursor at the beginning of the highlighted item/construction name.
Testing
@alex19EP please test if this resolves #39499.