Unminified versions of MediaElement.js and MediaElementPlayer not in core

[ericlewis]

Not sure how this fits into the new Grunt build process so I'll leave this to the big boys/girls.

[ericlewis]

Also mediaelementplayer.min.css

[ocean90]

We usually doesn't include the unminified versions of external libraries in core. Why should it be included?

[ericlewis]

Including unminified versions would allow developers to read the original JS/CSS source without leaving their development environment. I was looking for it in this case to copy, paste, and override media element's CSS. Also, If SCRIPT_DEBUG loaded them it would open up the opportunity to debug these packages if necessary.

[nacin]

We talked about a separate directory (unsure of what we'd call it) in the root of develop.svn that can have all of our external sources. I don't know what we'd call it, but I do know that we'd want script-loader to, if running a develop.svn checkout, automatically load all of these instead of the minified ones.

[kadamwhite]

+1 to that directory, that would be of notable interest to me. Re: the Grunt question, I would assume the build process wouldn't touch these files at all—We should be trusting 3rd-party libraries like Underscore and Backbone to already be providing the optimally minified versions of their files, so we just copy those over rather than running them through our own minification process.

[wonderboymusic]

In 4.2, I would like us to answer some JS questions like this

[ericlewis]

We're not too far off from where we currently are.

WordPress-specific source JS files (i.e. un-minified versions) can be loaded by toggling the SCRIPT_DEBUG constant ^​1.

Some third-party scripts (e.g. Backbone.js, all jQuery UI modules) will only load the unminified version if loaded from the src directory^​2. Otherwise, the minified version of the script is loaded.

I propose we bring in the source and unminified versions for all third-party modules into the src/ directory, and copy both in the build process. After that, modify the script loader to respect SCRIPT_DEBUG to the tee.

[DrewAPicture]

I'm not sure this ticket strictly belongs on a milestone since we're talking about adding external sources to a directory in the root develop.svn folder – in other words, has nothing to do with a release :-)

Moving to Future Release for triage purposes.

[swissspidy]

Let's focus on adding unminified scripts to src in #32358