Skip to content

Commit

Permalink
Bumped DevTools Chrome and Firefox versions in Babel config (facebook…
Browse files Browse the repository at this point in the history
…#21185)

We have been building DevTools to target Chrome 49 and Firefox 54. These are super old browser versions and they did not have full ES6 support, so the generated build is more bloated than it needs to be.

DevTools uses most modern language features. Off the top of my head, we it uses basically everything but async and generator functions.

Based on CanIUse charts– I believe that in order to avoid unnecessary polyfill/wrapper code being generated, we'd need to target Chrome 60+ (released 2017-07-25) and Firefox 55+ (released 2017-04-18). This seems like a reasonable set of browsers to target.

Note that we can't remove the IE 11 target from the react-devtools-core backend yet due to Hermes (React Native) ES6 support but that should be doable by the end of the year given current engineering targets. But we could update the frontend target, as well as the targets for the extensions and the react-devtools-inline package.

This commit increases the browser targets then for Chrome (from 49 to 60) and Firefox (from 54 to 55)
  • Loading branch information
Brian Vaughn authored and acdlite committed Apr 11, 2021
1 parent bf8e1b7 commit 7c54262
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion packages/react-devtools-extensions/chrome/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"version": "4.10.1",
"version_name": "4.10.1",

"minimum_chrome_version": "49",
"minimum_chrome_version": "60",

"icons": {
"16": "icons/16-production.png",
Expand Down
2 changes: 1 addition & 1 deletion packages/react-devtools-extensions/edge/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
"version": "4.10.1",
"version_name": "4.10.1",

"minimum_chrome_version": "49",
"minimum_chrome_version": "60",

"icons": {
"16": "icons/16-production.png",
Expand Down
2 changes: 1 addition & 1 deletion packages/react-devtools-extensions/firefox/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
"applications": {
"gecko": {
"id": "@react-devtools",
"strict_min_version": "54.0"
"strict_min_version": "55.0"
}
},

Expand Down

0 comments on commit 7c54262

Please sign in to comment.