Skip to content

Fix numbered headings for Arabic#16425

Merged
seanbudd merged 1 commit into
betafrom
fixNumberedHeadingsAr
May 10, 2024
Merged

Fix numbered headings for Arabic#16425
seanbudd merged 1 commit into
betafrom
fixNumberedHeadingsAr

Conversation

@seanbudd

@seanbudd seanbudd commented Apr 18, 2024

Copy link
Copy Markdown
Member

Link to issue number:

Discussed here: https://groups.io/g/nvda-translations/message/3819

Summary of the issue:

In 2023.2, the user guide heading numbering for the Arabic translation switched to using Western Arabic numbering.
2024.1 mistakenly switched this back to Eastern Arabic numbering.

Description of user facing changes

Restore Western Arabic numbering style for headings.

Description of development approach

Updated CSS to reflect Western Arabic numbering style i.e. .1.2.3, .1.2.4

Testing strategy:

Compared:

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.

@seanbudd seanbudd added this to the 2024.2 milestone Apr 18, 2024
@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Apr 23, 2024
@seanbudd seanbudd marked this pull request as ready for review April 23, 2024 02:11
@seanbudd seanbudd requested a review from a team as a code owner April 23, 2024 02:11
@seanbudd seanbudd requested review from michaelDCurran and removed request for a team April 23, 2024 02:11
@seanbudd seanbudd merged commit de874ad into beta May 10, 2024
@seanbudd seanbudd deleted the fixNumberedHeadingsAr branch May 10, 2024 02:15
seanbudd added a commit that referenced this pull request May 20, 2024
Fixup of #16425
Discussed here: groups.io/g/nvda-translations/message/3819

Summary of the issue:
In 2023.2, the user guide heading numbering for the Arabic translation switched to using Western Arabic numbering.
2024.1 mistakenly switched this back to Eastern Arabic numbering.
In #16425, the headers in the table of contents were fixed for Western Arabic, but not in the body.

Description of user facing changes
Restore Western Arabic numbering style for headings in the body of the document

Description of development approach
Updated CSS to reflect Western Arabic numbering style i.e. .1.2.3, .1.2.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. queued for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants