Should compress javascripts to save space

[alexkingorg]

Should compress javascripts to save space

[markjaquith]

Paging Andy Skelton:

Would this make Prototype upgrades more difficult? Any objections?

[andy]

As long as the only changes from the original script are whitespace, I wouldn't worry about code maintenance issues here.

[matt]

We never make any local mods or fork our JS libraries outside of our TinyMCE plugins.

I know the Dojo Toolkit folks and Alex Russell have a really kickass JS compression thing based on Rhino, perhaps we should run Prototype, DBX, and others through that to cut down on load times?

• ​http://dojotoolkit.org/docs/compressor_system.html

Online tools:

• ​http://www.dev411.com/dojo/javascript_compressor/
• ​http://alex.dojotoolkit.org/shrinksafe/

Someone should test to see which works best and we'll use it.

[masquerade]

Another library I've been using lately gets great compression by running their downloads through this class, and hasn't had any errors as some other compressors have been known to have.

​http://www.phpclasses.org/browse/package/3158.html

[matt]

Addressed in [4566].

[foolswisdom]

#3414 custom image header cropper error "_163 has no properties" in prototype.compress.js

[alexkingorg]

This version gets it down to 25k:

​http://www.stevekallestad.com/downloads/proto_150rc1_compressed.js

blog post here:

​http://www.stevekallestad.com/blog/prototype_and_scriptaculous_compressed.html

Worth looking into?

[alexkingorg]

Also, I think the copyright info should be added back to the compressed versions.

[shorty114]

Replying to alexkingorg:

Also, I think the copyright info should be added back to the compressed versions.

Yes, if anything keep the header intact.

[Nazgul]

Because of the problems associated with this ticket, I suggest to push it out of scope for 2.1 and take a new look at it for the next release.

If we do push it out of scope I think we should take out prototype.compressed.js for now as well.

[foolswisdom]

Marking FIXED for 2.1 .

Replying to Nazgul:

Because of the problems associated with this ticket, I suggest to push it out of scope for 2.1 and take a new look at it for the next release.

That would upset me because this ticket has already resulted in changes. Better to have a new ticket for the next cycle of work.

If we do push it out of scope I think we should take out prototype.compressed.js for now as well.

prototype.compressed.js was already removed in [4592]

Replying to alexkingorg:

Also, I think the copyright info should be added back to the compressed versions.

I disagree. The original "source" version is in the same folder and the file naming is appropriate. Source is easy to locate in the distribution and on the net. I don't think authors will find this offensive.

Replying to alexkingorg:

This version gets it down to 25k:

Awesome! Something to explore for 2.2 . ​Will other libraries still be being considered?

[foolswisdom]

[4769] [4770] removed all the compression files. First removed quicktags.compressed.js because people have customized quicktags.js , and then all (it seems) because there are enough other big variables in this release.

[pishmishy]

Feature request so bumping to 2.6 for feature freeze.

[azaozz]

No traction since 2.1, also it seems it's better to capture all js files with Gears instead of compressing them.