Change clear style to be a class instead of a [#986676]

Lines 359 and 371 of project.inc
Change

<br style="clear:both" />

to instead be

<br class="clear" />

Optionally you could make the class a unique name a then add a clear:both directive for it in project.css

The reason I make this request is in the theme I use, having a "clear:both" in the content breaks the theme. If you used styles instead of inline markup the one can easily override with the theme css.

If you will accept this change I will roll a quick patch. Thanks.

Comment	File	Size	Author
#3	project-br-clear.patch	1.09 KB	coderintherye

Comments

Comment #1

dww

we/he/they

commented 2 December 2010 at 02:59

Sure, I'd accept such a patch. I think core even has such a class already defined in the default CSS, so we probably don't even need anything in project.css for this.

Comment #2

coderintherye commented 2 December 2010 at 08:01

Core has something similar, but not quite there (from modules/system/defaults.css):


/*
** Markup free clearing
** Details: http://www.positioniseverything.net/easyclearing.html
*/
.clear-block:after {
  content: ".";
  display: block;
  height: 0;
  clear: both;
  visibility: hidden;
}

.clear-block {
  display: inline-block;
}

/* Hides from IE-mac \*/
* html .clear-block {
  height: 1%;
}
.clear-block {
  display: block;
}