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

add keystroke to jump to the end of landmarks #5482

Closed
sukiletxe opened this Issue Nov 11, 2015 · 2 comments

Comments

Projects
None yet
4 participants
@sukiletxe

sukiletxe commented Nov 11, 2015

Rationale: Sometimes, mostly in news sites, an aside landamark is found inside a main landmark. That makes the reading experience rather unpleasant. Adding a keyboard command to jump to the end of a landmark would improve the reading experience. Maybe it will make the announcement of the ending would make nVDA too chatty thoguh.

@jcsteh jcsteh added the Browse mode label Nov 11, 2015

@jcsteh

This comment has been minimized.

Show comment
Hide comment
@jcsteh

jcsteh Nov 11, 2015

Contributor

Rather than consuming an extra keystroke, it'd be good if we could have landmarks included in the move past end of container command (comma). This could be a bit tricky, though.

Contributor

jcsteh commented Nov 11, 2015

Rather than consuming an extra keystroke, it'd be good if we could have landmarks included in the move past end of container command (comma). This could be a bit tricky, though.

@derekriemer

This comment has been minimized.

Show comment
Hide comment
@derekriemer

derekriemer Oct 16, 2016

Collaborator

If I want to give this a try, are there any things I should know?

Collaborator

derekriemer commented Oct 16, 2016

If I want to give this a try, are there any things I should know?

derekriemer added a commit to derekriemer/nvda that referenced this issue Oct 17, 2016

In browse mode, it is now possible to skip past landmarks if there is…
… one using the skip to end/beginning of container command. If there is a container inside of a landmark, this command retains the behavior of only skipping to the end of the container. nvaccess#5482

This needs testing in edge, and look over from someone who knows more about browse mode. Also, I'm not sure why there are two methods, one in virtualBuffers/__init__.py and browseMode.py, so if someone can check my code for sanity, that would be great.

derekriemer added a commit to derekriemer/nvda that referenced this issue Oct 17, 2016

In browse mode, it is now possible to skip past landmarks if there is…
… one using the skip to end/beginning of container command. If there is a container inside of a landmark, this command retains the behavior of only skipping to the end of the container. nvaccess#5482

This needs testing in edge, and look over from someone who knows more about browse mode. Also, I'm not sure why there are two methods, one in virtualBuffers/__init__.py and browseMode.py, so if someone can check my code for sanity, that would be great.

jcsteh added a commit that referenced this issue Aug 2, 2017

In browse mode, it is now possible to skip past landmarks if there is…
… one using the skip to end/beginning of container command. Incubates #6474 (issue #5482).

@jcsteh jcsteh closed this in #6474 Aug 24, 2017

@nvaccessAuto nvaccessAuto added this to the 2017.3 milestone Aug 24, 2017

jcsteh added a commit that referenced this issue Aug 24, 2017

In browse mode, it is now possible to skip past/to the start of landm…
…arks using the skip to end/beginning of container commands. (issue #5482, PR #6474)

@jcsteh jcsteh modified the milestones: 2017.4, 2017.3 Aug 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment