Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

@GaryQian
Copy link
Contributor

@GaryQian GaryQian commented Apr 16, 2020

This implements the core engine message system for passing PlatformResolvedLocale from embedders up to the dart isolate.

There is also a very simple android usage of the API that will just produce en_US. This value is currently discarded by the framework until the necessary changes are implemented there.

Part of a series of changes to address flutter/flutter#47087

doc: go/flutter-embedder-localization

@auto-assign auto-assign bot requested a review from flar April 16, 2020 11:34
@GaryQian GaryQian removed the request for review from flar April 17, 2020 07:40
@GaryQian GaryQian changed the title [WIP] PlatformResolvedLocale localization message channel PlatformResolvedLocale localization message channel Apr 17, 2020
Copy link
Member

@cbracken cbracken left a comment

Choose a reason for hiding this comment

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

LGTM stamp from a Japanese personal seal

/// The locale that the platform's native locale resolution system resolves to.
///
/// This value may differ between platforms and is meant to allow flutter locale
/// resoltion algorithms to into resolving consistently with other apps on the
Copy link
Member

Choose a reason for hiding this comment

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

typo: "resolution"

also "to into resolving" doesn't sound right


/// The locale that the platform's native locale resolution system resolves to.
///
/// This value may differ between platforms and is meant to allow flutter locale
Copy link
Member

Choose a reason for hiding this comment

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

capitalize "Flutter"

@GaryQian GaryQian added waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. and removed waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. labels Apr 21, 2020
@GaryQian GaryQian merged commit 8dc9122 into flutter:master Apr 21, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 22, 2020
swift-kim added a commit to swift-kim/engine that referenced this pull request Jul 16, 2021
The code is dead because flutter#17755 has been reverted by flutter#19136
but flutter#18519 has not been properly reverted.
swift-kim added a commit to swift-kim/engine that referenced this pull request Jul 16, 2021
The code is dead because flutter#17755 has been reverted by flutter#19136,
although flutter#18519 has not been properly reverted.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants