As it is possible to note on the screen shot, the theme icon covers part of the graphic.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | Garland logo is gone after changing colors | 19.07 KB | jerdiggity |
| #4 | Garland logo appears, but colors don't change | 42.85 KB | jerdiggity |
| #4 | color.module.patch | 637 bytes | jerdiggity |
| Capture_095664.png | 19.8 KB | avpaderno |
Comments
Comment #1
brianV commentedComment #3
avpadernoI am changing the title to make the issue more generic. The problem appears also in a 404 page, when the left-side column is not used.
If the default logo would be some pixels less in height, the problem would not exist.
Comment #4
jerdiggity commentedChanging the title again to make it more specific; also upgrading priority to critical.
This issue is not solely occurring because of changes in page size, etc. In fact I believe the problem lies within the core
color.module(although I left this issue's component as Garland theme because I'd like a second opinion).While using Garland, if you change the color scheme to anything other than Blue Lagoon then the logo disappears altogether and is replaced with the site name (please see the first pic).
However after playing around with the code a little bit I discovered that in the
color.modulefile, if the following (at ≈ line 305):variable_set('color_' . $theme . '_logo', $paths['target'] . 'logo.png');were changed to this:
variable_set('color_' . $theme . '_logo', $paths['source'] . 'logo.png');then at least a logo starts to appear -- the down-side is that the logo's color scheme (well, actually the logo itself) does not change to match the rest of the theme. It appears to be reusing the Blue Lagoon logo regardless of the selected color scheme (see the second pic).
I also attached a patch for the above code changes, but even with that change applied, a new logo still needs to be rendered. (FTR, test was done using FF so it's not an IE/png issue...)
Comment #6
avpadernoRather than making the issue more specific, you changed the topic. What you report is a completely different issue, which needs a separate report.
Comment #7
avpadernoI am changing the metadata to the original one. Please open a different report for the issue you noticed.
Comment #8
jerdiggity commentedMy apologies.
Comment #9
jerdiggity commentedOops... my apologies again (looks like you beat me to it.)
:)
Comment #10
avpadernoThere is nothing to apologize for; all is returned as it was before, and nothing is lost. :-)
Comment #11
Jeff Burnz commented#216361: Logo image overlaps with center DIV.