Skip to content

Babel @kurkle/color for IE support#7099

Merged
etimberg merged 3 commits intochartjs:masterfrom
kurkle:babel
Feb 14, 2020
Merged

Babel @kurkle/color for IE support#7099
etimberg merged 3 commits intochartjs:masterfrom
kurkle:babel

Conversation

@kurkle
Copy link
Member

@kurkle kurkle commented Feb 14, 2020

  • Make Babel transpile @kurkle/color
  • Add @babel/plugin-transform-object-assign to polyfill the Object.assing in there
  • Remove helpers.extend and use Object.assing instead (to not have duplicate polyfills)
  • Build the UMD first (to reduce wait time when testing with browser by couple of seconds)

etimberg
etimberg previously approved these changes Feb 14, 2020
Copy link
Contributor

@benmccann benmccann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. just a few minor comments

@etimberg etimberg merged commit 7c1fb37 into chartjs:master Feb 14, 2020
@kurkle kurkle deleted the babel branch February 19, 2020 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants