Skip to content
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

Merged
merged 1 commit into from
Apr 15, 2020

Conversation

Qrox
Copy link
Contributor

@Qrox Qrox commented Apr 15, 2020

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.

@alex19EP
Copy link
Contributor

construction menu: works like a charm.
crafting menu: not working. i don't see the cursor on selected item.

@alex19EP
Copy link
Contributor

weird. the cursor is always on the second item from the bottom in the visible part of the list.

@Qrox Qrox force-pushed the ui-place-cursor branch from b3fb930 to a6bfeda Compare April 15, 2020 14:30
@Qrox
Copy link
Contributor Author

Qrox commented Apr 15, 2020

Perhaps crafting menu didn't work because the window was not refreshed. Please test the new patch and see if it works.

@alex19EP
Copy link
Contributor

yay! now all is working. @Qrox great job.
thank you.

@Kenan2000
Copy link
Contributor

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 :)

@Qrox
Copy link
Contributor Author

Qrox commented Apr 15, 2020

@alex19EP @Kenan2000 glad to help! :)

@ZhilkinSerg ZhilkinSerg added [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. Accessibility Issues regarding accessibility labels Apr 15, 2020
@ZhilkinSerg ZhilkinSerg merged commit 5db5df0 into CleverRaven:master Apr 15, 2020
@jbytheway
Copy link
Contributor

@Qrox are you able to record what you learned in the docs somewhere for future reference?

@alex19EP
Copy link
Contributor

@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.
it really helps in accessibility.
post scriptum.
So I think I'm the only blind player at the moment. I'm just a little embarrassed to ask for accessability fixes. 😳

@Qrox
Copy link
Contributor Author

Qrox commented Apr 16, 2020

@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.

@jbytheway
Copy link
Contributor

@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.

@alex19EP
Copy link
Contributor

ok @jbytheway . I received a carte blanche. ☺ so get ready. 😀 I will deal with this soon.

@Leland
Copy link
Contributor

Leland commented Apr 17, 2020

@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 🚀

@Qrox Qrox deleted the ui-place-cursor branch April 17, 2020 06:54
@alex19EP alex19EP mentioned this pull request Apr 17, 2020
13 tasks
@BlindGuyNW
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Issues regarding accessibility [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ui_adaptor: accessibility regression
7 participants