Panels adds a region class to every region, .panel-panel, and ds 6.x adds .ds-region to each region. This simply allows you to apply identical styles to all printed regions in panels or ds.
7.x-2.x is currently lacking this class. The attached patch adds .ds-group to each printed group in template, in line with the use of .group- class names already in use.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 1573106.patch | 1014 bytes | xtfer |
Comments
Comment #1
xtfer commentedPatch attached, and I've also pushed this in a named branch to make it easier to commit.
Comment #2
swentel commentedGoing to assign this to jyve, I don't have any strong opinion on this one.
Comment #3
aspilicious commentedPutting back on top, will ask a themer at the office soon.
Comment #4
aspilicious commentedOk asked the themers and they don't see any benefits. So closing this for now.
Comment #5
xtfer commentedThat was an easy win, lost to the opinions of a minority...
Comment #6
aspilicious commentedWell the reasoning behind this decision is simple. You can already accomplish this by adding the same class to each region in the UI.
Front end geniuses these days don't like unused classes apparently.
But because I appreciate your work I'm going to reopen this again. Maybe I'm missing something?
Comment #7
xtfer commentedTBH I never had much hope for this issue, but I appreciate it being reopened.
I honestly think this is useful, it certainly helped on a few builds with doing simple things like applying equal padding to all regions, borders etc. It's also marginally more semantic - currently the group can't be determined from a layout using child elements, since there may be intermediate elements.
However, given its age and the lack of interest, perhaps I am wrong.