Skip to content

Add plural forms for needed strings in NVDA's code base #15864

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
seanbudd merged 22 commits into from
Dec 8, 2023
Merged

Add plural forms for needed strings in NVDA's code base #15864

Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
34d8487
First work
CyrilleB79 Sep 10, 2023
95dc8f6
Merge branch 'master' into plural
CyrilleB79 Oct 9, 2023
f1fc977
Merge branch 'master' into plural
CyrilleB79 Nov 13, 2023
d6318de
fix syntax errors
CyrilleB79 Nov 13, 2023
8d7942d
Merge branch 'master' into plural
CyrilleB79 Nov 29, 2023
fca18e1
More files converted
CyrilleB79 Nov 29, 2023
bad1de6
Touch gestures
CyrilleB79 Nov 29, 2023
9b6ffe5
Change log
CyrilleB79 Nov 30, 2023
009744a
Add PR number in the log
CyrilleB79 Nov 30, 2023
d3e70e4
Fix check POT
CyrilleB79 Nov 30, 2023
795a6dd
Update source/globalCommands.py
CyrilleB79 Nov 30, 2023
dcea438
Linting
CyrilleB79 Nov 30, 2023
72f6475
Fix expressions for tables
CyrilleB79 Dec 1, 2023
4024f2f
Fix row/column count in virtual buffer
CyrilleB79 Dec 4, 2023
0f2fef6
Apply suggestions from code review
CyrilleB79 Dec 5, 2023
6a3f2c6
Remove unexpected file
CyrilleB79 Dec 5, 2023
9cc41e3
Simplify statement
CyrilleB79 Dec 5, 2023
5b4b35c
Fix spelling in moved comments
CyrilleB79 Dec 5, 2023
70d32e4
Sub-function to compute table size reporting
CyrilleB79 Dec 6, 2023
3ec7c14
Revert "Sub-function to compute table size reporting"
CyrilleB79 Dec 7, 2023
93abb27
Revert "Revert "Sub-function to compute table size reporting""
CyrilleB79 Dec 7, 2023
6db8c7d
Factor out common translations to report rows and columns
CyrilleB79 Dec 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Factor out common translations to report rows and columns
  • Loading branch information
@CyrilleB79
CyrilleB79 committed Dec 7, 2023
commit 6db8c7d90d3f0f1ebc14929c2e13be4cea1c08b0
58 changes: 28 additions & 30 deletions source/speech/speech.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1946,22 +1946,11 @@ def getPropertiesSpeech( # noqa: C901

def _rowAndColumnCountText(rowCount: int, columnCount: int) -> Optional[str]:
if rowCount and columnCount:
rowCountTranslation: str = ngettext(
# Translators: Sub-part of the compound string to speak number of columns and rows in a table
"{rowCount} row",
"{rowCount} rows",
rowCount,
).format(rowCount=rowCount)
colCountTranslation: str = ngettext(
# Translators: Sub-part of the compound string to speak number of columns and rows in a table
"{columnCount} column",
"{columnCount} columns",
columnCount,
).format(columnCount=columnCount)
rowCountTranslation: str = _rowCountText(rowCount)
colCountTranslation: str = _columnCountText(columnCount)
# Translators: Main part of the compound string to speak number of columns and rows in a table
# Example output: "with 3 rows and 2 columns"
# In this example {rowCountTranslation} will be replaced by "3 rows" and {colCountTranslation} by
# "2 columns"
# Example: If the reported compound string is "with 3 rows and 2 columns", {rowCountTranslation} will be
# replaced by "3 rows" and {colCountTranslation} by "2 columns"
return _("with {rowCountTranslation} and {colCountTranslation}").format(
rowCountTranslation=rowCountTranslation,
colCountTranslation=colCountTranslation,
Expand All @@ -1975,6 +1964,26 @@ def _rowAndColumnCountText(rowCount: int, columnCount: int) -> Optional[str]:
return None


def _rowCountText(count: int) -> str:
return ngettext(
# Translators: Sub-part of the compound string to speak number of rows and columns in a table.
# Example: If the full compound string is "table with 3 rwos and 2 columns", this substring is "3 rows".
"{rowCount} row",
"{rowCount} rows",
count,
).format(rowCount=count)


def _columnCountText(count: int) -> str:
return ngettext(
# Translators: Sub-part of the compound string to speak number of rows and columns in a table.
# Example: If the full compound string is "table with 3 rwos and 2 columns", this substring is "2 columns".
"{columnCount} column",
"{columnCount} columns",
count,
).format(columnCount=count)


def _shouldSpeakContentFirst(
reason: OutputReason,
role: int,
Expand Down Expand Up @@ -2804,23 +2813,12 @@ def getTableInfoSpeech(
textList=[]
if newTable:
columnCount=tableInfo.get("column-count",0)
# Translators: Sub-part of the compound string to report a table
rowCount=tableInfo.get("row-count",0)
columnCountText = ngettext(
# Translators: Sub-part of the compound string to report a table
"{columnCount} column",
"{columnCount} columns",
columnCount,
).format(columnCount=columnCount)
rowCountText = ngettext(
# Translators: Sub-part of the compound string to report a table
"{rowCount} rows",
"{rowCount} rows",
rowCount,
).format(rowCount=rowCount)
columnCountText = _columnCountText(columnCount)
rowCountText = _rowCountText(rowCount)
# Translators: Main part of the compound string to report a table
# Example output: table with 3 columns and 5 rows
# {columnCountText} is replaced by "3 columns" and {rowCountText} by "5 rows"
# Example: If the reported compound string is "table with 2 columns and 3 rows", {colCountTranslation}
# will be replaced by "2 columns" and {rowCountTranslation} by "3 rows"
text = _("table with {columnCountText} and {rowCountText}").format(
columnCountText=columnCountText,
rowCountText=rowCountText,
Expand Down