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

Fix: use navigator.userAgentData.platform if available #1483

Closed
wants to merge 1 commit into from
Closed

Fix: use navigator.userAgentData.platform if available #1483

wants to merge 1 commit into from

Conversation

bershanskiy
Copy link

Avoid calling navigator.platform on Chromium/Chrome where navigator.userAgentData.platform is available. This removes the annoying warning in Chromium "Issues" console which reads:

Audit usage of navigator.userAgent, navigator.appVersion, and navigator.platform

This PR does not make any real changes to functionality.

@shortcuts
Copy link
Member

shortcuts commented Sep 9, 2022

Hey, I was not able to add tests to your PR so created a new one here: #1548

Thanks a lot for the PR and the proposal!

@shortcuts shortcuts closed this Sep 9, 2022
@bershanskiy bershanskiy deleted the navigator.userAgentData branch September 9, 2022 18:00
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.

2 participants