Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Before (notice the logo overlaps the content):
After adding the following to the stylesheet:
body.in-maintenance #logo {
width:180px;
}
We get the correct size logo:
Comment | File | Size | Author |
---|---|---|---|
#10 | seven-logo-mant-1384224-10.patch | 330 bytes | oriol_e9g |
#5 | resize-logo-1384224-5.diff | 350 bytes | philbar |
#3 | resize-logo-1384224-3.diff | 635 bytes | philbar |
#2 | resize-logo-1384224-3.diff | 330 bytes | Plazik |
#1 | resize-logo-1384224-2.diff | 326 bytes | philbar |
Comments
Comment #1
philbar CreditAttribution: philbar commentedComment #2
Plazik CreditAttribution: Plazik commentedWhat happens if the logo smaller than 180px?
It's should be
max-width
instead ofwidth
.Comment #3
philbar CreditAttribution: philbar commentedmax-width
is not supported in IE6. Adding fallback toie6.css
.Comment #4
droplet CreditAttribution: droplet commented#2 looks good. can you reroll it for D8 first.
ummm. drupal is missing a good logo function, our logo are no dimension values.
Comment #5
philbar CreditAttribution: philbar commentedPatch rerolled against 8.x.
Note: this doesn't support IE6, which isn't a problem since it isn't supported in 8.x.
Comment #6
philbar CreditAttribution: philbar commentedComment #7
droplet CreditAttribution: droplet commentedComment #8
sunThis indeed looks good to go.
That said, I wouldn't bother with IE6 in the backport for D7 (it only supports
width
and that would break all logos that happen to be smaller).Comment #9
catchCommitted/pushed to 8.x.
Moving back to 7.x for backport.
Comment #10
oriol_e9gAs comment @sun ported without IE6 specific code.
Comment #11
sunThanks.
Comment #12
konichiwa50 CreditAttribution: konichiwa50 commentedthis is very nice.
Comment #13
webchickIck! Nice fix. Sounds like IE6 just ignores this so it won't look more ugly there than it does without the patch.
Committed and pushed to 7.x. Thanks! :)