In page.tpl.php

is used a couple of times. Following http://www.sitepoint.com/css-architectures-new-best-practices/ there is more modern ways to solve this issue.

CommentFileSizeAuthor
#1 issue-1963550.patch1.49 KBlsolesen

Comments

lsolesen’s picture

lsolesen commented
Status: Active » Needs review
StatusFileSize
new issue-1963550.patch1.49 KB

I used the existing clearfix on all the divs, and it seems to work allright.

acke’s picture

acke commented
Status: Needs review » Fixed

Good idea! Makes the code cleaner! Commited to 7.x-2.x HEAD and going into next release.

acke’s picture

acke commented

I found out that clearfix should be a class, for example

<div id="logo clearfix">

won't work. But:

<div id="logo" class="clearfix">

works. I change this and push to 7.x-2.x HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.