This is a follow-up of 473878: Vertical tabs are totally hosed.
As we saw there, the layout can be accidentally broken if the style is overwritten in other css rules.
This patch adds div.vertical-tabs ul to all the rules in vertical-tabs.css that start with .vertical-tabs-list li</li> and just for consistency <code>div to all starting with .vertical-tabs

CommentFileSizeAuthor
vertical-tabs.css_.patch2.52 KBtic2000
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tic2000’s picture

Status: Active » Needs review
webchick’s picture

This looks like it makes sense to me, but I'd love a review from someone else who's good with CSS.

geerlingguy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me. You can almost never be too specific when it comes to CSS specificity. Well, unless you add an id to every element. But that'd just be crazy!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Thanks. :) Committed to HEAD!

Status: Fixed » Closed (fixed)

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