Crisp's blog

Gilles: nee, maar CSS minifyen is redelijk simpel en kan met een paar reguliere expressies. Misschien wijdt ik daar nog wel een keer een blogpost aan

Probleem (in mijn ogen) bij het minifyen van CSS is dat je relatieve paden naar images goed moet herschrijven.

Omdat wij in CSS enkel relatieve paden gebruiken is dat simpelweg:

PHP:

1	<?php $css = preg_replace('/url\(\s*([^\s)]*)\s*\)/', 'url(../$1)', $css); ?>

Waar ik wel nieuwsgierig naar ben is hoe jullie dan jullie JS minifyen.

Wij minifyen naar een 'min'-subdir op het moment dat wij een checkout doen op de acceptatie-omgeving. Wij doen dat trouwens heel simpel door te kijken naar de modify-timestamp van alle *.js en *.css files en die te vergelijken met de timestamp van de minified versie.

Ik vraag me alleen af of dat nog voordelen bied..

Ik betwijfel het. Sowieso als je je HTML toch al met HTTP compressie verstuurd zal de serverside parsetime voor het minifyen denk ik niet meer opwegen tegen de paar bytes die je er mogelijk nog mee wint. Het voordeel van statische files minifyen is natuurlijk dat je die gewoon op je filesystem kan cachen zodat je dat maar 1 keer hoeft te doen.

Je kan beter kijken of je je javascript toch niet statisch kan maken, bijvoorbeeld door slimmer gebruik te maken van bepaalde selectors om events te koppelen

Kiphaas7: Da's een niet gestripte BOM (Byte Order Mark), dus dat probleem wordt veroorzaakt door het samenvoegen van de bestanden.

JSMin doet itt JSMin+ geen echte parsing en dus ook geen validatie. Volgens mij zou dit in een browser ook een JS-error opleveren. Ik denk dat je dit dus beter als bug bij Drupal kan melden.

[Comment edited on Sunday 02 August 2009 20:44]

Mja, ik ken Drupal niet wat dat betreft. Ik ben wel gevlijd door het feit dat ze JSMin+ zijn gaan gebruiken (net als TYPO3 overigens, en ook het minify project wil JSMin vervangen door JSMin+)

Ik hoop eerdaags wat tijd te vinden om variabele minification te gaan inbouwen; dat zou JSMin+ de eerste PHP-based tool maken die dat kan

nice catch; I'll look into it. Luckily we use sensible names for our functions

joey: I can't reproduce this error with the latest version of JSMin+ - a semicolon is added, so there is no parsing error as far as I can see...

[Comment edited on Thursday 19 November 2009 01:24]

Hi Steve,

Yeah, I read about the JSMin license issue - it's a bit pathetic...

As for the memory issues; it's something that I am aware of. I don;t think the problem is in the singlton pattern, but more in the fact that the whole javascript file is being tokenized before being handed off to the minification routine. PHP just isn't quite efficient memory-wise handling large (and deep) tree-like objects/arrays, and the default 64MB memorylimit can quickly get exhousted when minifying rather large javascriptfiles (such as complete libraries/frameworks).

I think the only solution to that would be to handle the minification in the JS parser itself, minifying completed block-scopes so we can throw away those tokens when done. It might even speed up the whole process a bit.

Unfortunately I don't have much time on my hands to try to fix that since it's a rather drastic change. It's on my radar though...