Update liblouis to 3.16.1#11888
Conversation
|
@LeonarddeR
Please give a review to this. I remember, when Russian eight dots table was added in the past, that the mentioned removed entry was needed, due to the upgrade from the older NVDA releases.
Sorry for being so brutal, but I think that in the future something needs to be done, so that NVDA can automatically discover liblouis tables. Now we have a problem in which we need to do this manually for every table, which is not so good.
|
|
Ugh, I"m sorry for going so wild on my keyboard |
I think you're not brutal at all but bringing up a fair point. There's an issue for that, see #11298 |
LeonarddeR
left a comment
There was a problem hiding this comment.
- Why is input disabled for the russian tables? I think it would be good to have a rationale in a comment.
- Given this release fixes two issues related to crashes and doesn't contain many additional changes to the core, I think we should consider shipping this with 2020.4. The beta period should be long enough to revert if it really breaks. cc @feerrenrut
I simply used provided metadata to add these tables: Any thought @Futyn-Maker?
👍 |
|
Hello contributers! I was waiting for this PR. The couple of words about Russian tables:
Andrey |
d221181 to
0b8e386
Compare
|
I also would like to know if it is possible to replace "grade 1" with "literary braille" in the UI, or it can be done separate for the Russian localization. The reason is that we don't use contracted braille almost at all, so names "computer braille" and "literary braille" can be more understandable for end users than "grade 1", because there's no any gradation in fact. But it's not super important. @zstanecic do You have any comments about it? |
|
Thanks. As for me, looks good. |
|
Added milestone 2020.4 for @feerrenrut or @michaelDCurran to triage. |
|
My review comment
Please add a Coptic table in scope of this PR. The maintainer asked about this on the mailing list.
|
|
@zstanecic In my opinion, Leonard should have added this braille table in #11537. |
|
Changelog entry for this one should probably mention the newly added tables. |
|
I support merging this into 2020.4, so that the NVDA and Liblouis stable versions become once again in sync. |
|
If we want this in 2020.4, it will need to be retargeted to beta |
See test results for failed build of commit f5fde1895e |
e57c785 to
339763b
Compare
Link to issue number:
Fixes #11013
Fixes #11754
Summary of the issue:
Liblouis 3.16.0 has just been released.
EDIT: update to 3.16.1.
Description of how this pull request fixes the issue:
Testing performed:
Tested from source.
Known issues with pull request:
None
Change log entry: