Closed (fixed)
Project:
Skinr
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
19 Nov 2010 at 23:23 UTC
Updated:
12 Jan 2011 at 01:10 UTC
Jump to comment: Most recent file
Skinr will provide a plugin containing groups for skin implementors to utilize when creating skins.
This will hopefully help to provide some consistency in the UI, by promoting the re-use of groups as categories.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | skinr_977110_13.patch | 799 bytes | moonray |
Comments
Comment #1
jacineHere's what I'm thinking for this:
Comment #2
nomonstersinme commentedno description field? only asking because the version of the skinr.inc that i looked at today had it. otherwise this makes sense to me.
Comment #3
moonray commentedSeconding what amanda said.
Comment #4
jacineNot if it's going to print on the settings form. This is what I wanted the admin description for, but...
Any other feedback on the actual groups?
Comment #5
jacineThe comments above each would serve as the description though, FTR.
Comment #6
jacineI discussed this with @sociotech and have tweaked it a little.
The "General" group is a bit of a concern, and we've agreed not to use it, and instead let "Style" group be the default. We've also slightly changed the order based on the frequency of use.
Comment #7
sunHm. Couple of issues here:
1) "styles" -- all skins are styles; doesn't make much sense for me. Contrary to that, layout and typography make perfectly sense to me. I'd rather go with a "default", "general", or "other" group for all skins that don't belong into layout and typography, and revisit the necessity for further default groups at a later point in time.
2) "advanced" and "information" do not seem to be actual groups for skins -- rather additional groups in the theme hook configuration form...?
Comment #8
jacineAll may technically be styles, but all styles are not equal. Default is definitely not better for this because it doesn't say anything about what type of skin is inside. I don't like General for the same reason, but I know we need something for a default and after discussing with @sociotech I feel good about Styles over General. I realize it's not perfect and I'm open to other names, but the purpose of the Styles group is pretty clear to me and the other people (that write skins) which have provided feedback so far. It's where block, pane and menu styles would go. If you can think of a better name, I'd love to hear it.
I don't know about this. I think advanced would be fine as a group, as there are other non-skins that could go in here eventually, such as template support, but information doesn't really belong here. Both of these buckets do need to go in, but I don't care "how" they get there, so I'm fine taking them both out of here.
Comment #9
ChrisBryant commentedThis looks good and makes sense to me.
I was thinking about specific examples and thought about if I had a skin that created a dropdown menu or a skin that implemented a Lightbox style effect, what would those be categorized as?
In this case I would put both of those skins under the default "Style" group, but I guess they could create their own groups for "Menu" and "Overlay" if it made the most sense.
Where would you recommend grouping those example skins?
Comment #10
jacineUnder Style. I went back and forth on including navigation as a tab, but decided it's probably best not to add it right now. Then it also gets confusing as to what qualifies as navigation and what doesn't. We don't want to make too many assumptions on how things should be named and grouped. If someone wants to make a group for "Lightbox" styles, they can.
I'm still not entirely happy with the "Styles" name for the group and have been trying to come up with a decent name to separate the presentational container styles from general content styles like lists, margins, paddings and other things that might be inside blocks, like more links, forms and buttons, etc. I really would like to separate these, but my mind is blank. This is the best I could come up with (adding general back, but not as the default, and calling "Styles" box style):
I also removed the advanced and information groups for now, because I don't know if they are needed here or not.
Comment #11
ChrisBryant commentedLooks great to me, thanks Jacine!
Comment #12
jacineCommitted: http://drupal.org/cvs?commit=466150 :)
Comment #13
moonray commentedLooks like we missed some typos... $group instead of $groups.
Here's the patch.
Comment #14
jacineThanks @moonray! Committed: http://drupal.org/cvs?commit=470880