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.

Sign up for GitHub

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

Jump to bottom

Range with negative values #11

Closed
sffc opened this issue Apr 29, 2020 · 3 comments
Closed

Range with negative values #11

sffc opened this issue Apr 29, 2020 · 3 comments
Labels
has-consensus Has consensus and ready to implement

Comments

@sffc
Copy link
Collaborator

sffc commented Apr 29, 2020

I think we should throw RangeError when either of the inputs to formatRange() is negative.

new Intl.NumberFormat().formatRange(-5, -3);  // RangeError

CLDR's number range data is very limited and does not cover this case in a cohesive way. Better to forbid it, and we can add it later if we need to.
@sffc sffc added the discuss Needs discussion to make progress label Oct 8, 2020
@sffc
Copy link
Collaborator Author

sffc commented Oct 8, 2020

Possible use cases:

@sffc
Copy link
Collaborator Author

sffc commented Oct 8, 2020

2020-10-08: We'll allow negative numbers. Adopt current behavior for negative plurals as well (always return "other").

@sffc sffc added has-consensus Has consensus and ready to implement and removed discuss Needs discussion to make progress labels Oct 8, 2020
@sffc
Copy link
Collaborator Author

sffc commented Oct 8, 2020

2020-10-08 discussion: https://github.com/tc39/ecma402/blob/master/meetings/notes-2020-10-08.md#range-with-negative-values-11

@sffc sffc mentioned this issue Jan 14, 2021
Closed
@sffc sffc closed this as completed in 8d8e9a6 Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
has-consensus Has consensus and ready to implement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update README with formatRange negative numbers and signDisplay tc39/proposal-intl-numberformat-v3
1 participant
@sffc