Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.This patch adds support for theme_div and theme_span.
I've tested this on CVS and it works fine.
Darix is also doing some work on this in modules.
Robin
| Comment | File | Size | Author |
|---|---|---|---|
| themedivspan.patch | 1.19 KB | Robin Monks | |
Comments
Comment #1
Steven CreditAttribution: Steven commentedI'm not exactly sure what the point of this is... how would a themer make use of these?
A more meaningful theme('wrapper') was being made by Bèr I think. It specifically required id's and/or classes to make sure each usage is easy to identify.
Comment #2
Robin Monks CreditAttribution: Robin Monks commentedHey, oddly enough, that's how my patch was at first, but Bèr told me not to require IDs and Classes.
Now I feel like I wasted an hours work...
Robin
Comment #3
Robin Monks CreditAttribution: Robin Monks commentedWhoa, wait. My logs decieve me, it was darix. Berkes, but deepest appoligies.
I realy can't connect people to nicks...
Robin
Comment #4
Robin Monks CreditAttribution: Robin Monks commentedThis is just duplication of the effort Berkes put in his patch. I'm going to close it. Teaches me to look before I code ;-)
Robin