[jifty-devel] CSS::Squish bug/patch

Thomas Sibley trs at bestpractical.com
Tue Jul 4 14:41:30 EDT 2006


Sean E. Millichamp wrote:
> CSS::Squish will do the following:
> - reads the default Jifty main.css
> - imports Jifty's app-base.css because it is in the same search path as
> main.css and ignores your app-base.css
> - imports the default Jifty css for everything else

Yeah, that's a bug.  Thanks for pointing it out.

> I have attached a patch that seems to fix this behavior with regards to
> Jifty but does break the CSS::Squish tests and may break other things.
> Of course, when using this patch you must also make this change to
> Jifty::Web, change line 911 to:
>         CSS::Squish->roots( $app, $jifty );
> instead of just:
>         CSS::Squish->roots( $jifty );

I think the right way to fix this is to have CSS::Squish search any 
user-specified roots _before_ searching relative to the importing file. 
  This way the module still works for people not using the extra roots. 
  Does that sound sane?

Tom


More information about the jifty-devel mailing list