Skip to content

Add more default task switching gestures to freedom scientific braille displays#7387

Merged
michaelDCurran merged 2 commits into
nvaccess:masterfrom
Oliver2213:fs-brl-taskswitching-gestures
Jun 13, 2018
Merged

Add more default task switching gestures to freedom scientific braille displays#7387
michaelDCurran merged 2 commits into
nvaccess:masterfrom
Oliver2213:fs-brl-taskswitching-gestures

Conversation

@Oliver2213

@Oliver2213 Oliver2213 commented Jul 12, 2017

Copy link
Copy Markdown
Contributor

Summary: The Freedom Scientific braille display driver only provides a gesture for alt+tab by default.

Fixes: this pr adds gestures for alt shift tab (dots 1-2-5-6 with space), and windows tab (dots 2-3-4 with space).

Suggested what's new entry (section 'changes'): Add default gestures for alt shift tab and windows tab with all supported Freedom Scientific braille displays.

@LeonarddeR

Copy link
Copy Markdown
Collaborator

cc @bramd

@bramd

bramd commented Oct 21, 2017

Copy link
Copy Markdown
Contributor

@Oliver2213 I don't see a problem in adding these, however I've two questions:

  1. How did you come up with the key assignments? Are these based on what's already implemented in other screenreaders or braille displays?
  2. Could you add this keys to the user guide as well?

@Oliver2213

Copy link
Copy Markdown
Contributor Author

I didn't think of adding them to the users guide, I'll certainly do that soon. As for how I came up with them... The command for alt+tab is space with dots 2-3-4-5, which is "t" in Braille; dots 1-2-5-6 are the reverse of that, which makes sense for alt+tab. The command for windows+tab is the "s" in Braille (which stands for switching, as in task switching); it's also nearly the same as the default alt_tab command, just take out dot 5.
I haven't checked what the equivalent commands are for other screen readers, or even if they have such. I can do so if you think it's important.

@bramd

bramd commented Oct 25, 2017 via email

Copy link
Copy Markdown
Contributor

@LeonarddeR LeonarddeR left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Feel free to update the copyright line

LeonarddeR pushed a commit that referenced this pull request Apr 30, 2018
@derekriemer

Copy link
Copy Markdown
Collaborator

Congrats on your first incubating PR!!

@michaelDCurran michaelDCurran merged commit 2d1a4fc into nvaccess:master Jun 13, 2018
@nvaccessAuto nvaccessAuto added this to the 2018.3 milestone Jun 13, 2018

@LeonarddeR LeonarddeR left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks to @derekriemer for reporting this privately.

"kb:alt" : ("br(freedomScientific):dot1+dot3+dot4+brailleSpaceBar",),
"kb:alt+tab" : ("br(freedomScientific):dot2+dot3+dot4+dot5+brailleSpaceBar",),
"kb:alt+shift+tab" : ("br(freedomScientific):dot1+dot2+dot5+dot6+brailleSpaceBar",),
"kb:win+tab" : ("br(freedomScientific):dot2+dot3+dot4+brailleSpaceBar",),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one is wrong and breaks the driver currently. it should be windows, not win.

@Oliver2213: Could you provide a pr that fixes this?

@Oliver2213

Copy link
Copy Markdown
Contributor Author

Sure, thanks for letting me know.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants