Integration with Skinr module

I was just looking at this. It seems like it would be really, really easy to add this integration. Just preprocess in the $skinr variable into the $classes_array.

Anyone want to roll a patch for it? :-D

Sure, I'd love to. I'll do it this weekend. :)

Hi!

Well, I totally forgot that Zen doesn't have views template file. Panel panes work out of the box with Zen because it also uses $classes and your code takes care of it (I added it during zen_preprocess()). However, views-view.tpl.php goes the manual route.

I'm not sure if you are interested in adding the views template file, so here are 2 patches ;)

Sorry, use this one. There was a slight boo-boo in the other. Also, an unwanted side effect (in this case) the hook name gets added to the class names. I have overriding views_view, like so:

$vars['classes_array'][0] = 'view';

Not sure if that's how you would want to do it.

Holy crap... That was the wrong patch. I need to stop doing this stuff at 4 AM. Time to go to bed. Sorry for the inbox overload.

This is the right one (hopefully).

I updated the zen_preprocess() function to prevent PHP warnings on templates where $vars['skinr'] is undefined. I also updated the if statement so that Zen continues to skip building a $classes_array when a module is already using $classes for other uses (i.e. views-view-list.tpl.php, etc.) Also, updated comments in the tpl.

Committed.

Jacine, I put you in the CHANGELOG twice. :-)

- #661628 by Jacine and JohnAlbin: Integration with Skinr module
- by Jacine: Added views-view.tpl.php with support for $classes variables.

Yay, awesome! Thank you :D

This is great, you guys ROCK! :-)

Thanks!!

Re: http://twitter.com/merlinofchaos/status/7682119358

Well, I should have asked merlinofchaos before just putting this patch up, so to make up for it I wrote a patch. If he adds it, Zen wont need it but what do you think about it? http://drupal.org/node/682888

FYI, the views-view.tpl.php that Jacine added is for Views 2.

Merlin wants to add $classes to Views 3, which is still in alpha, so its all good.