Updated File which can cause issues after minification.

[mahad-apex]

I used this file in my project and I was minifying all my JS (by the handler, generically) but this file used "default" keyword which is reserved keyword. So, in YUI JS minifier due to 'default' reserved keyword, it was breaking. In code, there are several places where .default is used. these are examples: function () { return t.default } function () { lt = document.body, re() }, !0); e.default = ee

[anseki]

Hi @mahad-apex, thank you for your PR. Sorry, I couldn't understand this PR. Why you replace the minified file? That is, the replacing is meaningless because the file is minified by building. And your file seems too big wastefully. Also, the default is already reserved keyword by ECMAScript, and the code was added by Webpack. What is problem in YUI JS minifier?

[mahad-apex]

YUI JS minifier actually breaks when 'new' and 'default' used as variable. So in my case i changed 2 stances of .default to ["default"] in my PR.

[anseki]

• Do you mean that the new and default were used as variable (not property) in the file? Could you indicate where are those?
• Why did you try to re-minify the file that was already minified?

[mahad-apex]

Dear Anseki, your pluggin is great and we used in our application. Find these lines in your minified version.

function () { return t.default }
function () { lt = document.body, re() }, !0); e.default = ee

The .default written in these lines can cause issues in minification.

Your point is absolutely right, there is no further need of minification of minified file, you can close PR! In my case, there is a handler in C# we read generically all Js files either third party plugins or our functional JS and minify it once before dumping in page via handler, we dont use Githubissues.

Githubissues is a development platform for aggregating issues.