[dev] [commits] Horde branch master updated. b31b393c5d7952e4c7391cc41c0b91b24a593fa6

Michael M Slusarz slusarz at horde.org
Mon Mar 18 20:48:35 UTC 2013


Quoting Rui Carneiro <rui.carneiro at portugalmail.net>:

> AFAIK if I start using Horde_PageOutput#**addStylesheet()
> to add this extra stylesheet an extra request will be made and this
> stylesheet will not be minified. Am I right?

Turns out I misspoke.  These files will always be parsed if CSS  
caching is enabled.  That's the side effect of the H5 theme change  
that requires support for multiple url's in a single CSS statement,  
for correct relative URL parsing, and for proper handling of any  
import statements in the CSS files.  (regular expressions can NOT be  
used for any of this anymore due to backtracking limitations  
potentially crashing the PHP server.  Not to mention that regular  
expressions is completely incorrect for parsing a structured language  
anyway).

So the ONLY proper solution is to rewrite csstidy if you want to  
support CSS3.  Otherwise, disable CSS caching.

michael

___________________________________
Michael Slusarz [slusarz at horde.org]



More information about the dev mailing list