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

ICU standards for numbers formatting works partially #175

Open
BigBadAlien opened this issue Sep 28, 2021 · 5 comments
Open

ICU standards for numbers formatting works partially #175

BigBadAlien opened this issue Sep 28, 2021 · 5 comments

Comments

@BigBadAlien
Copy link

BigBadAlien commented Sep 28, 2021

As described in the readme:

Message format is strictly implemented by ICU standards.

But I tested the number formatting from here:

https://unicode-org.github.io/icu/userguide/format_parse/numbers/skeletons.html

and most of the examples don't work, such as:

{value, number, ::.####}

or

.00/w

But

{value, number, ::.0####}

works

@BigBadAlien
Copy link
Author

Looks like the bug in another package. Created formatjs/formatjs#3193

@BigBadAlien
Copy link
Author

To resolve the issue require an update intl-messageformat to 9.9.3

@BigBadAlien BigBadAlien reopened this Oct 18, 2021
@cwtuan
Copy link
Collaborator

cwtuan commented Oct 19, 2021

intl-messageformat@9.9.3 would breaks the test cases.
I will take time to fix it.

@BigBadAlien
Copy link
Author

Any plans to update intl-messageformat to the last version? A lot of helpful skeletons there like integer-width, scale, etc... Also, the current version looks really ancient (7.8.4 compare to 10.2.5)

@cwtuan
Copy link
Collaborator

cwtuan commented Dec 25, 2022

Thanks for reporting this issue.
Since theintl-messageformat breaks API behavior, we are looking forward some way to solve it.
By the way, PR is also welcome.

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

No branches or pull requests

2 participants