Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#1267 closed enhancement (fixed)

Sync Braille display driver

Reported by: Casalino Owned by: jteh
Priority: minor Milestone: 2012.3
Component: Braille Version: master
Keywords: Cc:
Operating system: Blocked by:
Blocking:
Changes document entry (for developers):

Description

Here this drivwer to use Sync Braille from Hims with NVDA.

Attachments (2)

SyncBrailleNvdaDriver_v0.5.zip (8.5 KB) - added by Casalino 4 years ago.
syncbraille.zip (9.4 KB) - added by drein 3 years ago.
Stable driver

Download all attachments as: .zip

Change History (9)

Changed 4 years ago by Casalino

comment:1 Changed 3 years ago by jteh

(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.

comment:2 Changed 3 years ago by drein

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...

comment:3 Changed 3 years ago by drein

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.

Changed 3 years ago by drein

Stable driver

comment:4 Changed 3 years ago by jteh

  • Milestone set to 2012.3
  • Status changed from new to accepted
  • Type changed from task to enhancement

comment:5 Changed 3 years ago by jteh

  • Resolution set to fixed
  • Status changed from accepted to closed

Merged in changeset:ea9cd6fdb82adda3df707fd59828a7b77bbb5dbb 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.

comment:6 Changed 3 years ago by ateu

Hi

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

comment:7 Changed 3 years ago by jteh

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.

Note: See TracTickets for help on using tickets.