Details
-
Type:
Sub-task
-
Status: Closed
-
Priority:
Minor
-
Resolution: Won't Fix
-
Affects Version/s: 2.0, 2.1, 2.2
-
Component/s: Themes
-
Labels:
-
Testing Instructions:
-
Affected Branches:MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
-
Fixed Branches:MOODLE_20_STABLE, MOODLE_21_STABLE
-
Pull from Repository:
-
Pull Master Branch:
-
Pull Master Diff URL:
Description
The fix for MDL-29197 which addressed this same problem, was fixed a few weeks ago, however as mentioned at the time it was not the best solution, as I have since found out. Themes like Sky High, which exclude pagelayout.css where the patch was applied, it appears the problem still persists.
Fortunately, a fix for this has been put forward by John Stabinger, which adds to .clearfix:after css, which is already present in Base theme. John's solutions is as follows...
.clearfix:before,
|
.clearfix:after { content: "."; display: block; height: 0; overflow: hidden; }
|
.clearfix:after { clear: both;}
|
.clearfix {zoom: 1;}
|
Works great! So thanks for that John
Cheers
Mary