Import babel polyfill #813

Merged
merged 3 commits into from Dec 27, 2016

Projects

None yet

2 participants

@danvk
Owner
danvk commented Dec 27, 2016 edited

Fixes #790

See http://stackoverflow.com/questions/33828840/symbol-is-undefined-in-ie-after-using-babel

This increases the bundle size substantially (by ~14%). The reason is that I've started using some ES6 features (specifically for .. of). This means that older browsers require a Symbol and iterator polyfill. These are 20k.

An alternative would be to not include the polyfill and require users to load it themselves if they care about IE11.

@coveralls

Coverage Status

Coverage increased (+0.02%) to 90.136% when pulling 7b6d545 on babel-polyfill into 87f78fb on master.

added some commits Dec 27, 2016
@danvk Import babel polyfill
5218877
@danvk bump node version
f2776dc
@danvk promise polyfill
da63e12
@coveralls

Coverage Status

Coverage increased (+0.02%) to 90.136% when pulling da63e12 on babel-polyfill into 87f78fb on master.

@danvk danvk merged commit 627e054 into master Dec 27, 2016

5 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.02%) to 90.136%
Details
dist/dygraph-combined.js +19,955 bytes (+14.02%) → 142,351 bytes
dist/dygraph.min.js.gz +6,338 bytes (+14.81%) → 42,789 bytes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment