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

Jump to bottom

Say all throws an exception in a control that does not support setting the caret #1201

Open
nvaccessAuto opened this issue Nov 16, 2010 · 7 comments
Open

Say all throws an exception in a control that does not support setting the caret #1201

nvaccessAuto opened this issue Nov 16, 2010 · 7 comments
Labels
bug p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority

Comments

@nvaccessAuto
Copy link

Reported by jteh on 2010-11-16 06:31
Say all does nothing in a control which does not support retrieving the caret, as expected. However, in a control which supports retrieval but not setting of the caret, an unhandled exception occurs. This is harmless but unclean.

Str:

  1. Open a command prompt.
  2. Press NVDA+downArrow.
  3. Expected: No response.
  4. Actual: Unhandled exception.

We can catch this and then either abort or ignore it. Ignoring might be confusing, as say all would appear to work but wouldn't move the caret. However, maybe this is desirable in some cases?
@ehollig
Copy link
Collaborator

ehollig commented Jul 15, 2017

@jcsteh is this still the case? I did not see anything alarming in the log when I performed the steps that you provided.

@derekriemer
Copy link
Collaborator

Can still reproduce.

  1. open command prompt in win7.
  2. press say all.

IO - speech.speak (18:36:19):
Speaking [IndexCommand(1), IndexCommand(2), IndexCommand(3), IndexCommand(4), IndexCommand(5), IndexCommand(6), IndexCommand(7), IndexCommand(8), IndexCommand(9), IndexCommand(10), IndexCommand(11)]
ERROR - queueHandler.pumpAll (18:36:19):
error in generator 497
Traceback (most recent call last):
File "queueHandler.pyo", line 72, in pumpAll
File "sayAllHandler.pyo", line 151, in readTextHelper_generator
File "textInfos\offsets.pyo", line 493, in updateCaret
File "textInfos\offsets.pyo", line 178, in _setCaretOffset
NotImplementedError
IO - inputCore.InputManager.executeGesture (18:36:21):
Input: kb(laptop):NVDA+f1

@jcsteh jcsteh added the p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority label Jul 17, 2017
@jcsteh
Copy link
Contributor

jcsteh commented Jul 17, 2017

P3 because this is ugly and perhaps confusing, but it doesn't break any functionality that a user would expect to work.

@Adriani90
Copy link
Collaborator

This is still reproducible in Windows 10 1809 with NVDA 2019.1.1.

@Adriani90
Copy link
Collaborator

I cannot reproduce this with NVDA last alpha on Windows 11. @lukaszgo1 or @CyrilleB79 could you test if this exception still occurs under Windows 10? If not, then I vote for closing this issue since support for Windows 7 has ended.

@CyrilleB79
Copy link
Collaborator

Still reproducible on Windows 10 2004 (version 10.0.19041 Build 19041), which is out-of-date however. I will test on a more recent version later or next week.

@CyrilleB79
Copy link
Collaborator

And also reproducible in the console in Windows 10 22H2 (10.0.19045 Build 19045).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug p4 https://github.com/nvaccess/nvda/blob/master/projectDocs/issues/triage.md#priority
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@jcsteh @derekriemer @nvaccessAuto @CyrilleB79 @ehollig @Adriani90