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

Sync Braille display driver #1267

Closed
nvaccessAuto opened this issue Dec 8, 2010 · 9 comments
Closed

Sync Braille display driver #1267

nvaccessAuto opened this issue Dec 8, 2010 · 9 comments

Comments

@nvaccessAuto
Copy link

Reported by Casalino on 2010-12-08 13:50
Here this drivwer to use Sync Braille from Hims with NVDA.

@nvaccessAuto
Copy link
Author

Attachment SyncBrailleNvdaDriver_v0.5.zip added by Casalino on 2010-12-08 13:50
Description:

@nvaccessAuto
Copy link
Author

Comment 1 by jteh on 2011-11-07 06:42
(This is pretty much identical to my comment on #1266.)

It seems this ticket was overlooked. Sorry.

I've only looked at the code briefly so far. A few points:

  • Where did you obtain !SyncBraille.dll? Are you certain it is legal for us to distribute this with NVDA?
  • Are you willing to update this driver soon and maintain it into the future?
  • In order to get this included into NVDA now, it needs to be updated to use the new input gesture framework. That is, all keys must be recognised and converted to input gestures. This way, a user can provide their own key bindings if desired.

@nvaccessAuto
Copy link
Author

Comment 2 by drein on 2011-11-16 09:51
Gianluca told me that the drivers are quite ready, but unfortunately we have'nt a SyncBraille to test them, so we don't know if they work or not...

@nvaccessAuto
Copy link
Author

Comment 3 by drein on 2012-06-05 14:24
Here is a stable SyncBraille Driver.
We put the source code, the dll that should be in program files\NVDA\brailleDisplayDrivers with the other dll, and some lines of documentation in t2t format.
Since each family of Display Braille starts with the "++" symbol in the user Guide, I think that the syncbraille should have a deeper chapter because there is also the other driver that covers all other display braille.
Anyway, check it.
I tested the code with a SyncBraille and all worked well.

@nvaccessAuto
Copy link
Author

Attachment syncbraille.zip added by drein on 2012-06-05 14:26
Description:
Stable driver

@nvaccessAuto
Copy link
Author

Comment 4 by jteh on 2012-06-07 02:55
Changes:
Milestone changed from None to 2012.3

@nvaccessAuto
Copy link
Author

Comment 5 by jteh on 2012-06-07 08:19
Merged in ea9cd6f including several changes by me. If you're interested in the changes, please take a look at the merged revisions. Thanks for your great work.

I'm closing as fixed, but please test to make sure none of my changes broke anything. Please reopen if I broke something.
Changes:
State: closed

@nvaccessAuto
Copy link
Author

Comment 6 by ateu on 2012-06-07 11:59
Hi

In Braille Settings Dialog, SincBraille was placed after No braille.
I think it is not logical.

@nvaccessAuto
Copy link
Author

Comment 7 by jteh on 2012-06-07 12:29
It's alphabetical order of driver name; syncBraille sorts after noBraille. If it bothers you a lot, please file a separate ticket, as this isn't specific to syncBraille. Thanks.

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

No branches or pull requests

2 participants