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

Comments

andrew_mallis’s picture

andrew_mallis commented
Status: Active » Needs review
StatusFileSize
new admin_menu_spacing-1325386.patch360 bytes

See attached patch for fix

prabhatjn’s picture

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 commented
StatusFileSize
new 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 commented
Status: Reviewed & tested by the community » Needs review
andrew_mallis’s picture

andrew_mallis commented

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

paskainos’s picture

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. :)

sickness29’s picture

sickness29 commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

D7 reached its EOL back in January 2025, and there is no active release for D7 for this module anymore.
Development or support is not planned for D7. All D7-related issues are marked as outdated in a bunch.

Everyone can apply the patch above (not tested by the maintainers, tested by the community) to their D7 projects.
If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.

Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.