image urls should not be "chrome" urls

VERIFIED FIXED in M11

Status

P3
normal
VERIFIED FIXED
19 years ago
10 years ago

People

(Reporter: Brade, Assigned: rhp)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
when a user inserts an image into composer (or mail compose or html editing
widget), we should convert the url from a "chrome" url to a "file" (or other
appropriate) url.

I discovered this bug when someone sent me a message from 5.0 which I read with
4.61.  An image was included with the message but it was a "chrome" url so the
image wasn't "attached" to the message.
(Reporter)

Updated

19 years ago
Component: Editor → MIME
Product: Browser → MailNews

Comment 1

19 years ago
To reproduce in mail:
1) In 5.0, send yourself a message in html which uses :-)
2) The :-) gets converted to an image
3) View this message in 4.x.
4) The :-) image does not show up
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M11
(Assignee)

Comment 2

19 years ago
I have a fix for this in my tree..I'll check it in sometime today.

- rhp
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

19 years ago
This was a problem where we were replacing emoticons with images that should
only be done on the display side...not the send side. This should be fixed now.

- rhp

Updated

19 years ago
QA Contact: sujay → lchiang

Comment 4

19 years ago
assigning Mail team to this one...

Updated

19 years ago
QA Contact: lchiang → chuang

Comment 5

19 years ago
candice has kindly offered to help verify bugs.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 6

19 years ago
Verified using 1999-12-16-12-M12 Win32, Linux build and 1999-12-16-09-M12 Mac
build,  it is fixed.
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.