Opened 6 months ago

Closed 6 months ago

#1488819 closed Bugs (fixed)

absolute positioning in html mails miscalculated in larry

Reported by: JohnDoh Owned by: thomasb
Priority: 5 Milestone: 0.9-beta
Component: User Interface Version: git-master
Severity: normal Keywords:
Cc:

Description

if an html mail is received which contains elements placed using absolute positioning then the mail is displayed wrong because the position is calculated relative to the top of the preview frame rather than the #messagebody div. For example if a header in an html email has an absolute position of 0 0 it will be rendered over the message headers div rather than directly below it. Adding position: relative; to #messagebody on line #978 in mail.css solves this.

Attachments (2)

example.eml​ (1005 bytes) - added by JohnDoh 6 months ago.
example email
css.patch​ (748 bytes) - added by JohnDoh 6 months ago.

Download all attachments as: .zip

Change History (6)

Changed 6 months ago by JohnDoh

example email

comment:1 Changed 6 months ago by alec

  • Milestone changed from later to 0.9-beta
  • Owner set to thomasb

Proposed modification doesn't work on message print page.

comment:2 Changed 6 months ago by alec

What's more the red header overlaps message text. I can add also that the issue in print page is reproduceable also in classic skin.

Changed 6 months ago by JohnDoh

comment:3 Changed 6 months ago by JohnDoh

attached a simple patch to fix it in all 3 places. The red header overlapping the text is a mistake in the example message not Roundcube.

comment:4 Changed 6 months ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Thank you. Fixed in 14d00daa1a86d2eed71ff3dd5092281a48ad037c

Note: See TracTickets for help on using tickets.