mdn / browser-compat-data

This repository contains compatibility data for Web technologies as displayed on MDN
https://developer.mozilla.org
Creative Commons Zero v1.0 Universal
4.9k stars 1.97k forks source link

Mirroring ranged versions removes ranges for most browsers #23855

Open foolip opened 1 month ago

foolip commented 1 month ago

What type of issue is this?

Infrastructure issue

What is the issue?

Both Chrome and Firefox use ranges in this feature:

https://github.com/mdn/browser-compat-data/blob/9832a6516bfe3d8b1d18e503645d0332267c43a0/html/elements/iframe.json#L767-L800

The data we publish (after mirroring) is this:

          "allow-forms": {
            "__compat": {
              "description": "<code>sandbox=\"allow-forms\"</code>",
              "source_file": "html/elements/iframe.json",
              "spec_url": "https://html.spec.whatwg.org/multipage/browsers.html#attr-iframe-sandbox-allow-forms",
              "status": {
                "deprecated": false,
                "experimental": false,
                "standard_track": true
              },
              "support": {
                "chrome": {
                  "version_added": "≤49"
                },
                "chrome_android": {
                  "version_added": "49"
                },
                "edge": {
                  "version_added": "79"
                },
                "firefox": {
                  "version_added": "≤49"
                },
                "firefox_android": {
                  "version_added": "49"
                },
                "ie": {
                  "version_added": false
                },
                "oculus": {
                  "version_added": "5.0"
                },
                "opera": {
                  "version_added": "36"
                },
                "opera_android": {
                  "version_added": "36"
                },
                "safari": {
                  "version_added": "≤10.1"
                },
                "safari_ios": {
                  "version_added": "≤10.3"
                },
                "samsunginternet_android": {
                  "version_added": "5.0"
                },
                "webview_android": {
                  "version_added": "49"
                }
              }
            }
          },

Note that there are no ranges for any mirrored data except safari_ios, which is done correctly.

What behavior were you expecting?

Ranges should be preserved for browsers that mirror from Chrome and Firefox.

What version(s) of BCD is the issue present in?

Do you have anything more you want to share?

cc @queengooborg

foolip commented 1 month ago

@ddbeck @Elchi3 FYI, I discovered this when looking at web-features dist files, surprised to see ranges for Chrome but not Chrome Android.

foolip commented 1 month ago

I think I've found the bug and I'm writing some tests for this.