Closed (duplicate)
Project:
Drupal core
Version:
8.0.x-dev
Component:
toolbar.module
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
9 Mar 2013 at 23:35 UTC
Updated:
29 Jul 2014 at 22:00 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #1
jessehsPatch attached adds a min-height attribute.
Comment #2
jessehsComment #3
jmarkel commentedCould you document how to reproduce this?
I haven't see this issue and am not sure what steps are needed to make the problem manifest itself.
Comment #4
megachrizI found two ways to reproduce the issue:
1. Disable certain menu links of menu 'Administration menu':
2. Login as a user that has access to the toolbar, but not to the links:
Comment #5
megachrizThe patch fixes the "silly" looking toolbar submenu (tested in Safari only), but I wonder if the submenu should be shown at all when there are no links to display? Now, with the patch applied, you just have an empty toolbar submenu (see also the attached screenshot). That doesn't look useful to me (though it's probably better than the situation posted in #0).
Comment #6
rajneeshb commentedCreated a patch for hide toolbar submenu if there are no any links.
Comment #7
ZZman commentedI wanted try Your patch, but I can't apply it.

There isn't toolbar.base.css file on drupal folders.
It seems that there is a change in files structure.
And problem with toolbar sub-menu is fixed.
But - when I try reproduce suggestion #4 (variant 2) I found this behavior:
Ordinary user have after login page like this

If I elevate user permission to "Use the administration toolbar "

Comment #8
ZZman commentedThen main page changes as follows (white area appear on left side and administration toolbar cover right top corner)

Diferences in pages source code ...

Comment #9
jessebeach commentedThank you for logging this. I'm upping the priority to Normal. It may need to go to Major, but not yet.
This should be dealt with at the render layer, not the CSS. CSS is just masking bad behavior in the PHP.
jessehs, do you feel comfortable proposing a fix for the PHP side?
Comment #10
sam152 commentedClosing this as duplicate in favor of #2135445: Toolbar displays Manage tab even if the user is not permitted to see it, it has a better summary and a working patch.