[jifty-devel] CSS::Squish bug/patch
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?
More information about the jifty-devel