Skip to content

Share RenderInExponentialNotation AO with Number.prototype.toExponential - #165

Open
jessealama wants to merge 1 commit into
round-mathematical-valuesfrom
toexponential-shared-rendering
Open

jessealama wants to merge 1 commit into
round-mathematical-valuesfrom
toexponential-shared-rendering

Conversation

@jessealama

Copy link
Copy Markdown
Collaborator

This PR splits another chunk out of #161. RenderInExponentialNotation moves into a new ECMA-262 amendments section; Number.prototype.toExponential is updated to call it. The digit selection steps of .toExponential are untouched. Only its rendering steps are replaced by a call to RenderInExponentialNotation that passes the selected value and its digit count.

This work is stacked on top of #164, which is itself stacked on #110, but can be reviewed independently.

@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
PR Preview Action v1.8.1

QR code for preview link

🚀 View preview at
https://tc39.github.io/proposal-amount/pr-preview/pr-165/

Built to branch gh-pages at 2026-09-29 23:38 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@jessealama
jessealama requested a review from eemeli September 23, 2026 11:33
@jessealama jessealama changed the title Share RenderInExponentialNotation with Number.prototype.toExponential Share RenderInExponentialNotation AO with Number.prototype.toExponential Sep 23, 2026
`RenderInExponentialNotation` moves into a new ECMA-262 amendments section; `Number.prototype.toExponential` is updated to call it. The digit selection steps of `.toExponential` are untouched. Only its rendering steps are replaced by a call to `RenderInExponentialNotation` that passes the selected value and its digit count.
@jessealama
jessealama force-pushed the toexponential-shared-rendering branch from 21e3270 to 09ac85d Compare September 29, 2026 23:37
@jessealama
jessealama force-pushed the round-mathematical-values branch from 2db4c7f to 147c8a1 Compare September 29, 2026 23:37

This branch has not been deployed

No deployments
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.

1 participant