Remove Safari 6.1 #11156

queengooborg · 2021-06-19T14:06:26Z

This PR removes Safari 6.1 from the data and converts all of its entries to Safari 7. Details are described in #9423, but to summarize, it seems that Safari 6.1 and 6.2 are simply backports of Safari 7.0 and 7.1 for older macOS versions, similar to Safari 4.1 and 5.0.

foolip · 2021-06-19T14:50:01Z

I looked into making this changes yesterday on a branch, which I've created as #11157 now. That has a bunch of different commits, and only "Change where iOS is 7" is trivial IMHO. I'd especially want to look into the cases where iOS said 6.1, since that has the same WebKit version as iOS 6. I don't really believe that data, but if we did we'd change iOS 6.1 to 6, not 7. So for those we need to research what it should really be.

This subset of mdn#11156 does not need much scrutinty, as this results in the expected matching of Safari and iOS versions. There may still be errors in this data, but then it's wrong for both Safaris. Part of mdn#9423.

This subset of mdn#11156 does not need much scrutiny, as this results in the expected matching of Safari and iOS versions. There may still be errors in this data, but then it's wrong for both Safaris. Part of mdn#9423.

This subset of #11156 does not need much scrutiny, as this results in the expected matching of Safari and iOS versions. There may still be errors in this data, but then it's wrong for both Safaris. Part of #9423.

foolip · 2021-06-22T13:59:00Z

api/Window.json

@@ -1670,10 +1670,10 @@
              "version_added": "24"
            },
            "safari": {
-              "version_added": "6.1"
+              "version_added": "7"


Using http://mdn-bcd-collector.appspot.com/tests/api/Window/crypto it looks like window.crypto was actually added in Safari 5.1. Maybe the collector can be used to update a bunch of this data?

queengooborg · 2021-07-16T04:42:00Z

I'm marking this as "not ready" as I'm planning to talk to Daniel about what we want to do with Safari 6.1.

queengooborg · 2021-08-22T06:04:46Z

Removing the "not ready" flag -- we've just about come to a conclusion to remove Safari 6.1!

foolip · 2021-08-23T09:18:46Z

@vinyldarkscratch was most of this produced by a script so that you can easily recreate it? If so, could you separate it into multiple commits and PRs to make it easier to review? In particular, all the cases where 6.1+6.1 are changed to 7+7 are ones I don't feel need any review at all. Splitting just that out would probably make the rest small enough to review together.

queengooborg · 2021-08-23T10:23:14Z

It was generated by a quick grep and sed replacement, but I could instead run the collector on these and get accurate results? I can split that into separate PRs for easier review too.

foolip · 2021-08-23T10:36:09Z

A separate PR any APIs that the collector determines were supported before 6.1 would be great.

queengooborg · 2021-08-23T11:39:22Z

queengooborg · 2021-12-02T13:00:44Z

Current remaining entries are:

→ css.properties.align-self.flex_context
→ css.properties.clip-path
→ css.properties.clip-path.basic_shape
→ css.properties.clip-path.html
→ css.properties.clip-path.svg
→ css.properties.display.inline-flex
→ css.properties.flex-wrap
→ css.properties.font-feature-settings
→ css.properties.justify-content.flex_context
→ css.properties.min-height.fit-content
→ css.properties.position.sticky
→ css.properties.ruby-position
→ css.properties.width.animatable
→ css.properties.width.stretch
→ css.properties.word-spacing.percentages
→ css.selectors.cue
→ css.selectors.cue.selector_argument
→ css.selectors.future
→ css.selectors.past
→ css.types.image.gradient.linear-gradient
→ css.types.image.gradient.linear-gradient.interpolation_hints
→ css.types.image.gradient.linear-gradient.to
→ css.types.image.gradient.linear-gradient.unitless_0_angle
→ css.types.image.gradient.radial-gradient
→ css.types.image.gradient.radial-gradient.interpolation_hints
→ css.types.image.gradient.repeating-linear-gradient
→ css.types.image.gradient.repeating-linear-gradient.interpolation_hints
→ css.types.image.gradient.repeating-linear-gradient.to
→ css.types.image.gradient.repeating-linear-gradient.unitless_0_angle
→ css.types.length-percentage.vmax
→ css.types.length-percentage.vmin
→ css.types.length.vmax
→ css.types.length.vmin
→ html.elements.details
→ html.elements.details.open
→ javascript.builtins.Array.toLocaleString.locales_parameter
→ javascript.builtins.Array.toLocaleString.options_parameter
→ svg.elements.use.external_uri

queengooborg · 2021-12-04T22:38:25Z

The remaining features are all covered by #13867, #13868, #13869, #13870.

foolip · 2021-12-06T11:17:30Z

@queengooborg those PRs are merged, this now need rebasing.

queengooborg · 2021-12-06T14:09:34Z

Oops, one more: #13888!

foolip · 2021-12-06T14:18:58Z

Also merged!

queengooborg · 2021-12-06T14:24:26Z

Alright, this is now ready to go! Whoo!

* Bump version to v4.1.1 * Add initial release notes * Add release note for #11156 * Add stats * Fix previous release note version

github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jun 19, 2021

foolip mentioned this pull request Jun 19, 2021

Remove Safari 6.1 #11157

Closed

foolip mentioned this pull request Jun 21, 2021

Remove iOS releases with matching engine versions? #10723

Closed

4 tasks

foolip added the needs-release-note 📰 label Jun 21, 2021

foolip mentioned this pull request Jun 21, 2021

Change Safari 6.1 to 7 where the iOS version is already 7 #11184

Merged

github-actions bot added data:browsers 🌍 Data about browsers (versions, release dates, etc). This data is used for validation. data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS labels Jun 22, 2021

foolip reviewed Jun 22, 2021

View reviewed changes

queengooborg added the not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. label Jul 16, 2021

queengooborg removed the not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. label Aug 22, 2021

queengooborg added the not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. label Aug 23, 2021

Remove Safari (and Safari iOS) 6.1 from BCD

6593ec9

queengooborg force-pushed the browsers/remove-safari-6.1 branch from 79f2511 to 6593ec9 Compare August 24, 2021 10:21

queengooborg removed data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API data:css 🎨 Compat data for CSS features. https://developer.mozilla.org/docs/Web/CSS labels Aug 24, 2021

Merge branch 'main' into browsers/remove-safari-6.1

fe5ab20

queengooborg added not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. and removed not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. labels Oct 9, 2021

Merge branch 'main' into browsers/remove-safari-6.1

f01129e

queengooborg added rebase needed 🚧 infra 🏗️ Infrastructure issues (npm, GitHub Actions, releases) of this project and removed not ready ⛔ This is not yet ready to be merged. It's pending a decision, other PR, or a prerequisite action. labels Dec 6, 2021

Merge branch 'main' into browsers/remove-safari-6.1

0892e52

queengooborg removed the rebase needed 🚧 label Dec 6, 2021

Merge branch 'main' into browsers/remove-safari-6.1

23f4fa2

foolip approved these changes Dec 6, 2021

View reviewed changes

foolip merged commit 5a1494b into mdn:main Dec 6, 2021

queengooborg deleted the browsers/remove-safari-6.1 branch December 6, 2021 15:29

foolip mentioned this pull request Dec 6, 2021

Should we treat Safari 6.2 as 6.1 or a later version? #9423

Closed

ddbeck removed the needs-release-note 📰 label Dec 7, 2021

ddbeck added a commit to ddbeck/browser-compat-data that referenced this pull request Dec 7, 2021

Add release note for mdn#11156

722da8a

ddbeck added a commit that referenced this pull request Dec 7, 2021

Release v4.1.1 (#13906)

d2a7d13

* Bump version to v4.1.1 * Add initial release notes * Add release note for #11156 * Add stats * Fix previous release note version

jpmedley pushed a commit that referenced this pull request Jan 10, 2022

Remove Safari (and Safari iOS) 6.1 from BCD (#11156)

8c6a1b7

jpmedley pushed a commit that referenced this pull request Jan 10, 2022

Release v4.1.1 (#13906)

4d3a349

* Bump version to v4.1.1 * Add initial release notes * Add release note for #11156 * Add stats * Fix previous release note version

ddbeck mentioned this pull request Oct 30, 2023

Explicitly order browser releases #21124

Open

8 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Remove Safari 6.1 #11156

Remove Safari 6.1 #11156

queengooborg commented Jun 19, 2021

foolip commented Jun 19, 2021

foolip Jun 22, 2021

queengooborg commented Jul 16, 2021

queengooborg commented Aug 22, 2021

foolip commented Aug 23, 2021

queengooborg commented Aug 23, 2021

foolip commented Aug 23, 2021

queengooborg commented Aug 23, 2021 •

edited

Loading

queengooborg commented Dec 2, 2021

queengooborg commented Dec 4, 2021

foolip commented Dec 6, 2021

queengooborg commented Dec 6, 2021

foolip commented Dec 6, 2021

queengooborg commented Dec 6, 2021

Remove Safari 6.1 #11156

Remove Safari 6.1 #11156

Conversation

queengooborg commented Jun 19, 2021

foolip commented Jun 19, 2021

foolip Jun 22, 2021

Choose a reason for hiding this comment

queengooborg commented Jul 16, 2021

queengooborg commented Aug 22, 2021

foolip commented Aug 23, 2021

queengooborg commented Aug 23, 2021

foolip commented Aug 23, 2021

queengooborg commented Aug 23, 2021 • edited Loading

queengooborg commented Dec 2, 2021

queengooborg commented Dec 4, 2021

foolip commented Dec 6, 2021

queengooborg commented Dec 6, 2021

foolip commented Dec 6, 2021

queengooborg commented Dec 6, 2021

queengooborg commented Aug 23, 2021 •

edited

Loading