The Skinr Rule settings fieldset contains all enabled themes, whether the region definition exists for that theme or not. This should not happen. Showing these options for any theme that does not have the specified region is pointless, confusing and just adds unnecessary clutter to the UI.
I think we should be checking to see whether the region actually exists based on it's system name, and only print that fieldset if it's a region that is shared across themes. For example, if someone chooses the region type: region__triptych_last, that region only exists in Bartik, and therefore only the Bartik options should appear. If there are 3 themes enabled, and all themes share a region, like region__content, then all 3 themes should appear.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | skinr_943064.patch | 1010 bytes | moonray |
Comments
Comment #1
moonray commentedHere's a patch.
A theme's skinr settings will only appear when editing a rule if the region exists for that theme.
Comment #2
nomonstersinme commentedThis patch is working for me.
Comment #3
jacineCommitted :D http://drupal.org/cvs?commit=442140