Logo image overlaps with center DIV.
ao2 - January 31, 2008 - 16:30
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | Garland theme |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Description
When we set a void left panel, the bottom of the logo image overlaps with the top of the center DIV.
Steps to reproduce:
- Log in as Administrator
- Go to Administer -> Site building -> Blocks
- Move any items away from the Left sidebar region
- Save and logout
I would have expected no ovelaps. Have a look in the upper left part of the attached picture to see the current, wrong, behaviour.
Please, even if this is a really trivial bug, and people are going to replace the image logo, it would be nice to have that fixed before 6.0
Thanks,
Antonio
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| drupal-garland-logo-overlap-when-no-leftpanel.png | 94.17 KB | Ignored | None | None |
#1
This will pretty much fix it but there is a very, very subtle color mismatch due to the gradient. If we wanted to get rid of that too, the base.png and logo.png graphics would have to be tweaked.
This is also present in Minnelli.
#2
http://drupal.org/node/216848 was duplicate.
#3
Yes, it looks good to me, don’t notice the mismatch. But with a left sidebar I think the logo is noticeably too high.
This makes the same change but conditional on left sidebar presence. I’ve put a class on #header, in case anything else wants to depend on it.
#4
1.)
<body>already puts out the layout classes.2.) Is it the height really that noticeable?
IMO, It's not needed.
#5
This is not such a big change. Tested both themes, with and without right and left sidebars.
#6
this bug is still here in 6.11.
If it is not going to be fixed, can you devs at least mark it "won't fix"?
Thanks,
Antonio
#7
Even in drupal7
#8
does the CSS really need to be this specific?
+body.sidebar-left #wrapper #container #header h1 img, body.sidebars #wrapper #container #header h1 img {+ padding-top: 16px;
+}