Closed Bug 512454 Opened 12 years ago Closed 12 years ago

Improve the look of the inline attachment separator.

Categories

(SeaMonkey :: Themes, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.0b2

People

(Reporter: bwinton, Assigned: bwinton)

Details

Attachments

(1 file, 1 obsolete file)

In bug 509616 I made some changes to Thunderbird to make the attachment separator prettier, but I couldn't get a working build of SeaMonkey to test similar changes to it, so I left them out of that patch.

Anyways, here are the theme changes, still untested, and a fix for a nit that Neil found after it was checked in.

Thanks,
Blake.
Attachment #396426 - Flags: review?(neil)
Comment on attachment 396426 [details] [diff] [review]
A patch for theme changes to SeaMonkey.

(Applies to both themes)
>+.mimeAttachmentHeader {
>+  width: 98%;
Because of the way widths are calculated, this is actually relative to the window width, not the automatic width (which is 20px less). This means that if your window is 1000 pixels wide then the size works out to be the same and if your window is narrow then this actually makes the fieldset wider!

I guess you have three options:
1. Make the width less. 97% is borderline, 96% would be better, or even less.

2. Don't set the width; just use the defaults.

3. As 2. but also set an end margin, e.g. use -moz-margin-end: 2%;
(You may also want to set a start margin, to override the default 2px.)
Attachment #396426 - Flags: review?(neil) → review+
Attachment #396426 - Attachment is obsolete: true
Attachment #396443 - Flags: ui-review?(nisses.mail)
Attachment #396443 - Flags: review?(neil)
Attachment #396443 - Flags: review?(neil) → review+
Comment on attachment 396443 [details] [diff] [review]
The previous patch, with Neil's and Andreas's suggestions
[Checkin: Comment 4]

Tested on Linux, OS X and Vista. Looks good!
Attachment #396443 - Flags: ui-review?(nisses.mail) → ui-review+
Status: NEW → ASSIGNED
Assignee: nobody → bwinton
Keywords: checkin-needed
Comment on attachment 396443 [details] [diff] [review]
The previous patch, with Neil's and Andreas's suggestions
[Checkin: Comment 4]


http://hg.mozilla.org/comm-central/rev/a39d775470fc
Attachment #396443 - Attachment description: The previous patch, with Neil's and Andreas's suggestions. → The previous patch, with Neil's and Andreas's suggestions [Checkin: Comment 4]
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.0b2
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.