-
-
Notifications
You must be signed in to change notification settings - Fork 468
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.
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
FluentUI Widgets Localization Support #216
Conversation
Awesome ! 😄 |
You can consider this PR finished, so it can be merged if you like it. |
Were these generated by the translator? I don't like the translator idea. For example, the Portuguese translation is totally incorrect |
@bdlukaa English ones are the original ones an I've translated manually the Spanish ones. In this cases I always prefer a bad translation vs no-translation at all, but this is something that is very easy to fix with this new implementation. If you prefer it we can confirm two or three new localizations and remove the rest. @bdlukaa Do you know who can verify the other ones? |
It's better a confirmed translation IMO
I'll check on my discord server (https://discord.gg/674gpDQUVq) |
Sounds great! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Translator isn't that bad tho
Co-authored-by: Bruno D'Luka <45696119+bdlukaa@users.noreply.github.com>
Co-authored-by: Bruno D'Luka <45696119+bdlukaa@users.noreply.github.com>
Co-authored-by: Bruno D'Luka <45696119+bdlukaa@users.noreply.github.com>
{
"@@locale": "ru",
"backButtonTooltip": "Назад",
"closeButtonLabel": "Закрыть",
"searchLabel": "Поиск",
"closeNavigationTooltip": "Закрыть панель навигации",
"openNavigationTooltip": "Открыть панель навигации",
"clickToSearch": "Нажмите для поиска",
"modalBarrierDismissLabel": "Отмена",
"minimizeWindowTooltip": "Свернуть",
"restoreWindowTooltip": "Восстановить",
"closeWindowTooltip": "Закрыть",
"dialogLabel": "Диалог",
"cutActionLabel": "Вырезать",
"copyActionLabel": "Копировать",
"pasteActionLabel": "Вставить",
"selectAllActionLabel": "Выбрать все",
"newTabLabel": "Новая вкладка",
"closeTabLabelSuffix": "Закрыть вкладку",
"scrollTabBackwardLabel": "Прокрутить назад",
"scrollTabForwardLabel": "Прокрутить вперед",
"noResultsFoundLabel": "Результаты не найдены",
"copyActionTooltip": "Скопировать в буфер обмена",
"cutActionTooltip": "Вырезать и поместить в буфер обмена",
"pasteActionTooltip": "Вставить содержимое буфера обмена",
"selectAllActionTooltip": "Выбрать все"
} |
Update to the newer spelling, according to Wikipedia: https://fr.wikipedia.org/wiki/Presse-papier_(informatique)#:~:text=En%20informatique%2C%20un%20presse%2Dpapier%20(ou%20presse%2Dpapiers%20selon%20l%27ancienne%20orthographe Co-authored-by: Kevin Vuilleumier <WinXaito@users.noreply.github.com>
Add initial support for German
@alexmercerind Merged 👏🏻 thanks!! |
I've reviewed Simplified Chinese, thanks! {
"@@locale": "zh",
"backButtonTooltip": "返回",
"closeButtonLabel": "关闭",
"searchLabel": "搜索",
"closeNavigationTooltip": "关闭导航",
"openNavigationTooltip": "打开导航",
"clickToSearch": "点击搜索",
"modalBarrierDismissLabel": "取消",
"minimizeWindowTooltip": "最小化",
"restoreWindowTooltip": "恢复",
"closeWindowTooltip": "关闭",
"dialogLabel": "对话",
"cutActionLabel": "剪切",
"copyActionLabel": "复制",
"pasteActionLabel": "粘贴",
"selectAllActionLabel": "全选",
"newTabLabel": "添加新标签",
"closeTabLabelSuffix": "关闭标签",
"scrollTabBackwardLabel": "向后滚动标签列表",
"scrollTabForwardLabel": "向前滚动标签列表",
"noResultsFoundLabel": "没有找到结果",
"copyActionTooltip": "将选中的内容复制到剪贴板",
"cutActionTooltip": "将选中的内容剪切到剪贴板",
"pasteActionTooltip": "在当前位置插入剪贴板的内容",
"selectAllActionTooltip": "选择所有内容"
}
|
Merged @zacksleo, you're welcomed! 🚀 PS: We only need the contents of the |
I've added detailed instructions in |
I'm just waiting the German translation for remove |
@henry2man can this be marked as ready for review? |
I'm still waiting 🤷🏻♂️ for #216 (comment) review. If you want to release now, I can remove unverified localizations ( |
yup. sounds good! |
I translated the Arabic language.
|
@dmakwt Great! 🥇Can you create a new pull request like @dubh3 did in #267? Please follow the instructions here: https://github.com/bdlukaa/fluent_ui#contributing-new-localizations |
@henry2man yes sure 👍 #268 |
Hi, I have done a translation into Czech. { |
This PR adds support for onboard widgets localization support using
intl
package.Related with #213
I've included a wide list of most spoken languages:
Arabic, Bengali, English, Spanish, French, Hindi, Indonesian, Portuguese, Russian and Simplified Chinese using Google Translate.UPDATE: ⭐️ @bdlukaa and I (@henry2man) agreed that we prefer confirmed localizations vs "Google Translate" ones, so we need some help!How to contribute:
Current status:
I've added detailed instructions for adding new localizations in the
README.md
: https://github.com/bdlukaa/fluent_ui/pull/216/files?short_path=b335630#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5If you run the example app and change the language of the browser, the corresponding messages are shown.
Also I've done some little tweaks to shortcuts label implementation. AFAIK they aren't strictly that needs a translation (
Ctrl
orCmd
are standards names for corresponding keys).Please take notice of changes here: https://github.com/bdlukaa/fluent_ui/pull/216/files#diff-3705042816f3bd9d24304192811bed9d5104fa569a36f063a4166cfea97b845aR376-R377 and here https://github.com/bdlukaa/fluent_ui/pull/216/files#diff-8b7e9df87668ffa6a04b32e1769a33434999e54ae081c52e5d943c541d4c0d25R13-R14 in order to support multilanguage material localizations.
Lastly, take a look at this
List<Locale>
constant: https://github.com/bdlukaa/fluent_ui/pull/216/files#diff-ccec5c5fa2c4d7bf6bc0e9748a8836c4bc5ff4a8c59461448d14047f6b242617R128-R139As the note says, this would be defined into
DefaultFluentLocalizations
but I cannot be able to make it work, so this was a working workarround.Pre-launch Checklist
dartfmt
on all changed filesCHANGELOG.md
with my changesflutter pub publish --dry-run
and addressed any warnings