I ran across this issue...
http://drupal.org/node/564248

...and Wim stated that we can target the Hierarchical Select by targetting the .hierarchical-select-wrapper class. That's true, but when this module is used in conjunction with Content Taxonomy (actually not sure if this means anything at all), targeting .hierarchical-select-wrapper does not allow the title and description of the HS to be targeted. I'd like to add a simple border around each group. I've attached a screenshot. It is currently styled using .hierarchical-select-wrapper. I see there's an ID, in the following format, that would allow me to target the whole thing, but unfortunately it's an ID, not a class, and I'd like to just be able to target a class and have it applied everywhere.

Format of the ID: hierarchical-select-42-wrapper-wrapper
If we could get something like hierarchical-select-wrapper-wrapper added at the same level as the ID, that would solve the issue. Can this be added? I attempted to look at the code for the module, but I cannot find "wrapper-wrapper" anywhere in the files, which is strange, but if someone can tell me how to edit it, I could make the change now and wait for it to be implemented (if Wim will add it). :-)

Thanks so much, and if I'm missing anything, please let me know. This is one of my top 3 modules. I love it.

CommentFileSizeAuthor
HS-styling.png16.87 KBdmetzcher
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers’s picture

Status: Active » Needs work

Great to hear that you love it so much! :)

The "wrapper-wrapper" part of the class is generated automatically by Forms API.

You should be able to do this by changing the appropriate functions in theme.inc. It'd be great if you could roll a patch, I'd be happy to commit it! :)

dmetzcher’s picture

Unfortunately, I wouldn't even know where to begin with regard to writing that patch. :(
Maybe I'll get lucky and someone else who knows how to do it will pass by this issue.

Wim Leers’s picture

Assigned: Unassigned » Wim Leers
Status: Needs work » Fixed

dmetzcher has kindly offered to sponsor this patch through a donation! Thanks again, dmetzcher! :)

Patch committed that adds the exact requested functionality, without breaking backwards compatibility with previously written CSS: http://drupal.org/cvs?commit=417116

dmetzcher’s picture

It works great! I can now target .hierarchical-select-wrapper-wrapper for styling very easily.

Thank you for the super-fast turnaround time on this!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.