Skip to content

Rework magnifier edge messages to ease translation#20123

Merged
seanbudd merged 2 commits into
nvaccess:betafrom
CyrilleB79:magEdgeMessages
May 15, 2026
Merged

Rework magnifier edge messages to ease translation#20123
seanbudd merged 2 commits into
nvaccess:betafrom
CyrilleB79:magEdgeMessages

Conversation

@CyrilleB79
Copy link
Copy Markdown
Contributor

Link to issue number:

Reported in this thread of the translators mailing list.

Summary of the issue:

Magnifier had 2 strings "left" for 2 different uses:

  • one used as adverb when panning left command was used while already at left edge of the screen to report no move
  • one other used in compound string "{edge} edge" as an adjective when panning to left edge command was used while already at left of the screen

In some languages, both strings do not translate the same.

Same for "right", "top" and "bottom" strings.

Description of user facing changes:

Panning command feedback when at edge can now be correctly translated.

Description of developer facing changes:

N/A

Description of development approach:

  • Used 1 string per action with no duplicate
  • Do not use a compound string since it is no longer needed; this coumpound string was useful only when paning commands were always reporting something but this behaviour has been withdrawn. Moreover, compound strings may be less easy to translate for translators.

Testing strategy:

  • Manual tests of all 8 paning commands at 8 edges.

Known issues with pull request:

None

Code Review Checklist:

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

@CyrilleB79
Copy link
Copy Markdown
Contributor Author

@Boumtchack you may want to check this PR.

@seanbudd seanbudd added this to the 2026.2 milestone May 13, 2026
@CyrilleB79 CyrilleB79 marked this pull request as ready for review May 13, 2026 05:47
@CyrilleB79 CyrilleB79 requested a review from a team as a code owner May 13, 2026 05:47
@CyrilleB79 CyrilleB79 requested a review from SaschaCowley May 13, 2026 05:47
@CyrilleB79
Copy link
Copy Markdown
Contributor Author

Failing tests are unrelated.

Comment thread source/_magnifier/commands.py Outdated
@seanbudd seanbudd merged commit e81c456 into nvaccess:beta May 15, 2026
30 of 37 checks passed
@CyrilleB79 CyrilleB79 deleted the magEdgeMessages branch May 16, 2026 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants