Resources should be minified

[Malvoz]

Google's Lighthouse reports a potential saving of 148.6 kb in transfer size (not including CSS):

We should:

• Uglify all JS files
• Minify and concatenate CSS files (fixed in Split keyboardInteraction.test.js into three parts, with one describe… #606)

Of course, license text should be preserved.

[ahmadayubi]

A minified version of mapml is provided in the build branch and in releases, but having a minified version of the other javascript files would be good too.

[Malvoz]

A minified version of mapml is provided in the build branch and in releases

@ahmadayubi Is mapml.min.js used anywhere?

Would it be acceptable to simply uglify/minify everything in the dist folder without outputting .min.(js|css) file names?

[ahmadayubi]

It's currently not used anywhere as the tests and experiments files people usually debug on so it's important they can read it (without having to pretty print in chrome debugger), you could definitely minify everything but it would add an extra step to debugging.

It wouldn't be a big deal though, it's mostly a preference thing I believe. @prushforth has developed the minification process through GruntJS, maybe he could shed light on his decision to offer both .min.js and .js

I'm not sure what the mapml.min.js file is for currently, it could be used in the mapml-viewer.min.js file if that's ever added in the future.

[Malvoz]

you could definitely minify everything but it would add an extra step to debugging.

Unless we generate source maps using grunt-contrib-uglify's sourceMap option?

[ahmadayubi]

That would definitely work (after setting up and testing to make sure it works as expected), I hadn't considered using source maps in the past.

[prushforth]

Addressed by #705