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

Impossible to set caret to the position of the review cursor in rich edit controls when handle keys from other applications is enabled. #13968

Closed
lukaszgo1 opened this issue Jul 30, 2022 · 8 comments
Labels
Abandoned requested reports or updates are missing since more than 1 year, author or users are not available. p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.

Comments

@lukaszgo1
Copy link
Contributor

Initially described in issue #13420

Steps to reproduce:

  1. Open NVDA's log viewer or Wordpad,
  2. Set your caret somewhere and move review cursor to the top of the edit field.
  3. Try to move caret to the position of the review cursor by pressing NVDA+Shift+numpad minus twice.
  4. In keyboard settings disable "Handle keys from other applications" and repeat steps 2 to 3.

Actual behavior:

When Handle keys from other applications is enabled caret fails to move.

Expected behavior:

Caret should be moved to the review position regardless of the "Handle keys from other applications" being enabled or disabled.

NVDA logs, crash dumps and other attachments:

Log snipped with "Handle keys from other applications" enabled:

IO - inputCore.InputManager.executeGesture (15:02:16.639) - winInputHook (3116):
Input: kb(desktop):shift+NVDA+numpadMinus
IO - speech.speech.speak (15:02:16.649) - MainThread (7152):
Speaking ['Move focus']
IO - inputCore.InputManager.executeGesture (15:02:16.649) - winInputHook (3116):
Input: kb(desktop):NVDA+
IO - inputCore.InputManager.executeGesture (15:02:16.759) - winInputHook (3116):
Input: kb(desktop):shift+NVDA+numpadMinus
IO - speech.speech.speak (15:02:16.769) - MainThread (7152):
Speaking ['Move focus']
IO - inputCore.InputManager.executeGesture (15:02:16.769) - winInputHook (3116):
Input: kb(desktop):NVDA+

Notice the additional lines Input: kb(desktop):NVDA+ which should not be there.

Log snipped with "Handle keys from other applications" disabled:

IO - inputCore.InputManager.executeGesture (15:03:29.438) - winInputHook (3116):
Input: kb(desktop):shift+NVDA+numpadMinus
IO - speech.speech.speak (15:03:29.448) - MainThread (7152):
Speaking ['Move focus']
IO - inputCore.InputManager.executeGesture (15:03:29.588) - winInputHook (3116):
Input: kb(desktop):shift+NVDA+numpadMinus
IO - speech.speech.speak (15:03:29.648) - MainThread (7152):
Speaking ['Developer Scratchpad mode enabled\r']

System configuration

NVDA installed/portable/running from source:

Both installed and portable.

NVDA version:

Latest Alpha, 2022.2

Windows version:

Tested on Windows 7 but should not matter.

Name and version of other software in use when reproducing the issue:

Wordpad, NVDA's log viewer, QRead

Other information about your system:

None relevant

Other questions

Does the issue still occur after restarting your computer?

Yes

Have you tried any other versions of NVDA? If so, please report their behaviors.

The same issue occurs with 2019.2

If NVDA add-ons are disabled, is your problem still occurring?

Yes

Does the issue still occur after you run the COM Registration Fixing Tool in NVDA's tools menu?

Haven't tried.

@lukaszgo1
Copy link
Contributor Author

@cary-rowen Would you be able to confirm that disabling "Handle keys from other applications" makes it possible to move caret to the position of the review cursor in bookworm?

@cary-rowen
Copy link
Contributor

yes i can confirm this.

@cary-rowen
Copy link
Contributor

Also, as you said in #13420, Say all needs to be improved.

@seanbudd seanbudd added p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation. labels Aug 4, 2022
@LeonarddeR
Copy link
Collaborator

@lukaszgo1 Can you still reproduce this with Wordpad in current alpha versions? Note that in NVDA itself, we forcefully disable UIA.

@cary-rowen
Copy link
Contributor

Hi @LeonarddeR

"Handle keys from other applications" is checked.

Excitingly, I can no longer reproduce this in WordPad and Bookworm using NVDA-alpha-28931,186a8d70. But still reproducible in NVDA's log viewer.

Thanks

@LeonarddeR
Copy link
Collaborator

I'm afraid we can not use UIA in NVDA's log viewer. That said, we could more safely consider disabling ITextDocument2 by default for the log viewer.

@cary-rowen
Copy link
Contributor

Can we assume that #15314 has fixed (or partially fixed) this issue?

@Adriani90 Adriani90 added the Abandoned requested reports or updates are missing since more than 1 year, author or users are not available. label Nov 20, 2024
@Adriani90
Copy link
Collaborator

We didn't hear an update from the initial user since more than 2 years upon request, and I cannot reproduce it either. Maybe it was fixed as referencced above. Closing as abandoned. if anyone can still reproduce this, please comment and we can reopen.

@Adriani90 Adriani90 closed this as not planned Won't fix, can't repro, duplicate, stale Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Abandoned requested reports or updates are missing since more than 1 year, author or users are not available. p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority triaged Has been triaged, issue is waiting for implementation.
Projects
None yet
Development

No branches or pull requests

5 participants