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

Issue with "Navigator.platform": premature recommendation to move to NavigatorUAData.platform #11934

Closed
alexmiddeleer opened this issue Jan 11, 2022 · 1 comment · Fixed by #12017
Assignees
Labels
Content:WebAPI Web API docs

Comments

@alexmiddeleer
Copy link

alexmiddeleer commented Jan 11, 2022

MDN URL: https://developer.mozilla.org/en-US/docs/Web/API/Navigator/platform

What information was incorrect, unhelpful, or incomplete?

It's unhelpful to recommend NavigatorUAData.platform when it is not supported by several major browsers yet. At least without a caveat or suggested polyfill.

Specific section or headline?

Note: The recommended alternative to this property is NavigatorUAData.platform, which is returned from Navigator.userAgentData.

What did you expect to see?

The following would be better. Best would be some sort of polyfill or other recommendation for adopting this new approach.

Note: The recommended alternative to this property is NavigatorUAData.platform, which is returned from Navigator.userAgentData. However, this property is not yet supported by some major browsers, and is still in the draft phase.

Did you test this? If so, how?

These two pages support my statement that the recommended alternative is still a draft and not yet supported by some major browsers.
I looked at https://wicg.github.io/ua-client-hints/#dom-navigatoruadata-platform and https://developer.mozilla.org/en-US/docs/Web/API/NavigatorUAData/platform

MDN Content page report details
@alexmiddeleer alexmiddeleer changed the title Issue with "Navigator.platform": (short summary here please) Issue with "Navigator.platform": premature recommendation to move to NavigatorUAData.platform Jan 11, 2022
@github-actions github-actions bot added the needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. label Jan 11, 2022
@sideshowbarker sideshowbarker added Content:WebAPI Web API docs help wanted If you know something about this topic, we would love your help! effort: small This task is a small effort. and removed needs triage Triage needed by staff and/or partners. Automatically applied when an issue is opened. labels Jan 14, 2022
@sideshowbarker sideshowbarker self-assigned this Jan 15, 2022
@sideshowbarker sideshowbarker removed help wanted If you know something about this topic, we would love your help! effort: small This task is a small effort. labels Jan 15, 2022
@sideshowbarker
Copy link
Collaborator

Thanks for reporting this — I’ve opened #12017 with a fix.

After that fix is merged, this issue will automatically be marked as resolved, and closed.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Content:WebAPI Web API docs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
@sideshowbarker @alexmiddeleer and others