^ I moved this to Needs Analysis as I'm uncertain of the priority. @bmansurov, @ovasileva: Over to you, I guess.

I'd say the priority is normal.

ResourceLoader doesn't re-minify ("filter" in RL parlance) the already minified asset

I added this as an AC because minifying more than once is advised against and, even if we're confident that the RL minifier won't alter the minified bundle, it's pretty trivial to get this working using webpack's BannerPlugin plugin:

module.exports = {
  /* ... */
  plugins: [
    /* ... */
    new webpack.BannerPlugin( {
      banner: '/*@nomin*/', // ResourceLoader::FILTER_NOMIN
      raw: true // Don't wrap the banner in a comment. It's easier to keep the banner in sync with ResourceLoader::FILTER_NOMIN this way.
    } )
  ]
}

@phuedx Awesome, much better! Link to the webpack v2 docs on webpack.js.org, webpack.github.io are the docs for v1

@Jhernandez: Updated my comment accordingly. Thanks 👊

Change 353136 had a related patch set uploaded (by Jhernandez; owner: Jhernandez):
[mediawiki/extensions/Popups@master] Hygiene: Make webpack.config.js conform to eslint

https://gerrit.wikimedia.org/r/353136

Change 353137 had a related patch set uploaded (by Jhernandez; owner: Jhernandez):
[mediawiki/extensions/Popups@master] Enable production settings for the production bundle

https://gerrit.wikimedia.org/r/353137

Change 353138 had a related patch set uploaded (by Jhernandez; owner: Jhernandez):
[mediawiki/extensions/Popups@master] Tools: Add pre-commit git hook to auto test and build

https://gerrit.wikimedia.org/r/353138

I'm missing a patch for the documentation on the repo and moving my on-wiki comparison to a page under Page previews, I'll get to that soon.

Change 353136 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Hygiene: Make webpack.config.js conform to eslint

https://gerrit.wikimedia.org/r/353136

Open patchsets:

• https://gerrit.wikimedia.org/r/#/c/353137
• https://gerrit.wikimedia.org/r/#/c/353138

Change 353137 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Enable production settings for the production bundle

https://gerrit.wikimedia.org/r/353137

Change 353138 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Tools: Add pre-commit git hook to auto test and build

https://gerrit.wikimedia.org/r/353138

To add ADR and wiki docs now.

Sorry, I'll leave it in QA now, we just crossed comments.

Change 353563 had a related patch set uploaded (by Jhernandez; owner: Jhernandez):
[mediawiki/extensions/Popups@master] Docs: Add ADR about using webpack's production mode and UglifyJS

https://gerrit.wikimedia.org/r/353563

Change 353563 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Docs: Add ADR about using webpack's production mode and UglifyJS

https://gerrit.wikimedia.org/r/353563

I did some testing on the beta site. Using browserstack, I ran some basic testcases for page previews. Everything worked normally. Here are the systems/browser that I tested.

Mac Sierra - Safari 10
Mac El Capitan - Firefox 53
Mac Sierra - Chrome 58
Mac Sierra - Opera 44
Mac Mountain Lion - Firefox 10

Windows 10 - Edge 15
Windows 10 - Chrome 58
Windows 8.1 - Firefox 53
Windows 8.1 - Opera 44
Windows 7 - Opera 10.6

@phuedx I can try some more systems or browsers or whatever. If there are any other tests I can run, let me know.

Thanks, @ABorbaWMF. I've also tested PP in IE10 and 11 on Windows 7 using BrowserStack so I think we're good 👍