Skip to content

Cupertino localization step 12: push translation for all supported languages - #31644

Merged
xster merged 6 commits into
flutter:masterfrom
xster:push
May 10, 2019
Merged

xster merged 6 commits into
flutter:masterfrom
xster:push

Conversation

@xster

@xster xster commented Apr 26, 2019 •

Copy link
Copy Markdown
Member

Fixes #13452

When making change suggestions, please double check in iOS so I can justify it with our translators.

@xster
xster requested a review from HansMuller April 26, 2019 05:02
@goderbauer goderbauer added p: cupertino_ui cupertino_ui package in flutter/packages p: material_ui material_ui package in flutter/packages framework flutter/packages/flutter repository. See also f: labels. labels Apr 29, 2019
@xster

xster commented May 1, 2019 •

Copy link
Copy Markdown
Member Author

Pending another xml push into TC before pulling again

@xster xster changed the title Push translations from TC Cupertino localization step 12: push translation for all supported languages May 3, 2019
@xster xster added the a: internationalization Supporting other languages or locales. (aka i18n) label May 3, 2019
@@ -0,0 +1,22 @@
{
"todayLabel": "a temporary hack",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

the latin script in this file is a placeholder while the remaining 3 strings are being translated

@@ -0,0 +1,22 @@
{
"todayLabel": "a temporary hack",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

the latin script in this file is a placeholder while the remaining 3 strings are being translated

Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_ru.arb
Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_ru.arb
Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_ru.arb
Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_ru.arb

@lgfranco lgfranco left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I made the changes to reflect the iOS in pt and pt_PT

@@ -0,0 +1,22 @@
{
"todayLabel": "Hoje",
"datePickerHourSemanticsLabelOne": "$hour em ponto",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

"$hour hora"

Suggested change
"datePickerHourSemanticsLabelOne": "$hour em ponto",
"datePickerHourSemanticsLabelOne": "$hour hora",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks for diff'ing and reviewing. We'll backport these into the translation console

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I assume these are both for brazilian and portugal portuguese?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yes, the use of "em ponto" is not for this situation.

The use wold be for something like 8 o'clock (8 "horas em ponto").

Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_pt.arb
Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_pt.arb
Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_pt.arb
"timerPickerHourLabelOther": "horas",
"timerPickerMinuteLabelOne": "min",
"timerPickerMinuteLabelOther": "min",
"timerPickerSecondLabelOne": "s",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The correct would be s but the iOS writes "seg" in this locale.

Suggested change
"timerPickerSecondLabelOne": "s",
"timerPickerSecondLabelOne": "seg",

Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_pt.arb Outdated
"timerPickerHourLabelOther": "horas",
"timerPickerMinuteLabelOne": "min",
"timerPickerMinuteLabelOther": "min",
"timerPickerSecondLabelOne": "seg",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
"timerPickerSecondLabelOne": "seg",
"timerPickerSecondLabelOne": "s",

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seems like you're suggesting opposite things between here and the same label in the previous file.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Yes, for some reason the iOS abbreviates seconds in different ways in pt and pt_PT.

I changed my iPhone to pt_PT, accessed the Timer (in the clock App), and used the strings I found there.

"timerPickerMinuteLabelOne": "min",
"timerPickerMinuteLabelOther": "min",
"timerPickerSecondLabelOne": "seg",
"timerPickerSecondLabelOther": "seg",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggested change
"timerPickerSecondLabelOther": "seg",
"timerPickerSecondLabelOther": "s",

Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_pt_PT.arb
Comment thread packages/flutter_localizations/lib/src/l10n/cupertino_pt_PT.arb

@HansMuller HansMuller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

RSLGTM

Translation feedback will have to be funneled through the Google TC

@lgfranco

lgfranco commented May 7, 2019

Copy link
Copy Markdown

@xster

xster commented May 10, 2019

Copy link
Copy Markdown
Member Author

I'm merging this one in for now. The feedback received will cycle through via another update once the TC translations are internalized.

@xster
xster merged commit 10922df into flutter:master May 10, 2019
@xster
xster deleted the push branch May 10, 2019 06:19
xster added a commit that referenced this pull request May 10, 2019
xster added a commit that referenced this pull request May 10, 2019
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

a: internationalization Supporting other languages or locales. (aka i18n) framework flutter/packages/flutter repository. See also f: labels. p: cupertino_ui cupertino_ui package in flutter/packages p: material_ui material_ui package in flutter/packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need Cupertino Translations

6 participants