197735 - [patch] Improve amount of visible Subject header when attachments present

Status: NEW

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Jesse Glick]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3) Gecko/20030314
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3) Gecko/20030314

I work on a laptop with a 1024x768 display and use Mozilla Mail (currently Moz
1.3 release) in the layout with the list of folders occupying the left of the
screen. This leaves maybe 800 pixels for the width of the message pane. It is
often the case that I am looking at a message with a moderately long subject
line and one or two attachments, with the message header ("Normal" - only
important headers) expanded (e.g. Subject on its own line).

In this circumstance it is common for the subject to be truncated by the
attachments pane. However I do not really need to see all that blank space for
the attachments pane (in the normal case that there are <= 3 attachments). The
attached patch lets the subject line stretch all the way to the edge of the
window, pushing the attachments pane beneath it. That still leaves room for
three attachments, which is normally plenty, and seems a more efficient use of
space.

See the attached screenshot, from a Moz dev build with the patch in place.
Without the patch, the subject is truncated around the "W" in "Web".

Of course other headers *might* be long but Subject is much more commonly a long
string than the others (except possibly To/Cc). So the patch only stretches the
Subject header.

Reproducible: Always

Steps to Reproduce:
1. Open a message with a longish subject and at least one attachment, with the
message window around 800 pixels.
Actual Results:  
The subject is truncated by the attachments pane.

Expected Results:  
Let the subject run to the end of the window and put the attachments pane below it.

Comment 1

[Jesse Glick]

Attachment: Suggested patch (made against dev sources)

Patch should be self-explanatory. Lightly tested in a dev build.

Comment 2

[Jesse Glick]

Attachment: Screenshot of the patch working

Taken using Little Mozilla 1.3 theme. Message being displayed is from
nntp://news.netbeans.org/netbeans.nbusers

Comment 3

[Felix Miata]

This is needed XP.

Comment 4

[Olivier Cahagne]

Marking NEW as it has a patch.

Comment 5

[Karsten Düsterloh]

The patch here doesn't solve the actual problem, it only delays it a bit.