141835 - image/jpg attachments open in the mail window instead of a new browser window

Status: VERIFIED FIXED

(SeaMonkey :: UI Design, defect)

Description

[Scott MacGregor]

Stephen has a message containing a url which refers to a jpg. The server gives a
content type of:

image/jpg
instead of the standard: image/jpeg. 

If you try to click on this link in a message pane or open it as an attachment,
we load the attachment in the message pane instead of in a browser window. This
is because the browser window is registered as a content handler for image/jpeg
but NOT for image/jpg. 

Simple change to make that work.

Comment 1

[Scott MacGregor]

Attachment: the fix, register image/jpg

Comment 2

[Scott MacGregor]

accepting.

Comment 3

[Navin Gupta]

Comment on attachment 82092 [details] [diff] [review]
the fix, register image/jpg

r=naving, that was fast!

Comment 4

[(not reading, please use [e-mail] instead)]

Comment on attachment 82092 [details] [diff] [review]
the fix, register image/jpg

sr=sspitzer

Comment 5

[Scott MacGregor]

fix checked in. Stephend already has a test case that demonstrates this problem. 

Comment 6

[Scott MacGregor]

this should have been nominated.

Comment 7

[Jaime Rodriguez, Jr.]

guyatri/esther - who should verify this fix on the trunk?

Comment 8

[Gayatri Rimola]

Stephen, 
please verify this on the trunk.Thanks.

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

My testcase is simple - 

1. Load http://10.169.103.61/scripts/nph-cookie_with_jpg.pl in the browser
2. File | Send Page to self.
3. Read the message.
4. Click on the url in the message.

On all trunk builds (2002-6-17-08), this resulted in the image loading in the
browser window (either the existing one, or if none existed, it opened a new
window).

Verified FIXED on the trunk with Mac OS X 10.1.5, Windows 2000, RedHat 7.3 Linux.

Comment 10

[Paul Wyskoczka]

Adding adt1.0.1+ on behalf of the adt for checkin to the 1.0 branch.  Please get
drivers approval before checking in.

Comment 11

[Judson Valeski]

please checkin to the 1.0.1 branch. once there, remove the "mozilla1.0.1+"
keyword and add the "fixed1.0.1" keyword.

Comment 12

[Randell Jesup [:jesup] (needinfo me)]

Comment on attachment 82092 [details] [diff] [review]
the fix, register image/jpg

as per jud

Comment 13

[Scott MacGregor]

fixed on the 1.0.1 branch

Comment 14

[[e-mail] (gone - use [e-mail] instead)]

This is fixed with all branch 1.0 builds from 6/25/2002 on:

Mac OS X 10.1.5
Mac OS 9.2.2
Windows 2000
Linux RedHat 7.3