The teaser text and teaser image should be top aligned but are not in IE6 and IE7.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jacine’s picture

Status: Active » Needs review

[edit] oops, wrong issue.

Jacine’s picture

Status: Needs review » Active
Todd Nienkerk’s picture

Assigned: Unassigned » Todd Nienkerk

Looking into it.

willmoy’s picture

Project: Drupal core » Bartik
Version: 7.x-dev »
Component: other » Code
Issue tags: -Bartik

Now we have a bartik project, moving there.

stephthegeek’s picture

Version: » 7.x-1.x-dev
Assigned: Todd Nienkerk » Unassigned
FileSize
26.96 KB

Not only are the top edges not aligned, but we're not getting any nice float wrapping around the image in any IEs. This and the misalignment are mainly due to all the clearfixes being thrown around the fields -- apparently that's in core now: http://drupal.org/node/622330

Not exactly sure how to fix this nicely, will take more investigation on the rationale behind clearfix on all the fields in core.

Jeff Burnz’s picture

Perhaps this is more to do with the margin top on the p element, line 36 style.css

p {
  margin: 1.2em 0 0.6em;
}

I tested by overriding theme_field and removing the clearfix which did not resolve the issue, while digging around in firebug I noticed the margin top on paragraphs, removed it (put clearfixes back in) and viola, fixed, e.g.

p {
  margin: 0 0 1.8em;
}
jensimmons’s picture

Will you roll a patch for this Jeff?

Jeff Burnz’s picture

Status: Active » Needs review
FileSize
663 bytes

Heres the patch, added margin-top to .content to compensate for the lost margin top for the first paragraph.

jensimmons’s picture

Status: Needs review » Fixed

Committed — with an adjustment. Made .content { margin-top: 1em; } instead of 1.2 ems. It was affecting the block headlines too much. We can always adjust more if needed.

:)

Status: Fixed » Closed (fixed)

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