Update liblouis to 3.18.0#12526
Conversation
|
There are several grade 2 tables that are not declared as contracted. I think these are mistakes. Here are the corresponding lines ( Should I make another Pull Request for that? CC @LeonarddeR |
|
Hello! Can it be included in NVDA 2021.1? Or Or is it too late? |
|
I'm afraid it's too late for that. |
|
Thanks again, this looks like a great update! I noticed that the "PR description is up to date is unticked. Can you say why that is? Is it because we are uncertain of the following known issue:
@LeonarddeR @AAClause - should we proceed with this given the raised issue?
Sorry, this will have to wait until 2021.2. |
That's my question :). Only 4 SLOC are affected but given that it is not related to liblouis 3.18 it's likely irrelevant in this PR (?). Also I'd like to suggest to change the display names of some braille tables in the GUI, like this:
This way all Kurdish and Vietnamese tables would be gathered (currently scattered due to alphanumeric sorting). What do you think? |
Cool, I will wait to see what @LeonarddeR thinks as well.
I agree, that would also be a good change. Might be worth noting that it differs to liblouis in a comment if that is the case. |
|
I think it would be best to merge this pr as is. @AAClause raises valid points and therefore it makes sense to address them, but preferably in a follow up. |
Link to issue number:
None
Summary of the issue:
Liblouis 3.18.0 has just been released.
Description of how this pull request fixes the issue:
Updates liblouis to 3.18.0 which adds support for many new languages and other awesome things.
Testing strategy:
Known issues with pull request:
None
Change log entries:
Section: Changes
Code Review Checklist: