Skip to content

Update Liblouis to 3.27.0#15435

Merged
seanbudd merged 3 commits into
nvaccess:masterfrom
codeofdusk:liblouis-3.27
Oct 3, 2023
Merged

Update Liblouis to 3.27.0#15435
seanbudd merged 3 commits into
nvaccess:masterfrom
codeofdusk:liblouis-3.27

Conversation

@codeofdusk

Copy link
Copy Markdown
Contributor

Link to issue number:

None

Summary of the issue:

Liblouis 3.27.0 has been released.

Description of how this pull request fixes the issue:

Update Liblouis to 3.27.0, including new Braille tables.

Testing strategy:

Verified that Braille functions without errors and that new tables are selectable.

Known issues with pull request:

None known

Change log entries:

== Changes ==

  • Updated LibLouis braille translator to 3.27.0.
    • Added new Thai and Romanian Braille tables.

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • Security precautions taken.

@codeofdusk codeofdusk requested a review from a team as a code owner September 12, 2023 19:57
@codeofdusk codeofdusk requested review from seanbudd and removed request for a team September 12, 2023 19:57
@seanbudd seanbudd changed the base branch from beta to master September 13, 2023 05:58
Comment thread source/brailleTables.py
addTable("ro-g0.utb", _("Romanian 6 dot"))
# Translators: The name of a braille table displayed in the
# braille settings dialog.
addTable("ro.ctb", _("Romanian"))

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.

Should this one be renamed in the GUI?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm not sure.

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.

Having a 6 dot and a nothing is a bit confusing.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hmmm, how about Romanian (legacy)?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perhaps it's worth asking our romanian translators: https://github.com/nvaccess/mrconfig/blob/master/scripts/addressData.py#L358

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Have we heard anything back yet?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm assuming we haven't heard anything back yet. In absence of a response, lets merge and fix the strings later if required?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No response yet, sounds reasonable to me

@seanbudd seanbudd added this to the 2024.1 milestone Sep 14, 2023
@seanbudd seanbudd added the blocked/needs-product-decision A product decision needs to be made. Decisions about NVDA UX or supported use-cases. label Sep 25, 2023
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Oct 2, 2023
@seanbudd seanbudd merged commit 0760482 into nvaccess:master Oct 3, 2023
seanbudd pushed a commit that referenced this pull request Oct 12, 2023
Link to issue number:
Fix-up of #15435

Description of how this pull request fixes the issue:
Add @codeofdusk attribution for 2024.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked/needs-product-decision A product decision needs to be made. Decisions about NVDA UX or supported use-cases. conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants