@TrevorBurnham there's anything we could do to unblock this one? Without it, we are prevented to use rollup: rollup/rollup-plugin-babel#87
In time, I don't think it's rollup job to fix the issue, but here on the dist bundle.

Thanks

Hi,
it would be amazing if this would finally work....

This should absolutely be merged @HubSpot @TrevorBurnham.

I have forked the branch by @kt3k and included a new build on it. As a temporary solution you can refer to Tether 1.3.4 browserified build like this in your package.json:

{
  "dependencies": {
    "tether": "balloob/tether#7e0d48e4ac2162e251dc2b69d7c13e6c29a5667f"
  }
}

close this in favor of #359

@kt3k That's a different project. Issue still exists in this project. Please reopen. There are multiple forks being maintained with your changes until @HubSpot merges this.

Yes this should be reopened and merged, fork is being used by reactstrap to work around this but would be best to correct here

Oops, thanks for pointing it.

By the way, now I think rollup is better choice for bundling this project because it produces much simpler and smaller bundle than browserify.

I'm trying to use 'reactstrap' and the issue caused by 'reactstrap-tether' brought me here.

It seems this PR will create a UMD which cannot be consumed by 'browserify'.
The repro is simple. Use an empty project with 'reactstrap-tether':

{
  "dependencies": {  
    "reactstrap-tether": "^1.3.4",  
    "browserify": "^13.0.0"  
  },  
  "name": "bad-require",
  "private": true,
  "scripts": {
    "start": "browserify -d -e test.js",
  },
}

The test.js is one line:

var tether = require('reactstrap-tether');

Run

npm run start

will generate errors saying

Error: Cannot find module './utils' from .....

The webpack might work with this properly. But why break browserify (or did i miss anything) ?

bump

@zekedroid see https://github.com/HubSpot/tether/blob/master/README.md#rotating_light-project-status-rotating_light

Or use a fork

Thanks @awerlang! I think I'll try to fix it here first because the change is needed for reactstrap-tether which is needed for reacstrap. It would be simplest to make the patch. Unless there's already a fork that is good enough.

Closing for inactivity.