mikeytown's patches in #652690-15: Only process CSS if the browser is IE contain some ideas to refactor this module:
- move some code out into unlimited_css_link($files, &$styles).
- move some code out into unlimited_css_import($files, &$styles).

(or better, remove the &$styles argument and use a return value instead).

I think it would make sense to do this stuff separately and before the other ideas of the above patch, so we have a common ground of discussion for future work.

CommentFileSizeAuthor
#1 unlimited_css-855040.patch2.95 KBmikeytown2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeytown2’s picture

Status: Active » Needs review
FileSize
2.95 KB
donquixote’s picture

Thanks.
Now that antonio gave me CVS access, I guess I will take care of this. But not today..

donquixote’s picture

klonos’s picture

So, is this fixed/committed or not? I ask because it is set to NR.

donquixote’s picture

Duh, that's still in -dev...
yeah, should get it committed.
I'm doing my taxes and some party first :)

klonos’s picture

What can I say, a man's gotta do what a man's gotta do ;)

donquixote’s picture

It's now in 6.x-1.3
Please report if anything behaves strangely.