Remove Safari iOS 5.1 #11175
Merged
Remove Safari iOS 5.1 #11175
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
data:api
4 tasks
|
I've found some additional sources of the UA strings: https://stackoverflow.com/a/7825888 says:
https://pypi.org/project/user-agents-next/ says, in an example:
So it looks like the WebKit version is 534.46 for both and the corresponding Safari version is 5.1 for both. |
foolip
reviewed
Jun 21, 2021
foolip
added a commit
to foolip/browser-compat-data
that referenced
this pull request
Jun 29, 2021
Based on UA strings: mdn#11175 (comment)
|
@vinyldarkscratch are you planning to go through the comments to finish this? |
|
I am, yeah -- I found out that BrowserStack has an iPad on iOS 4.3 emulator, so I can collect the results for iOS 4.2! I'll be tackling that soon. |
foolip
reviewed
Jul 16, 2021
foolip
approved these changes
Jul 16, 2021
ddbeck
added a commit
to ddbeck/browser-compat-data
that referenced
this pull request
Jul 22, 2021
ddbeck
added a commit
to ddbeck/browser-compat-data
that referenced
this pull request
Jul 22, 2021
ddbeck
added a commit
that referenced
this pull request
Jul 22, 2021
* Bump version to v3.3.12 * Add release note for #11661 * Add release notes for #11173 and #11175 * Add release note for #11534 * Add release note for #11544 * Add release note for #11551 * Add release note for #11555 * Add release note for #11556 * Add release note for #11557 * Add release note for #11633 * Add release note for #11636 * Add release note for #11637 * Add release note for #11530 * Add release stats and date * Consolidate Safari for iOS version consolidation notes * Format PR URL
foolip
added a commit
to foolip/browser-compat-data
that referenced
this pull request
Jan 3, 2022
Based on UA strings: mdn#11175 (comment)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR removes Safari iOS 5.1 from the browser data, and changes everything set to Safari iOS 5.1 to mirrored data from Safari Desktop. Safari iOS 5.1's WebKit version is the exact same as iOS 5.0's, as confirmed by querying WhatIsMyBrowser.
Using the Safari iOS 4 emulator in BrowserStack, I've spot-checked a few entries to ensure this methodology is correct. I have also found that all of the data for Safari iOS 351 came from the initial wiki tables, and most entries are accompanied by Safari Desktop set to 5.1, which further leads me to believe that Safari iOS 5.1 was mistakenly added.
An additional note is that worker support was indeed pushed back to a later release in Safari iOS than Safari Desktop, probably due to OS implementation or the likes. I've set its version to 5.0.