Skip to content
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

feat: Add i18n (FR language) + l10n (calendar + currency formatting) #20

Merged
merged 7 commits into from
Dec 13, 2022

Conversation

liamjack
Copy link
Contributor

Hi @tim-smart,

I've started to work on a few suggestions from issue #19:

  • i18n: Translate to different languages: added french language
  • l10n: Uses user's locale to choose start day for the week (date picker)
  • l10n: Uses user's locale to choose currency and format numbers correctly

Also changed the "Toggle flash" button to an icon button (the french translation is pretty long)

I'm new to Flutter and Dart so I've probably done a few things wrong, but I'm interested in your feedback. I've only been able to test on an Android device.

Here are some screenshots:

Date picker in french with week starting on monday:

Screenshot_20221213-233739

Euro currency + french number formatting:

Screenshot_20221213-233805

"Toggle flash" button replaced with flash icon:

Screenshot_20221213-233821

French language in new voucher form:

Screenshot_20221213-233837

@tim-smart tim-smart merged commit 6ec617d into tim-smart:main Dec 13, 2022
@tim-smart
Copy link
Owner

Thanks!

If you can add the remaining translations for the menu in another PR that would be great.

I'll push out a beta release on the Play store soon.

@liamjack
Copy link
Contributor Author

liamjack commented Dec 14, 2022

If you can add the remaining translations for the menu in another PR that would be great.

Yes I'll get round to that shortly, just need to work out how to use i18n inside an enum !

Thanks for merging and for future release :)

@tim-smart
Copy link
Owner

I have already solved that for you, the only thing missing now is the translations.

@liamjack
Copy link
Contributor Author

Sorry I didn't notice your commits on top of the PR, thanks for that, I'll add the translations right away

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