Closed Bug 167815 Opened 22 years ago Closed 22 years ago

Compose mail to context menu from mailto in message header always brings up plain text compose

Categories

(MailNews Core :: Composition, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.3beta

People

(Reporter: esther, Assigned: neil)

References

Details

(Keywords: regression, Whiteboard: [adt2])

Attachments

(1 file)

Using the trunk build 20020910 on winxp and linux when you use the context menu on a email address in the header of a viewed message the compose window comes up in plain text even if the identity is set for html compose. This is regression from the released branch build. 1. Launch app 2. Have all your mail accounts set to use HTML compose 3. Open a mail message (I opened both a plain text and html compose msg) in the message pane, 4. Right mouse click on one of the recipients. Result: The compose window comes up in plaintext compose Expected: In this case, since I had all my mail accounts including the default account set to HTML compose, I should have has an HTML compose window.
.
Assignee: ducarroz → varada
Keywords: regression
I see this with mailto links withing a mail message too.
I see this in Win 2000 too (2002092511)
This is true from mailto links in web pages also, such as those in Bugzilla
*** Bug 176504 has been marked as a duplicate of this bug. ***
*** Bug 180453 has been marked as a duplicate of this bug. ***
*** Bug 183084 has been marked as a duplicate of this bug. ***
*** Bug 183359 has been marked as a duplicate of this bug. ***
*** Bug 183375 has been marked as a duplicate of this bug. ***
*** Bug 183233 has been marked as a duplicate of this bug. ***
*** Bug 183603 has been marked as a duplicate of this bug. ***
The web page mailto: behaviour is a security feature, to stop malicious HTML being embedded into the message.
Is there a way to switch Composer manually to HTML mode (i.e. a menu command)? This might be a satisfactory workaround, especially for web page mailto links.
Keywords: nsbeta1
Security Behavior? There isn't going to be any extra HTML that I didn't already trigger when I viewed the message, is there?
Is the security concern that an html mailto: link can carry text content as a parameter? If so, it would seem smarter to simply limit the treatment of importing those parameters to plain text and ensure that no html is included, the same way keyboard text is limited. There is no way to type in html tags, for example.
*** Bug 184125 has been marked as a duplicate of this bug. ***
taking all of varada's bugs.
Assignee: varada → sspitzer
This bug is probably due to changes made in bug 90728, "mailto: link treats body= as HTML".
Mail triage team: nsbeta1+/adt2
Keywords: nsbeta1nsbeta1+
Whiteboard: [adt2]
*** Bug 191069 has been marked as a duplicate of this bug. ***
Attached patch Proposed patch (deleted) — Splinter Review
Comment on attachment 112986 [details] [diff] [review] Proposed patch I think this should open the compose window correctly; I thought it would be nice to use the full name as displayed, not just the address.
Attachment #112986 - Flags: superreview?(sspitzer)
Attachment #112986 - Flags: review?(ducarroz)
Attachment #112986 - Flags: review?(ducarroz) → review?(cavin)
Comment on attachment 112986 [details] [diff] [review] Proposed patch r=cavin.
Attachment #112986 - Flags: review?(cavin) → review+
Comment on attachment 112986 [details] [diff] [review] Proposed patch sr=sspitzer nice, neil.
Attachment #112986 - Flags: superreview?(sspitzer) → superreview+
over to neil. since it's a regression, how about 1.3 beta? I'll seek approval
Assignee: sspitzer → neil
Target Milestone: --- → mozilla1.3beta
Comment on attachment 112986 [details] [diff] [review] Proposed patch a=asa (on behalf of drivers) for checkin to 1.3beta.
Attachment #112986 - Flags: approval1.3b? → approval1.3b+
Fix checked in. While writing this patch I noticed that the new message (both before and after) always uses the default identity; is that another regression?
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
"While writing this patch I noticed that the new message (both before and after) always uses the default identity; is that another regression?" we'd want to use the proper, current identity. can you spin up a new bug about that?
per comment 29 : bug 191732 was opened for that.
*** Bug 189407 has been marked as a duplicate of this bug. ***
If i click a mailto link in browser, the composer comes up in plaintext mode (current CVS, Linux. According to bug 190120, users on both Linux, Mac and Win2000 are still seeing this bug. What exactly was fixed here? Did something regress again?
> What exactly was fixed here? The "links" in the message header pane.
Sorry - then a lot of bugs have mistakenly been dup'ed against this bug. I reopened them and dup'ed against bug 190120 instead.
Those who voted for this bug believing it was about mailto links in browser (and not only mailto in a mailnews message header) - please move your votes to bug 190120 instead: FLLJimmB@earthlink.net castaban@yahoo.ca jc@thp.org
Using trunk builds 20030303 on winxp macosx and linux and the original scenario: mailto links in message header, this is fixed. Verified Note all incorrectly dup'd bugs have been correctly dup'd to 190120.
Status: RESOLVED → VERIFIED
I see this again in 1.4b (20030507). The mailto: link opens a plain text compose window disregarding the mail composition settings.
Using branch build 20030617 (1.4) this is still working as fixed. Note to nebigredfan@netscape.net in comment 37 If you click on a mailto link within a mail message in an account other than the default mail account or from a browser page, it will use the identity and compose format of the default account (it does not use the identity of the account where you are reading the message). However, If you right click on an email address in a header of a mail message to select "Compose Mail to" it will use the identity of the account from which you are reading the message. This still works correctly with the 20030617 branch build.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: