thunderbird does not show xhtml attachments inline

NEW
Unassigned

Status

MailNews Core
MIME
13 years ago
5 months ago

People

(Reporter: Darin Fisher, Unassigned)

Tracking

({helpwanted})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

13 years ago
Thunderbird does not show xhtml attachments inline.

I'm using the Linux Thunderbird 0.5 (20040208) build.

It seems like Thunderbird should be able to render xhtml attachments inline.

Comment 1

13 years ago
i'm not sure where we decide if a part is inline or not. it's probably in
libmime hard coded as text/* and image/* or something along those lines....

what's the content type for xhtml files?
Status: NEW → ASSIGNED
application/xhtml+xml
application/xml would be a valid content type as well, I think... as well as
text/xml.

Comment 4

13 years ago
Created attachment 142807 [details] [diff] [review]
patch to show application/xhtml as inline content

Updated

13 years ago
Target Milestone: --- → Thunderbird0.6

Comment 5

13 years ago
this stuff can go into the 1.8 trunk
Target Milestone: Thunderbird0.6 → Thunderbird0.7
Might it be worth following the pref stored in the html_as variable?

Updated

13 years ago
Target Milestone: Thunderbird0.7 → Thunderbird0.8

Comment 7

13 years ago
not a .8 stopper. I may not have time to bounce back to this before 1.0
Keywords: helpwanted
Target Milestone: Thunderbird0.8 → Thunderbird1.0

Updated

13 years ago
Target Milestone: Thunderbird1.0 → Thunderbird1.1

Updated

12 years ago
Target Milestone: Thunderbird1.1 → Thunderbird1.5
Maybe this should use nsIWebNavigationInfo (see bug 283125) so that this code
doesn't have to be updated every time we add support for a new MIME type?  (That
said, it does need to restrict things a bit, e.g., not using plugins.)
QA Contact: front-end

Comment 9

10 years ago
Some findings:
Changing mailnews/mime/mimei.cpp is not enough. "text/html" and TEXT_HTML are treated specially in several places in mailnews/mime/*.

A MIME part whose Content-Type is not one of the currently supported ("text/html", "multipart/*" etc.) shows up as an attachment. 

application/xhtml+xml is currently parsed as HTML tag soup rather than XML.

Updated

9 years ago
Assignee: mscott → nobody
Status: ASSIGNED → NEW
Target Milestone: Thunderbird1.5 → ---
(In reply to comment #9)
> Some findings:
> Changing mailnews/mime/mimei.cpp is not enough. "text/html" and TEXT_HTML are
> treated specially in several places in mailnews/mime/*.
> 
> A MIME part whose Content-Type is not one of the currently supported
> ("text/html", "multipart/*" etc.) shows up as an attachment. 
> 
> application/xhtml+xml is currently parsed as HTML tag soup rather than XML.

Yes, changing mailnews/mime/mimei.cpp makes application/xhtml+xml attachments display inline but they are still loaded with the HTML parser.
I'm just wondering... isn't libmime simply splitting the different parts of a message? So we need to indicate elsewhere in mail/ or mailnews/ that the file has to be loaded as application/xhtml+xml?
Component: Mail Window Front End → MIME
Product: Thunderbird → MailNews Core
You need to log in before you can comment on or make changes to this bug.