-
-
Notifications
You must be signed in to change notification settings - Fork 634
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
Report more object properties on mouse enter #15518
Conversation
I am a bit concerned by the fact that there seem to be few mouse users active on this GitHub repo. Could we Cc some known people? I also wonder if a person using the mouse may need so much information than someone who cannot see anything and uses only the keyboard. I understand the need to know the cell's coordinates; but I imagine that a link can be seen more easily by some people. Note: I am not advocating against this PR. I just would like to double check if it's relevant and for which use case. |
CC @britechguy
|
cc @hwf1324 |
Does this fix, by any chance, possibly resolve what I was asking about in issue #15397 ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reads well, good work
Actually, I've asked about the mouse tracking being able to report object states many years ago. |
My thoughts on this. Reading out attributes such as clickable, links, etc. while in browse mode is sometimes a redundancy for me. So I would switch it off in the document format. This is the same for mouse navigation. But after my testing, it doesn't follow the document formatting settings. Not sure this can be done without making the behavior of mouse navigation and browsing modes consistent. Of course normally I don't check this checkbox. |
See test results for failed build of commit 0d1925b773 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @LeonarddeR
Not sure if this started after this PR, but NVDA reports Text every time when entering a plain text with the mouse. Can this be turned off somehow? Role Text is actually not really useful for the user to hear every time in the browser. |
Pretty sure this was already the case before. You're best to create a new issue for this. |
See #15397 This seems to be caused by the accessibility interface used by Chromium?Firefox does not have this problem. |
@LeonarddeR is there anything that argues against having this enabled by default? |
@Adriani90, do you mean having "Report object when mouse enters it" checked by default? I imagine that some partially sighted people using the mouse can see objects role and property (e.g. checked) and just want the text to be read. I cannot say how many want just the text and how many also want the role and the properties. But it's not obvious to me that the default value of this setting should be changed. |
Link to issue number:
Fixes #15420
Summary of the issue:
When the mouse option "Report role when mouse enters object" is enabled, NVDA reports the role of objects when the mouse enters them. However, the role is in pretty insufficient in many situations, for example:
Description of user facing changes
Description of development approach
Implemented some additional logic to speak objects with reason controlTypes.OutputReason.MOUSE, and made event_mouseMove use speech.speakObject.
Testing strategy:
Known issues with pull request:
I didn't touch the config spec as I think it is a bit useless to do so, just for renaming a setting. Feel free to shout out if you think otherwise.
Code Review Checklist: