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

polyfill: parsing UTCOffset with sub-minute precision #3097

Open
fabon-f opened this issue Feb 26, 2025 · 1 comment
Open

polyfill: parsing UTCOffset with sub-minute precision #3097

fabon-f opened this issue Feb 26, 2025 · 1 comment
Labels
no-spec-text PR can be ignored by implementors non-prod-polyfill THIS POLYFILL IS NOT FOR PRODUCTION USE! test262

Comments

@fabon-f
Copy link
Contributor

fabon-f commented Feb 26, 2025

Temporal.ZonedDateTime.from("2025-01-01T00:00:00+00:0000[Europe/London]");

In Firefox Nightly, this code raises an error (Uncaught RangeError: can't parse zoned date-time: time separator ':' must be consistent). But reference polyfill doesn't raise an error.

My understanding is that spec requires UTCOffset with sub-minute precision should be hh:mm:ss or hhmmss formats, which means the polyfill doesn't align the spec.

https://tc39.es/proposal-temporal/#prod-UTCOffset

@ptomato ptomato added non-prod-polyfill THIS POLYFILL IS NOT FOR PRODUCTION USE! test262 no-spec-text PR can be ignored by implementors labels Feb 28, 2025
@ptomato
Copy link
Collaborator

ptomato commented Feb 28, 2025

Thanks for the report. I think your understanding is correct. This should be fixed in the polyfill, but should also have test262 coverage added; the fact that it went unnoticed probably means it's not covered by tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-spec-text PR can be ignored by implementors non-prod-polyfill THIS POLYFILL IS NOT FOR PRODUCTION USE! test262
Projects
None yet
Development

No branches or pull requests

2 participants