polished icon indicating copy to clipboard operation
polished copied to clipboard

UMD of polished is not valid on the browser side

Open jmfrancois opened this issue 4 years ago • 1 comments

  • polished version: 3.6.0

What You Are Seeing

I see global variable for package that do not exists in UMD. It exists since 3.6.0 version. 3.5.2 is working well:

You can just look at the signature on the first 4 lines

(function (global, factory) {
  typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports, require('@babel/runtime/helpers/esm/extends'), require('@babel/runtime/helpers/esm/assertThisInitialized'), require('@babel/runtime/helpers/esm/inheritsLoose'), require('@babel/runtime/helpers/esm/wrapNativeSuper'), require('@babel/runtime/helpers/esm/taggedTemplateLiteralLoose')) :
  typeof define === 'function' && define.amd ? define(['exports', '@babel/runtime/helpers/esm/extends', '@babel/runtime/helpers/esm/assertThisInitialized', '@babel/runtime/helpers/esm/inheritsLoose', '@babel/runtime/helpers/esm/wrapNativeSuper', '@babel/runtime/helpers/esm/taggedTemplateLiteralLoose'], factory) :
  (global = global || self, factory(global.polished = {}, global.extends, global.assertThisInitialized, global.inheritsLoose, global.wrapNativeSuper, global.taggedTemplateLiteralLoose));
}(this, (function (exports, _extends, _assertThisInitialized, _inheritsLoose, _wrapNativeSuper, _taggedTemplateLiteralLoose) { 'use strict';

This seems to have been introduced by #7886d942b03ba452be4926a8d566ff67c4ac4161

What You Expected To See

I exepct to see only umd dependencies that exists in UMD.

(function (global, factory) {
  typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports) :
  typeof define === 'function' && define.amd ? define(['exports'], factory) :
  (global = global || self, factory(global.polished = {}));
}(this, (function (exports) { 'use strict';

And a chapter on the README explaining how to setup a script tag to use this umd build it in the browser.

jmfrancois avatar Feb 17 '21 10:02 jmfrancois

The latest version (4.1.1) tries to require babel-runtime helpers too. Would be awesome if those were just inlined within the UMD build. 🙏

https://unpkg.com/browse/[email protected]/dist/polished.js

It appears that 3.6.0 has the same issue.

ctrlplusb avatar Feb 23 '21 22:02 ctrlplusb