Storybook, Webpack, and Polyfills #1452

coldlink · 2022-02-04T14:28:01Z

What does this change?

Remove corejs in favour of using polyfill.io which only polyfills things if that browser needs the pollyfill
- This reduces the bundle size we send to the client
Fix storybook IE11 by using Errors in IE11 Storybook 6.4.9 storybookjs/storybook#17057 (comment)
Use webpack5 loader for storybook: https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#webpack-5

Co-authored-by: Oliver Barnwell <ob6160@users.noreply.github.com>

Fixes IE11 issues using storybookjs/storybook#17057 Uses webpack5 loader for storybook https://github.com/storybookjs/storybook/blob/next/MIGRATION.md#webpack-5 Updates dev deps Uses polyfill.io for storybook too Co-authored-by: Oliver Barnwell <ob6160@users.noreply.github.com> feat: remove corejs

feat: use polyfill.io instead of corejs

7a27d57

Co-authored-by: Oliver Barnwell <ob6160@users.noreply.github.com>

coldlink changed the title ~~Mm/use polyfill~~ Storybook, Webpack, and Polyfills Feb 4, 2022

coldlink marked this pull request as ready for review February 4, 2022 14:32

coldlink requested a review from a team as a code owner February 4, 2022 14:32

liamness approved these changes Feb 4, 2022

View reviewed changes

coldlink force-pushed the mm/use-polyfill branch from 5c220ea to aa196ae Compare February 4, 2022 15:04

coldlink requested a review from ob6160 February 4, 2022 15:04

coldlink merged commit 3f66dbb into main Feb 4, 2022

coldlink deleted the mm/use-polyfill branch February 4, 2022 15:12

coldlink mentioned this pull request Feb 7, 2022

Add globalThis polyfill to fix legacy browser regression #1461

Merged