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

Correction: Use null instead of NaN when stationary #173

Merged
merged 1 commit into from
Aug 14, 2024
Merged

Conversation

reillyeon
Copy link
Member

@reillyeon reillyeon commented Aug 14, 2024

This ammends the existing correction clarifying the units and conditions for the heading property. This matches existing implementations which return null in this scenario.

Closes #171.

The following tasks have been completed:

  • Modified Web platform tests (link to pull request)

Implementation commitment (and no objections):

  • WebKit (existing behavior)
  • Chromium (existing behavior)
  • Gecko (existing behavior)

Documentation (new feature):

For documentation, either create an issue or pull request in MDN's Content repo - providing as much information as you can. PR is prefered.


Preview | Diff

This ammends the existing correction clarifying the units and conditions
for the heading property. This matches existing implementations which
return null in this scenario.
reillyeon added a commit to reillyeon/content that referenced this pull request Aug 14, 2024
Matches specification change in w3c/geolocation#173 and discussion about existing implementations in w3c/geolocation#171.
@reillyeon reillyeon merged commit 9d5829e into main Aug 14, 2024
4 checks passed
@reillyeon reillyeon deleted the remove_nan branch August 14, 2024 19:46
github-actions bot added a commit that referenced this pull request Aug 14, 2024
SHA: 9d5829e
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Aug 15, 2024
SHA: 9d5829e
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Josh-Cena pushed a commit to mdn/content that referenced this pull request Aug 24, 2024
* Remove reference to NaN heading value

Matches specification change in w3c/geolocation#173 and discussion about existing implementations in w3c/geolocation#171.

* Update heading/index.md as well
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.

GeolocationCoordinates.toJSON() - Should the spec cover how to handle NaN headings?
2 participants