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

Consider using the browser version, instead of the OS version, for iOS Safari #6338

Closed
othermaciej opened this issue Jun 26, 2020 · 4 comments
Labels
data:browsers 🌍 Data about browsers (versions, release dates, etc). This data is used for validation. idle 🐌 Issues and pull requests with no recent activity

Comments

@othermaciej
Copy link

The browser compat data seems to use the OS version number as the iOS Safari version. This is not the same as for any other desktop or mobile browser, where the browser version is used instead.

This also has the potential to confuse, because the same number can be used at different times as the OS version and the browser version. The Safari browser version does not align with the OS, it aligns with Safari for other platforms. For example, iOS 13 shipped with Safari 13, but iOS 13.4 shipped with Safari 13.1.

If this is not changed, at least the column header or entries should make clear that this is the OS version, not the browser version. Both are present in the Safari User-Agent string.

For example, here's the UA string for Safari on iOS 13.4.1. Note Version/13.1. That's the browser version.

Mozilla/5.0 (iPhone; CPU iPhone OS 13_4_1 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.1 Mobile/15E148 Safari/604.1

@queengooborg queengooborg added data:browsers 🌍 Data about browsers (versions, release dates, etc). This data is used for validation. duplicate 👯‍♀️ Duplicate issues or pull requests. This one is closed in favor of the other issue or pull request. invalid 🚫 Invalid issues or pull requests (wrong repo, spam, duplicates, etc.). This won't get merged. Sorry! labels Jun 26, 2020
@queengooborg
Copy link
Collaborator

We use the iOS and iPadOS version for Safari for iOS/iPadOS per direct request of Apple; see #2006 (comment). Seeing as this discussion has come up a few times, however, something to help clarify on the MDN web docs would probably be helpful -- we're certainly open to suggestions!

Issues for the layout and UI on MDN web docs are in the kuma repository, if you'd like to open an issue there: https://github.com/mdn/kuma/issues/new?assignees=&labels=status%3A+needs+triage&template=change-request.md&title=

(This is a duplicate of #5776, so I'm going to close this.)

@othermaciej
Copy link
Author

I'm syncing up with @apple-web-evangelist offline.

@queengooborg
Copy link
Collaborator

I am now just realizing that you work at Apple, specifically on WebKit -- oops, my bad! 😅

I'll go ahead and reopen this issue then. Yeah, let us know what you would like us to do going forward!

@queengooborg queengooborg reopened this Jun 26, 2020
@queengooborg queengooborg removed duplicate 👯‍♀️ Duplicate issues or pull requests. This one is closed in favor of the other issue or pull request. invalid 🚫 Invalid issues or pull requests (wrong repo, spam, duplicates, etc.). This won't get merged. Sorry! labels Jun 26, 2020
@othermaciej
Copy link
Author

othermaciej commented Jun 26, 2020

Well, to be fair, I didn't say it in my original issue. We're going to sync internally on the best way to represent iOS Safari versions.

@github-actions github-actions bot added the idle 🐌 Issues and pull requests with no recent activity label May 25, 2022
@Elchi3 Elchi3 closed this as completed Aug 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:browsers 🌍 Data about browsers (versions, release dates, etc). This data is used for validation. idle 🐌 Issues and pull requests with no recent activity
Projects
None yet
Development

No branches or pull requests

3 participants