Skip to content

Drop the notation, compactDisplay, & numberingSystem options #1015

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

Merged
merged 1 commit into from
Feb 17, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 2 additions & 4 deletions spec/formatting.md
Original file line number Diff line number Diff line change
Expand Up @@ -116,10 +116,10 @@ which makes it available for use in later _expressions_ and _markup_ _options_.
> For example, in
> ```
> .input {$a :number minimumFractionDigits=3}
> .local $b = {$a :integer notation=compact}
> .local $b = {$a :integer useGrouping=never}
Copy link
Member

Choose a reason for hiding this comment

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

We should make this change regardless

> .match $a
> 0 {{The value is zero.}}
> * {{In compact form, the value {$a} is rendered as {$b}.}}
> * {{Without grouping separators, the value {$a} is rendered as {$b}.}}
> ```
> the _resolved value_ bound to `$a` is used as the _operand_
> of the `:integer` _function_ when resolving the value of the _variable_ `$b`,
Expand Down Expand Up @@ -992,5 +992,3 @@ The _Default Bidi Strategy_ is defined as follows:
> the character sequence in the formatted string representation
> of `resval`.



3 changes: 0 additions & 3 deletions spec/functions/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,6 @@ Such values can be useful to users in cases where local usage and support exists
> for currency amounts as the _operand_ in the _function_ `:currency`.
> - A Java implementation might _accept_ a `java.time.chrono.Chronology` object
> as a value for the _date/time override option_ `calendar`
> - ICU4J's implementation might _accept_ a `com.ibm.icu.text.NumberingSystem` object
> instead of using a [Unicode Numbering System Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeNumberSystemIdentifier)
> for the option `numberingSystem` in _functions_ such as `:number` or `:integer`.
Comment on lines -76 to -78
Copy link
Member

Choose a reason for hiding this comment

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

In an ideal world we would find a replacement here, but not this morning.


Future versions of this specification MAY define additional _options_ and _option_ values,
subject to the rules in the [Stability Policy](#stability-policy),
Expand Down
4 changes: 1 addition & 3 deletions spec/functions/datetime.md
Original file line number Diff line number Diff line change
Expand Up @@ -272,13 +272,11 @@ the functions `:datetime` and `:time`:
- `true`
- `false`

The following _options_ and their values are RECOMMENDED to be available on
The following _option_ and its values are RECOMMENDED to be available on
the functions `:datetime`, `:date`, and `:time`.

- `calendar`
- valid [Unicode Calendar Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeCalendarIdentifier)
- `numberingSystem`
- valid [Unicode Number System Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeNumberSystemIdentifier)

The following _option_ and its values are **Proposed** for
inclusion in the next release of this specification but have not yet been
Expand Down
39 changes: 2 additions & 37 deletions spec/functions/number.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,17 +26,6 @@ The following options and their values are REQUIRED to be available on the funct
- `plural` (default; see [Default Value of `select` Option](#default-value-of-select-option) below)
- `ordinal`
- `exact`
- `compactDisplay` (this option only has meaning when combined with the option `notation=compact`)
- `short` (default)
- `long`
- `notation`
- `standard` (default)
- `scientific`
- `engineering`
- `compact`
- `numberingSystem`
- valid [Unicode Number System Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeNumberSystemIdentifier)
(default is locale-specific)
- `signDisplay`
- `auto` (default)
- `always`
Expand Down Expand Up @@ -87,12 +76,12 @@ with _options_ on the _expression_ taking priority over any option values of the
> For example, the _placeholder_ in this _message_:
>
> ```
> .input {$n :number notation=scientific minimumFractionDigits=2}
> .input {$n :number minimumFractionDigits=2 signDisplay=always}
> {{{$n :number minimumFractionDigits=1}}}
> ```
>
> would be formatted with the resolved options
> `{ notation: 'scientific', minimumFractionDigits: '1' }`.
> `{ minimumFractionDigits: '1', signDisplay: 'always' }`.

#### Resolved Value

Expand Down Expand Up @@ -132,9 +121,6 @@ The following options and their values are REQUIRED to be available on the funct
- `plural` (default; see [Default Value of `select` Option](#default-value-of-select-option) below)
- `ordinal`
- `exact`
- `numberingSystem`
- valid [Unicode Number System Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeNumberSystemIdentifier)
(default is locale-specific)
- `signDisplay`
- `auto` (default)
- `always`
Expand All @@ -158,8 +144,6 @@ In general, these are included in the resolved option values of the _expression_
with _options_ on the _expression_ taking priority over any option values of the _operand_.
Option values with the following names are however discarded if included in the _operand_:

- `compactDisplay`
- `notation`
- `minimumFractionDigits`
- `maximumFractionDigits`
- `minimumSignificantDigits`
Expand Down Expand Up @@ -355,15 +339,6 @@ The following options and their values are REQUIRED to be available on the funct
- `currency`
- well-formed [Unicode Currency Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeCurrencyIdentifier)
(no default)
- `compactDisplay` (this option only has meaning when combined with the option `notation=compact`)
- `short` (default)
- `long`
- `notation`
- `standard` (default)
- `compact`
- `numberingSystem`
- valid [Unicode Number System Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeNumberSystemIdentifier)
(default is locale-specific)
- `currencySign`
- `accounting`
- `standard` (default)
Expand Down Expand Up @@ -492,15 +467,6 @@ unless otherwise indicated:
- `short` (default)
- `narrow`
- `long`
- `compactDisplay` (this option only has meaning when combined with the option `notation=compact`)
- `short` (default)
- `long`
- `notation`
- `standard` (default)
- `compact`
- `numberingSystem`
- valid [Unicode Number System Identifier](https://unicode.org/reports/tr35/tr35.html#UnicodeNumberSystemIdentifier)
(default is locale-specific)
- `signDisplay`
- `auto` (default)
- `always`
Expand Down Expand Up @@ -783,7 +749,6 @@ representing its decimal value:
- `minimumIntegerDigits`
- `minimumSignificantDigits`
- `maximumSignificantDigits`
- `notation`

```abnf
integer = "0" / ["-"] ("1"-"9") *DIGIT
Expand Down