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

Review always jumps to focus when returning to browse mode document #4369

Closed
nvaccessAuto opened this issue Aug 8, 2014 · 5 comments
Closed

Review always jumps to focus when returning to browse mode document #4369

nvaccessAuto opened this issue Aug 8, 2014 · 5 comments
Assignees
Milestone

Comments

@nvaccessAuto
Copy link

@nvaccessAuto nvaccessAuto commented Aug 8, 2014

Reported by jteh on 2014-08-08 05:49
Str:

  1. Open http://www.nvaccess.org/
  2. Press control+home. Ensure that the NV Access link is focused.
  3. Press h to move to the first heading.
  4. Ensure you're in document review.
  5. Alt+tab out and back in.
  6. Press NVDA+upArrow.
  • Result (correct): The heading.
    1. Press numpad8.
  • Expected: The heading.
  • Actual: The link.
@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Aug 8, 2014

Comment 1 by jteh on 2014-08-08 05:49
Changes:
Milestone changed from None to next

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Aug 12, 2014

Comment 2 by Michael Curran <mick@... on 2014-08-12 04:59
In [4d522c8]:

Ensure that the  review cursor is set to the caret's position when focus moves inside of a treeInterceptor rather than moving to the focused object. Re #4369.

Also ensure that the review mode is correctly set to document if the navigator object ever goes inside a working TreeIntercetor, whether or not the navigator object moved due to focus.

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Aug 12, 2014

Comment 3 by Michael Curran <mick@... on 2014-08-12 05:00
In [ea8aa94]:

Merge branch 't4369' into next. Incubates #4369

Changes:
Added labels: incubating

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Sep 4, 2014

Comment 4 by Michael Curran <mick@... on 2014-09-04 04:25
In [ece3bf2]:

Merge branch 't4369'. Fixes #4369

Changes:
Removed labels: incubating
State: closed

@nvaccessAuto
Copy link
Author

@nvaccessAuto nvaccessAuto commented Sep 4, 2014

Comment 5 by mdcurran on 2014-09-04 04:27
Changes:
Milestone changed from next to 2014.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.