Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make ElGrapho UMD friendly for node+webpack environments #2

Merged
merged 4 commits into from
Apr 7, 2019
Merged

Make ElGrapho UMD friendly for node+webpack environments #2

merged 4 commits into from
Apr 7, 2019

Conversation

jakobo
Copy link
Contributor

@jakobo jakobo commented Mar 30, 2019

This is a couple of fixes to make ElGrapho UMD friendly, allowing it work in the browser (via window), and in node.js / webpack / babel (via UMD).

  • Concrete reference needs to resolve
  • No absently writing to the window object (let webpack take care of that)
  • Removed stray console.logs

The original webpack 3 signature for UMD was typeof self !== \'undefined\' ? self : this, but we can use the window object safely with a typeof guard in webpack 4.

* upstream/master: (37 commits)
  fix comment
  no double quotes
  removed webcola and added d3-force
  first cached result
  fix changelog
  models to layouts
  v2
  another readme change
  more changes
  readme change
  updates
  tweaks
  new FDG, and collapsing model
  stuff
  finished updating all of the examples with new API
  more grinding through new API work
  first pass big API change
  half done stuff
  more stuff
  more stuff
  ...
@ericdrowell ericdrowell merged commit 0b687b3 into ericdrowell:master Apr 7, 2019
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.

2 participants