somehow a clearfix made it into the taxonomy-term.tpl.php, where its hard to se whats its actually doing here besides of making it impossible to float anything left or right of the terms.

this

print $term->tid; " class=" print $classes; clearfix">

should be this

print $term->tid; " class=" print $classes; ">
CommentFileSizeAuthor
#2 taxonomy-clearfix-1198088-01.patch557 bytesJacine
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mortendk’s picture

hurray and i forgot the code tagging...

somehow a clearfix made it into the taxonomy-term.tpl.php, where its hard to se whats its actually doing here besides of making it impossible to float anything left or right of the terms.

This

<div id="taxonomy-term-<?php print $term->tid; ?>" class="<?php print $classes; ?> clearfix">

should be this

<div id="taxonomy-term-<?php print $term->tid; ?>" class="<?php print $classes; ?>">
Jacine’s picture

Status: Active » Needs review
FileSize
557 bytes

Here's a patch that removes the clearfix class.

sun’s picture

Title: die clearfix die - taxonomy-term.tpl.php dont want you. » Obsolete .clearfix in default taxonomy-term.tpl.php template
Status: Needs review » Reviewed & tested by the community

Looks good.

@mortendk: A clear issue title helps others to find issues, manages expectations, speeds up reviews, and most commonly leads to faster commits.

mortendk’s picture

@sun yup i was kinda fired up by the holy fire ;)

webchick’s picture

Does this need to be fixed in 8.x too?

Jacine’s picture

Version: 7.x-dev » 8.x-dev
Issue tags: +Needs backport to D7

Yes, it needs to go into both. Oops.

xjm’s picture

Tagging issues not yet using summary template.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x and 8.x. Thanks!

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