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

Can't switch to screen review if object has no location #3454

Closed
nvaccessAuto opened this issue Aug 20, 2013 · 2 comments
Closed

Can't switch to screen review if object has no location #3454

nvaccessAuto opened this issue Aug 20, 2013 · 2 comments
Labels
bug
Milestone
2013.3

Comments

@nvaccessAuto
Copy link

Reported by jteh on 2013-08-20 12:13
(Spun off ticket:1913#comment:10.)

If you're in object review and your navigator object is on an object with no location (obj.location is None), switching to screen review throws this error:

ERROR - scriptHandler.executeScript (22:09:53):
error executing script: <bound method GlobalCommands.script_reviewMode_next of <globalCommands.GlobalCommands object at 0x05631BB0>> with gesture u'NVDA+numpad 7'
Traceback (most recent call last):
  File "scriptHandler.py", line 165, in executeScript
    script(gesture)
  File "globalCommands.py", line 283, in script_reviewMode_next
    label=review.nextMode()
  File "review.py", line 133, in nextMode
    return label or nextMode(prev=prev,startMode=newMode)
  File "review.py", line 132, in nextMode
    label=setCurrentMode(newMode)
  File "review.py", line 113, in setCurrentMode
    pos=func(obj)
  File "review.py", line 62, in getScreenPosition
    pos=DisplayModelTextInfo(s,obj)
  File "displayModel.py", line 199, in __init__
    super(DisplayModelTextInfo, self).__init__(obj, position)
  File "textInfos\offsets.py", line 267, in __init__
    start,end=self._getOffsetsFromNVDAObject(position)
  File "displayModel.py", line 375, in _getOffsetsFromNVDAObject
    raise RuntimeError
RuntimeError

This is not a regression; it occurs when switching to flat review with 2013.1 as well.
Blocking #3517
@nvaccessAuto
Copy link
Author

Comment 2 by Michael Curran <mick@... on 2013-11-15 01:08
In [269c7be]:

displayModel.DisplayModelTextInfo._getOffsetsFromNVDAObject: raise LookupError instead of RuntimeError when an object has no location. Allows switching to screen review when the navigator object has no screen location. Fixes #3454

Changes:
State: closed

@nvaccessAuto
Copy link
Author

Comment 3 by jteh on 2013-11-26 23:26
Changes:
Milestone changed from next to 2013.3

@nvaccessAuto nvaccessAuto added this to the 2013.3 milestone Nov 10, 2015
This was referenced Nov 10, 2015
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
2013.3
Development

No branches or pull requests
1 participant
@nvaccessAuto