defaults.css in the system.module contains the following section:
/*
** Markup free clearing
** Details: http://www.positioniseverything.net/easyclearing.html
*/
.clear-block:after {
content: ".";
display: block;
height: 0;
clear: both;
visibility: hidden;
}
In Firefox this works as expected, but in Internet Explorer 8 (tested in standards mode) the height: 0;
is ignored, because content: ".";
causes a minimum height of one line-height.
The easy workaround for Internet Explorer is to add font-size: 0;
to the section.
A demo is at http://winhlp.com/, where defaults.css is already patched and has the additional font-size: 0;
line. You can use IE's Developer Tools to disable that line and observe that a one-line gap opens between the main table's body (the longest of the three columns) and the blue footer.
You can try the same in Firefox and observe that nothing happens there, i.e. for that browser the font-size: 0;
line is not needed.
I have not tested this with other browsers, but it seems obvious that the zero font-size cannot possibly do any harm if the intention is to have zero height anyway.
I'm not conversant with patching and would be grateful if any of the insiders could do it.
Comment | File | Size | Author |
---|---|---|---|
#2 | Drupal_Core-Clearfix_IE8-857098-2.patch | 911 bytes | patrikpicoli |
Comments
Comment #1
hgmichna CreditAttribution: hgmichna commentedForgot to mention: It would probably be better to rework this CSS issue entirely on the basis of this newer article:
http://www.sitepoint.com/blogs/2005/02/26/simple-clearing-of-floats/
But as long as nobody finds the time, at least the little fix at the top would be helpful.
Comment #2
patrikpicoli CreditAttribution: patrikpicoli commentedJust apply patch to resolve problem to clearfix on IE8.
Comment #3
patrikpicoli CreditAttribution: patrikpicoli commentedChanging issue status to "needs review".
Comment #5
patrikpicoli CreditAttribution: patrikpicoli commentedChanging version to pass on patch test
Comment #6
patrikpicoli CreditAttribution: patrikpicoli commented#2: Drupal_Core-Clearfix_IE8-857098-2.patch queued for re-testing.