browser-compat-data icon indicating copy to clipboard operation
browser-compat-data copied to clipboard

Mirroring ranged versions removes ranges for most browsers

Open foolip opened this issue 1 year ago • 2 comments

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?

  • [X] The current BCD release
  • [X] The current version of the main branch

Do you have anything more you want to share?

cc @queengooborg

foolip avatar Jul 19 '24 08:07 foolip

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

foolip avatar Jul 19 '24 08:07 foolip

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

foolip avatar Jul 19 '24 08:07 foolip