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)
MailNews Core
Composition
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.3beta
People
(Reporter: esther, Assigned: neil)
References
Details
(Keywords: regression, Whiteboard: [adt2])
Attachments
(1 file)
(deleted),
patch
|
cavin
:
review+
sspitzer
:
superreview+
asa
:
approval1.3b+
|
Details | Diff | Splinter Review |
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.
Comment 3•22 years ago
|
||
I see this in Win 2000 too (2002092511)
This is true from mailto links in web pages also, such as those in Bugzilla
Comment 5•22 years ago
|
||
*** 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. ***
Comment 8•22 years ago
|
||
*** Bug 183359 has been marked as a duplicate of this bug. ***
*** Bug 183375 has been marked as a duplicate of this bug. ***
Comment 10•22 years ago
|
||
*** Bug 183233 has been marked as a duplicate of this bug. ***
Comment 11•22 years ago
|
||
*** Bug 183603 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 12•22 years ago
|
||
The web page mailto: behaviour is a security feature, to stop malicious HTML
being embedded into the message.
Comment 13•22 years ago
|
||
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.
Comment 14•22 years ago
|
||
Security Behavior? There isn't going to be any extra HTML that I didn't already
trigger when I viewed the message, is there?
Comment 15•22 years ago
|
||
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.
Comment 16•22 years ago
|
||
*** Bug 184125 has been marked as a duplicate of this bug. ***
Comment 18•22 years ago
|
||
This bug is probably due to changes made in bug 90728, "mailto: link treats
body= as HTML".
Comment 19•22 years ago
|
||
Mail triage team: nsbeta1+/adt2
Comment 20•22 years ago
|
||
*** Bug 191069 has been marked as a duplicate of this bug. ***
Assignee | ||
Comment 21•22 years ago
|
||
Assignee | ||
Comment 22•22 years ago
|
||
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)
Updated•22 years ago
|
Attachment #112986 -
Flags: review?(ducarroz) → review?(cavin)
Comment 23•22 years ago
|
||
Comment on attachment 112986 [details] [diff] [review]
Proposed patch
r=cavin.
Attachment #112986 -
Flags: review?(cavin) → review+
Comment 24•22 years ago
|
||
Comment on attachment 112986 [details] [diff] [review]
Proposed patch
sr=sspitzer
nice, neil.
Attachment #112986 -
Flags: superreview?(sspitzer) → superreview+
Comment 25•22 years ago
|
||
over to neil.
since it's a regression, how about 1.3 beta? I'll seek approval
Assignee: sspitzer → neil
Target Milestone: --- → mozilla1.3beta
Updated•22 years ago
|
Attachment #112986 -
Flags: approval1.3b?
Comment 26•22 years ago
|
||
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+
Assignee | ||
Comment 27•22 years ago
|
||
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
Comment 28•22 years ago
|
||
Regression bug 191452.
Comment 29•22 years ago
|
||
"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?
Comment 30•22 years ago
|
||
per comment 29 : bug 191732 was opened for that.
Comment 31•22 years ago
|
||
*** Bug 189407 has been marked as a duplicate of this bug. ***
Comment 32•22 years ago
|
||
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?
Assignee | ||
Comment 33•22 years ago
|
||
> What exactly was fixed here?
The "links" in the message header pane.
Comment 34•22 years ago
|
||
Sorry - then a lot of bugs have mistakenly been dup'ed against this bug.
I reopened them and dup'ed against bug 190120 instead.
Comment 35•22 years ago
|
||
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
Reporter | ||
Comment 36•22 years ago
|
||
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
Comment 37•21 years ago
|
||
I see this again in 1.4b (20030507). The mailto: link opens a plain text
compose window disregarding the mail composition settings.
Reporter | ||
Comment 38•21 years ago
|
||
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.
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•