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

Script to move braille to the focus and cursor #5250

Closed
nvaccessAuto opened this issue Jul 28, 2015 · 4 comments
Closed

Script to move braille to the focus and cursor #5250

nvaccessAuto opened this issue Jul 28, 2015 · 4 comments
Assignees
@jcsteh
Labels
blocked/needs-code-review component/braille enhancement
Milestone
2016.3

Comments

@nvaccessAuto
Copy link

Reported by dkager on 2015-07-28 15:27
Add a script that routes braille to the focus or cursor position. If braille is tethered to review, call script_navigatorObject_toFocus. If braille is tethered to focus, scroll the braille display so the cursor/caret is shown. In dialog boxes, should probably also scroll the display to the initial position it's in when first encountering a control. That is, if a control plus meta information is too long to fit on the reading unit and the user has scrolled the display, put it back to the original position.

Used for the Alva BC6 bindings as per #5206. May also apply to #5243.
Blocking #5206
@nvaccessAuto
Copy link
Author

Comment 2 by dkager on 2015-08-05 10:55
I had a try at this in t5250. The name of this script is a little difficult, toFocus may be too generic whereas toCursor may be too specific. The same goes for the help text. A final observation is that this script may not play nicely with messages being displayed. The mainBuffer is only updated if it's active, which seems good enough, but I'm no expert. :)

@nvaccessAuto
Copy link
Author

Comment 4 by jteh on 2015-10-25 22:50
Changes:
Milestone changed from None to 2016.1

@nvaccessAuto
Copy link
Author

Comment 5 by dkager on 2015-10-26 19:45
One final note, if #5410 is implemented then the script added by this ticket should likewise scroll to the selection if there is no cursor. It all comes together...

@nvaccessAuto nvaccessAuto added this to the 2016.1 milestone Nov 10, 2015
@dkager dkager mentioned this issue Nov 14, 2015
Merged
dkager added a commit to dkager/nvda that referenced this issue Nov 14, 2015
@dkager
Alva BC6: Use the new braille_toFocus script. Requires nvaccess#5250.
65716eb
@jcsteh jcsteh modified the milestones: 2016.2, 2016.1 Jan 18, 2016
@nvaccessAuto nvaccessAuto mentioned this issue Nov 13, 2015
Closed
@jcsteh jcsteh modified the milestones: 2016.2, 2016.3 May 6, 2016
michaelDCurran pushed a commit that referenced this issue Jul 24, 2016
@dkager @michaelDCurran
Improve support for the Alva BC6 braille displays (#5493)
fb76661 
* Alva BC6: Add more key bindings.

* Alva BC6: Add detection of secondary cursor routing buttons, currently unassigned.
These are virtual buttons that are triggered by a long-press of the cursor routing buttons.
For backwards compatibility the original name, routing, is still used for the primary action.

* Alva BC6: Update user documentation.

* Alva BC6: Add two more bindings for controlling the cursor shape and state. Requires #5198.

* Alva BC6: Use the new braille_toFocus script. Requires #5250.

* Improve command names for the BC6 in the user guide.
@jcsteh
Copy link
Contributor

jcsteh commented Aug 5, 2016

Merged in 4c671a6.

@jcsteh jcsteh closed this as completed Aug 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jcsteh jcsteh

Labels
blocked/needs-code-review component/braille enhancement
Projects
None yet
Milestone
2016.3
Development

No branches or pull requests
2 participants
@jcsteh @nvaccessAuto