I added 

#admin_menu { 
  display: none;
}

in the css to hide it by default.

CommentFileSizeAuthor
#6 admin_menu_dropdown-allow-menu-to-show-by-default-1259182-6.patch320 bytesizmeez
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cosmodrom’s picture

cosmodrom CreditAttribution: cosmodrom commented

Thanks for the nice ninja trick anyway, it is very helpful as I'm constructing absolute positioned elements over the site.

Josephnewyork’s picture

Josephnewyork CreditAttribution: Josephnewyork commented

Add this in your CSS, that way it works the way it was suppose to even if "hide by default" was not checked...

body.adm_menu_hidden #admin-menu {
  display: none;
}
Josephnewyork’s picture

Josephnewyork CreditAttribution: Josephnewyork commented
Status: Active » Needs review

I noticed that there is a CSS file with the module... That means that the code must be added to admin_menu_drop.css and this is not a fix situation, but incomplete code. ALSO, the css that is there isn't correct either and has no effect. The heiarchy of CSS also depends on the complexity of the selector hence overwriting the margin-top body.adm_menu_hidden has no effect and must be body.admin-menu.adm_menu_hidden. Its a full rewrite, so I'm not attaching a patch. Please review:

admin_menu_drop.css

/* $Id$ */

body.admin-menu.adm_menu_hidden {
  margin-top:0px !important;
}
body.adm_menu_hidden #admin-menu {
  display: none;
}
izmeez’s picture

izmeez CreditAttribution: izmeez commented
Version: 7.x-3.0-alpha1 » 7.x-3.x-dev
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

The fix in #3 works as expected.
It includes two components.
The correction to the existing css file is a duplicate of #2047537: Body margin did got get reset properly

The fix from #2 works for when the configuration "hide by default" box is not checked, allowing the menu to be shown as default.

izmeez’s picture

izmeez CreditAttribution: izmeez commented
Title: Admin menu does not hide by default » Admin menu allow to be shown by default

I have changed the title to reflect what this fix actually doing.

izmeez’s picture

izmeez CreditAttribution: izmeez commented
Status: Reviewed & tested by the community » Needs review
FileSize
admin_menu_dropdown-allow-menu-to-show-by-default-1259182-6.patch320 bytes

Here is the fix rolled as a patch against the current 7.x-3.x branch