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

Add Traditional Chinese localization #667

Merged

Conversation

Gasol
Copy link
Contributor

@Gasol Gasol commented Dec 10, 2024

This change adds Traditional Chinese localization and return it when the
locale is zh-Hant. We have to use the languageCode and scriptCode to
differentiate between Simplified and Traditional Chinese.

Summary of changes:

  • Implement TraditionalChineseAssetPickerTextDelegate
  • Add TraditionalChineseAssetPickerTextDelegate to the list of delegates.
  • Sort the list of delegates alphabetically.
  • Remove all toLowerCase() calls to make sure we can return the correct
    delegate.
RPReplay_Final1733835936.mp4

Copy link
Member

@AlexV525 AlexV525 left a comment

Choose a reason for hiding this comment

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

Please rebase on the latest changes, and also provide a screen recording using TalkBack or VoiceOver to ensure the accessibility feature works well with the delegate.

example/lib/l10n/app_zh-hant.arb Outdated Show resolved Hide resolved
example/lib/l10n/gen/app_localizations.dart Outdated Show resolved Hide resolved
example/lib/l10n/gen/app_localizations_zh.dart Outdated Show resolved Hide resolved
lib/src/delegates/asset_picker_text_delegate.dart Outdated Show resolved Hide resolved
test/delegates_test.dart Outdated Show resolved Hide resolved
@Gasol Gasol force-pushed the traditional-chinese-text-delegate branch from 9ad3d36 to 96310ae Compare December 11, 2024 15:53
@Gasol Gasol requested a review from AlexV525 December 11, 2024 16:01
@Gasol Gasol force-pushed the traditional-chinese-text-delegate branch from 96310ae to e6e1a2f Compare December 12, 2024 02:11
@AlexV525
Copy link
Member

AlexV525 commented Dec 12, 2024

I saw the video with the VoiceOver but it doesn't show the actual grid. Please attach a video that walks through actual assets. If the video needs to be no public sharing, send it to fluttercandies@googlegroups.com.

EDIT: There might be attachment size limitations with Google Groups. Let us know if you need to send the email and was sent.

This change adds Traditional Chinese localization and return it when the
locale is zh-Hant. We have to use the languageCode and scriptCode to
differentiate between Simplified and Traditional Chinese.

Summary of changes:
- Implement TraditionalChineseAssetPickerTextDelegate
- Add TraditionalChineseAssetPickerTextDelegate to the list of delegates.
- Sort the list of delegates alphabetically.
- Remove all toLowerCase() calls to make sure we can return the correct
  delegate.
@Gasol Gasol force-pushed the traditional-chinese-text-delegate branch from e6e1a2f to 227558a Compare December 12, 2024 08:50
@Gasol
Copy link
Contributor Author

Gasol commented Dec 12, 2024

I saw the video with the VoiceOver but it doesn't show the actual grid. Please attach a video that walks through actual assets. If the video needs to be no public sharing, send it to fluttercandies@googlegroups.com.

EDIT: There might be attachment size limitations with Google Groups. Let us know if you need to send the email and was sent.

Here you go, The link will expire on Jan 11, 2015.

https://share.icloud.com/photos/08cMUC9AatA-2zmAVQRRVVcSA

@Gasol Gasol requested a review from AlexV525 December 12, 2024 14:53
Copy link
Member

@AlexV525 AlexV525 left a comment

Choose a reason for hiding this comment

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

Cool. Thanks for the contribution! I'll take care of the rest.

@AlexV525 AlexV525 merged commit 1f31568 into fluttercandies:main Dec 12, 2024
1 check passed
@AlexV525
Copy link
Member

@all-contributors Add @Gasol for localizations.

Copy link
Contributor

@AlexV525

I couldn't determine any contributions to add, did you specify any contributions?
Please make sure to use valid contribution names.

@AlexV525
Copy link
Member

@all-contributors Add @Gasol for translation

Copy link
Contributor

@AlexV525

I've put up a pull request to add @Gasol! 🎉

AlexV525 pushed a commit that referenced this pull request Dec 12, 2024
Adds @Gasol as a contributor for translation.

This was requested by AlexV525 [in this
comment](#667 (comment))

[skip ci]

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
tourdt pushed a commit to tourdt/flutter_wechat_assets_picker that referenced this pull request Dec 16, 2024
…at_assets_picker

* 'main' of /~https://github.com/fluttercandies/flutter_wechat_assets_picker:
  docs: add Gasol as a contributor for translation (fluttercandies#669)
  Add Traditional Chinese localization (fluttercandies#667)
  🚀 Support matching script and country code of locales with the text delegate (fluttercandies#668)
  💡 Refreshes tests copyright
  🔖 9.4.1
  🔖 9.4.1
  Fix selecting when reached the max asset limit in the preview (fluttercandies#648)
  🚸 Improves the default sort conditions (fluttercandies#662)
  ⬆️ Allows `extended_image: ^9.0.0` (fluttercandies#659)
  🐛 Allows assets changing when no path previously (fluttercandies#656)
  👷 Try to run on minimum SDK requirement (fluttercandies#643)
  🔖 9.3.2
  🚀 Adapt the image provider file type getter (fluttercandies#638)
  🔖 9.3.1
  🐛 Fix UI layouts (fluttercandies#634)
tourdt pushed a commit to tourdt/flutter_wechat_assets_picker that referenced this pull request Dec 16, 2024
* main:
  docs: add Gasol as a contributor for translation (fluttercandies#669)
  Add Traditional Chinese localization (fluttercandies#667)
  🚀 Support matching script and country code of locales with the text delegate (fluttercandies#668)
  💡 Refreshes tests copyright
  🔖 9.4.1
  🔖 9.4.1
  Fix selecting when reached the max asset limit in the preview (fluttercandies#648)
  🚸 Improves the default sort conditions (fluttercandies#662)
  ⬆️ Allows `extended_image: ^9.0.0` (fluttercandies#659)
  🐛 Allows assets changing when no path previously (fluttercandies#656)
  👷 Try to run on minimum SDK requirement (fluttercandies#643)
  🔖 9.3.2
  🚀 Adapt the image provider file type getter (fluttercandies#638)
  🔖 9.3.1
  🐛 Fix UI layouts (fluttercandies#634)

# Conflicts:
#	lib/src/delegates/asset_picker_builder_delegate.dart
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