The Administration Menu module serves as a (better) drop-in replacement for the toolbar. The toolbar was hiding the admin sidebar before a recent CSS fix and I am proposing the fix also apply to this condition.

CommentFileSizeAuthor
#3 admin-admin_menu-fix-1325386-3.patch603 bytespaskainos
#1 admin_menu_spacing-1325386.patch360 bytesAndrew_Mallis
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Andrew_Mallis’s picture

Andrew_Mallis CreditAttribution: Andrew_Mallis commented
Status: Active » Needs review
FileSize
admin_menu_spacing-1325386.patch360 bytes

See attached patch for fix

prabhatjn’s picture

prabhatjn CreditAttribution: prabhatjn commented
Status: Needs review » Reviewed & tested by the community

I think it is working well with all other configurations (whether admin_menu or toolbar or neither of them is enabled).

Perry.

paskainos’s picture

paskainos CreditAttribution: paskainos commented
FileSize
admin-admin_menu-fix-1325386-3.patch603 bytes

I use this method:

/**
 * Playing nice with admin menu
 **/
body.admin-menu div#admin-toolbar.vertical {
  top: 30px;
}
paskainos’s picture

paskainos CreditAttribution: paskainos commented
Status: Reviewed & tested by the community » Needs review
Andrew_Mallis’s picture

Andrew_Mallis CreditAttribution: Andrew_Mallis commented

um. The patch in #3 is pretty much exactly the same as the one I submitted.

paskainos’s picture

paskainos CreditAttribution: paskainos commented

Yes, 'pretty much', but not exactly. The current state of the recommended release of 7.x-2.x (currently 7.x-2.0-beta3) includes div#admin-toolbar.vertical instead of body.toolbar #admin-toolbar.vertical referenced in #1 (and 7.x-2.x-dev is radically different). The difference is admittedly miniscule, and perhaps merely an update to reflect the current module state.

P.S. throwing the patch in the issue queue was also a shameless way to simply include it in an install profile. :)