Skip to content

/ nvda

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

Jump to bottom

mouse shape changes not announced in windows 10 #5595

Closed
JamaicanUser opened this issue Dec 7, 2015 · 2 comments
Closed

mouse shape changes not announced in windows 10 #5595

JamaicanUser opened this issue Dec 7, 2015 · 2 comments
Assignees
@jcsteh
Labels
bug bug/regression
Milestone
2016.1

Comments

@JamaicanUser
Copy link

@JamaicanUser JamaicanUser commented Dec 7, 2015

Using Windows 10, 64 bit, The report mouse shape changes and enable mouse tracking options checked. NVDA is not announcing the shape change as the mouse pointer is moved. I don't know if this applies to any other variant of Windows, though.
@jcsteh jcsteh added this to the 2016.1 milestone Dec 8, 2015
@jcsteh
Copy link
Contributor

@jcsteh jcsteh commented Dec 8, 2015

Confirmed.

It looks like this broke in two phases:

  1. #3801 (NVDA 2014.2): mouseHandler only reports the shape after it is pumped 10 times to avoid flooding. However, we no longer pump constantly, only on demand. This needs to be changed to a timer instead.
  2. #5302 (NVDA 2015.4): We seem to be missing most cursor shape changes. I'm pretty sure this is because these get blocked by eventHandler.shouldAcceptEvent. The fix is probably to allow events from the desktop window.
@jcsteh jcsteh self-assigned this Dec 9, 2015
jcsteh added a commit that referenced this issue Dec 9, 2015
@jcsteh
Reporting of mouse shape changes again works as expected.
bc7643a 
* Cursor events end up firing on the desktop window, so shouldAcceptEvent had to be taught this.
* The previous code relied on counting core ticks, but now that core ticks are on-demand, this needed to be updated to use a timer.

Fixes #5595.
@nvaccessAuto
Copy link

@nvaccessAuto nvaccessAuto commented Dec 9, 2015

Incubated in ae3b586.
@jcsteh jcsteh closed this in 8a15cd1 Jan 6, 2016
@nvaccessAuto nvaccessAuto removed the incubating label Jan 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcsteh jcsteh

Labels
bug bug/regression
Projects
None yet
Milestone
2016.1
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@jcsteh @nvaccessAuto @JamaicanUser