Closed
Bug 370077
Opened 17 years ago
Closed 12 years ago
blockquote not rendered as an indented block
Categories
(Thunderbird :: Theme, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 14.0
People
(Reporter: mail, Assigned: Paenglab)
References
Details
Attachments
(6 files, 1 obsolete file)
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; de; rv:1.8.1.1) Gecko/20061204 Firefox/2.0.0.1 Build Identifier: Version 1.5.0.9 (20061207) Blockquote in HTML message is not displayed as indented block Reproducible: Always Steps to Reproduce: Open the attached message file "blockquote.eml" Actual Results: Three lines of text without indent will be shown. Expected Results: The second line shold be displayed as indented block.
Reporter | ||
Comment 1•17 years ago
|
||
Reporter | ||
Updated•17 years ago
|
Version: unspecified → 1.5
Comment 2•17 years ago
|
||
WFM onMozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.2pre) Gecko/20070208 Thunderbird/2.0pre ID:2007020804
Reporter | ||
Comment 3•17 years ago
|
||
added "padding-left: 1em;" to the blockquote-rule in /Applications/Thunderbird.app/Contents/MacOS/res/html.css (starting at line 75) which fixed it for me -- but I am not a developer.
I can confirm this bug in 2.0.0.6 (20070728) and in 3.0a1pre (2007081003) under Mac OS 10.4.9. The indented text (blockquote) is rendered as an indented block in the stand-alone compose window, but when the message is received the blockquote is not rendered as indented in the Message Pane or in a stand-alone window. Additionally, if the composed message is saved as a draft and viewed in the Message Pane it also renders incorrectly. The attachment shows the 2.0.0.6 rendering in the compose window (top) and the rendering of the saved draft in the Message Pane (bottom). This bug does not appear to affect Linux, another person other than Magnus couldn't reproduce this bug on Linux with 2.0.0.6.
Comment 6•17 years ago
|
||
Is this mac specific? The testcase shows the second line indented (as it should) for me, on linux trunk.
Comment 7•17 years ago
|
||
Ah, should read all of the comments before I comment;)
Whiteboard: [mac only]
Comment 8•17 years ago
|
||
I also see this in 2.0.0.9 (20071031) on 10.4.11. It's mostly a problem for RSS feeds, which tend to use blockquote extensively.
Updated•16 years ago
|
Assignee: mscott → nobody
Comment 9•13 years ago
|
||
I also have this issue with Thunderbird/3.1.9 on MacOS X 10.6 That's very annoying because blockquote HTML elements are created while using the "indent right" button in the HTML mail composition window. In the end multipart/alternative mails are badly interpreted by thunderbird himself. Attached are: - a sample multipart/alternative email - a snapshot of the composition window - a snapshot of what it looks like when viewed in thunderbird
Comment 10•13 years ago
|
||
Comment 11•13 years ago
|
||
Comment 12•13 years ago
|
||
Comment 13•12 years ago
|
||
This bug still persists, even with Thunderbird 10, which I am running under MacOS X 10.6. As reported above, a blockquote or indention does not get rendered in OS X, while this works properly with the Windows version.
Comment 14•12 years ago
|
||
Someone on mac should probably look at this. Msg body seems to lack blockquote[type=cite] rules (adding padding). msg body: http://mxr.mozilla.org/comm-central/source/mail/themes/pinstripe/mail/messageBody.css#68 composition: http://mxr.mozilla.org/comm-central/source/mail/themes/pinstripe/mail/compose/messengercompose.css#998
Comment 15•12 years ago
|
||
Richard , Andreas can one of you look into this ?
Component: Mail Window Front End → Theme
QA Contact: front-end → theme
Assignee | ||
Comment 16•12 years ago
|
||
The OSX version used margin: inherit for blockquotes which removed the indentation. I copied the version from Windows to still make the correct margins for cited text.
Comment 17•12 years ago
|
||
I just updated to Thunderbird 11 on OSX and, unfortunately, it still does not render correctly. Richard, can you merge your fix into the development branch? Or, have you already done so? To be honest, I am not familiar with software development and I am not sure if you provided a workaround or a definitive fix. Thanks, Chris
Assignee | ||
Comment 18•12 years ago
|
||
(In reply to Chris from comment #17) > I just updated to Thunderbird 11 on OSX and, unfortunately, it still does > not render correctly. > > Richard, can you merge your fix into the development branch? Or, have you > already done so? To be honest, I am not familiar with software development > and I am not sure if you provided a workaround or a definitive fix. The patch is awaiting the review. If it is accepted the patch lands in trunk (now TB 14). Then I can ask for approval for TB 12 and 13. If this all happens then TB 12 would have the fix. As a workaround, not tested but you can try to put this in userContent.css: .moz-text-flowed blockquote, .moz-text-plain blockquote { margin: inherit !important; } If you don't know what's userContent.css, read this: <https://developer.mozilla.org/en/Thunderbird/Thunderbird_Configuration_Files>
Comment 19•12 years ago
|
||
Comment on attachment 601673 [details] [diff] [review] Fix Yoinks, stealing this review request.
Attachment #601673 -
Flags: review?(bwinton) → review?(mconley)
Comment 20•12 years ago
|
||
Comment on attachment 601673 [details] [diff] [review] Fix Review of attachment 601673 [details] [diff] [review]: ----------------------------------------------------------------- r=me if we set margin: 0;. ::: mail/themes/pinstripe/mail/messageBody.css @@ +70,1 @@ > margin: inherit; I think we should set margin: 0;, to be consistent with gnomestripe and qute. Modulo that, it seems to fix the issue, so r=me.
Attachment #601673 -
Flags: review?(mconley) → review+
Assignee | ||
Comment 21•12 years ago
|
||
Yeah, it makes sense to give margin: 0; for consistency with the other themes. Carrying over r+ from previous patch
Attachment #601673 -
Attachment is obsolete: true
Attachment #609792 -
Flags: review+
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 22•12 years ago
|
||
http://hg.mozilla.org/comm-central/rev/d2cd202d83ec
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite?
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [mac only]
Target Milestone: --- → Thunderbird 14.0
You need to log in
before you can comment on or make changes to this bug.
Description
•