This project is archived and is in readonly mode.
Minify packaged javascript libraries
-
Nathaniel Bibler
- Tag changed from javascript, performance, prototype, scriptaculous to javascript, patch, performance, prototype, scriptaculous
Here is a sample patch with the libraries minified using the default YUI Compressor settings, but retaining the file comment headers.
-
Jacek Becela
On the other side:
"Minifying" is not the right solution for bandwidth issues. In my opinion it is responsibility of server that is hosting the app to compress its output.
Also the reduction of 40% takes place only in these javascripts, so overall reduction (including html/css/images/etc.) is far worse.
I'd like to be able to just read this code. It saves me sometimes.
-
Nathaniel Bibler
Fair enough. I was chatting with someone in
#rails-contriblast night who had the same thoughts. I guess I just rely more heavily on the API documentation than reading the actual source and assume that the packaged libraries are working and bug-free. :)I guess the better solution would be to encourage people to tie in the YUI Compressor (or some other solution) with a capistrano deployment or continue to push the Google AJAX API.
Thanks for the feedback on it.
-
Jacek Becela
Ability to read code is in no way excuse for not having solid API docs, I prefer that too ;)
-
DHH
- State changed from new to wontfix
Most people use gzip as well, which pretty much nullifies the benefits of minifying.
-
Nathaniel Bibler
True. Thanks for the consideration and feedback, guys. :)
-
Jeff Kreeftmeijer
Automatic cleanup of spam.
