ft-interactive / starter-kit

A template for IG pages
65 stars 15 forks source link

Add fetch polyfill. #170

Closed notlee closed 6 years ago

notlee commented 6 years ago

In a previous PR I removed babel-polyfill to reduce bundle size and possible conflicts with polyfill.io. I also removed the features=default,fetch param from the polyfill.io request as I incorrectly thought all required polyfills for a browser are included by default, which they are not.

https://github.com/ft-interactive/starter-kit/pull/161#discussion-diff-163408639L62 https://polyfill.io/v2/docs/features/#default-sets

Somehow I managed to trick myself into thinking fetch was polyfilled with a Trump quote demo which it was not.

screen shot 2018-01-24 at 11 06 55

But it is now with these changes:

screen shot 2018-01-24 at 11 04 58