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

Documenting the new client-customizable locale detection system #38

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mesr
Copy link
Contributor

@mesr mesr commented Jul 19, 2023

  • Updated the description of the language detection system at the top of the page;
  • Added a section for the new setLocaleDetectionRule(rule) function;
  • Disambiguated the role of setLocale(locale) vs setLocaleDetectionRule(rule) when the latter is called with a string argument.

@mesr mesr changed the title Update create-i18n.md Documented the new user-customizable locale detection system Jul 19, 2023
@mesr mesr changed the title Documented the new user-customizable locale detection system Documented the new client-customizable locale detection system Jul 19, 2023
@mesr mesr force-pushed the main branch 10 times, most recently from af20ad7 to 86a0e05 Compare July 24, 2023 19:03
  - Updated description of the locale detection procedure at the top of page `i18n/create-i18n.md`;
  - Added section `detectLocale(callback)` to pages `extending-components/with-i18n.md` and `extending-components/with-18n-reactive.md`;
  - Added sections `setLocaleDetectionRule(rule)` and `detectLocale(callback)` to page `i18n/create-i18n.md`.
@mesr mesr changed the title Documented the new client-customizable locale detection system Documenting the new client-customizable locale detection system Jul 24, 2023
@mesr
Copy link
Contributor Author

mesr commented Jul 27, 2023

@ducksoupdev, please wait before committing this PR. There is an additional feature that I wish to propose and that might change the definition and signature of function detectLocale(callback).

@ducksoupdev
Copy link
Contributor

@mesr thank you for all your contributions so far, it is much appreciated. Once this one is ready to merge, I'll create a blog post off the back of it to promote the new features.

@mesr
Copy link
Contributor Author

mesr commented Jul 31, 2023

My pleasure, @ducksoupdev. Thank you for welcoming my contribution. The appreciation goes in both directions!

The documentation is almost ready to go.

If you haven't already, would you mind taking a look at this proposal? Because that's an easy one to get done and, if you think it's a good idea, I feel that it would conclude very nicely this batch of new features by tying together the new reactive lang attribute with the new I18n.detectLocale() method and its callback argument.

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.

None yet

2 participants