Closed
Bug 346966
Opened 18 years ago
Closed 18 years ago
message headers spaced further apart (2.0a1 vs 1.5.0x) - too much vertical space
Categories
(Thunderbird :: Mail Window Front End, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird2.0
People
(Reporter: moco, Assigned: mscott)
References
Details
(Keywords: fixed-seamonkey1.1.1, fixed1.8.1.2, regression)
Attachments
(9 files)
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mnyromyr
:
review+
neil
:
superreview+
|
Details | Diff | Splinter Review |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
image/png
|
Details |
message headers spaced further apart (2.0a1 vs 1.5.0x)
I'm seeing more space in between the subject, from, sender, date headers in the message pane.
screen shots coming...
Reporter | ||
Comment 1•18 years ago
|
||
Reporter | ||
Comment 2•18 years ago
|
||
Reporter | ||
Updated•18 years ago
|
Attachment #231726 -
Attachment mime type: application/octet-stream → image/png
Comment 3•18 years ago
|
||
trunk seems to be like 2.0a1.
Is this considered a good thing (tm) or how should it be?
I actually like it that the lines don't stick together too much.
Assignee | ||
Comment 4•18 years ago
|
||
weird, I don't see this on Windows.
Reporter | ||
Comment 5•18 years ago
|
||
> weird, I don't see this on Windows.
I think I know how to reproduce this. Go to display properties, the Appearance tab, and switch from Normal to Large Fonts under "Font Size".
With large fonts, 2.0a1 will look different than 1.5.0.x. But under "Normal", they look the same.
Reporter | ||
Comment 6•18 years ago
|
||
Comment 7•18 years ago
|
||
20060802 is the same. No problem with this. I like it. No need for Large fonts. o_O
Comment 8•18 years ago
|
||
Looks like bug 335973 probably caused this.
Comment 9•18 years ago
|
||
(In reply to comment #8)
> Looks like bug 335973 probably caused this.
(In reply to comment #27 from bug 335973)
> screenshot - SM after with .headerName, .headerValueBox {margin-bottom: 0
> !important;} in userChrome.css
So this is all that need to be done to restore previous behavior
I still don't understand why this bug is open ... no comment shows which style you prefer...
> I like this much better, but would prefer bug 9942 be fixed.
Comment 10•18 years ago
|
||
I use Large Fonts under Windows 2000, I see this with 3a1-0804, and I would prefer the more compact display of previous builds.
Severity: normal → minor
Comment 11•18 years ago
|
||
*** Bug 345789 has been marked as a duplicate of this bug. ***
Comment 12•18 years ago
|
||
Also on linux, and osx per bug 345789 comment 0. tm from bug 345789
OS: Windows XP → All
Target Milestone: --- → Thunderbird2.0
Comment 13•18 years ago
|
||
I put .headerName, .headerValueBox { margin-bottom: 0 !important; } in userChrome.css to rid the excess space from SM on 21 July, shortly after bug 335973 fix was checked in, so it seems somehow this and 335973 must actually be core.
Comment 14•18 years ago
|
||
space, the final frontier :)
I don't see this as a matter of style or asthetics: a) there are no bugs filed that lines be put closer together, b) if bug 335973 is the cause, intentional or not, doing more than improving the presentation of "e-mail addresses" is beyond the bug's scope, and finally c) it's just _wasting_ space - spacing needs to be more compact
(also affects trunk of course)
Blocks: 335973
Severity: minor → normal
Keywords: regression
Summary: message headers spaced further apart (2.0a1 vs 1.5.0x) → message headers spaced further apart (2.0a1 vs 1.5.0x) - too much vertical space
Comment 15•18 years ago
|
||
Scott requested a larger margin in bug 335973 comment 17.
Comment 16•18 years ago
|
||
(In reply to comment #15)
> Scott requested a larger margin in bug 335973 comment 17.
(saw that)
and then tested with other font sizes?
Assignee | ||
Comment 17•18 years ago
|
||
I agree there's too much space when running with large fonts on windows. I think we nailed it for regular fonts on windows, but I didn't test large fonts at the time.
Assignee | ||
Comment 18•18 years ago
|
||
Neil, what do you think of something like this? This makes the padding between the addresses fixed at 5px instead of something that scales with the font size.
There's still a little bit more padding than there was in 1.5 for large fonts, but it's better than it is today and I think the little bit of padding is fine.
Normal fonts still have the right amount of padding. Any smaller,and normal fonts start to suffer from being too cramped I think.
If this looks like the right idea to you, I'll make a patch that fixes the seamonkey theme too.
Assignee | ||
Comment 19•18 years ago
|
||
Comment on attachment 252942 [details] [diff] [review]
[fixed branch and trunk]possible fix
Neil see comment 18 :)
Attachment #252942 -
Flags: review?(neil)
Updated•18 years ago
|
Attachment #252942 -
Flags: review?(neil) → review+
Assignee | ||
Comment 20•18 years ago
|
||
as promised, here are the same CSS changes for the suite.
Attachment #253128 -
Flags: superreview?(neil)
Assignee | ||
Updated•18 years ago
|
Attachment #252942 -
Attachment description: possible fix → [fixed branch and trunk]possible fix
Updated•18 years ago
|
Attachment #253128 -
Flags: superreview?(neil)
Attachment #253128 -
Flags: superreview+
Attachment #253128 -
Flags: review?(mnyromyr)
Assignee | ||
Comment 21•18 years ago
|
||
fixed branch and trunk.
Comment 22•18 years ago
|
||
Wayne, since you were seeing this on Windows it would be good for you verify it as well in tomorrow's build.
Comment 23•18 years ago
|
||
SM has not improved AFAICT, modern and classic themes.
TB no improvement - default theme
Comment 24•18 years ago
|
||
(In reply to comment #23)
> SM has not improved AFAICT, modern and classic themes.
> TB no improvement - default theme
Really? I see a clear improvement in both SM and TB trunk.
However, I've decided 5px is still too wide and have implemented a 2px margin in my userChrome.css. My userChrome.css has this now, which is applying the margin slightly differently than seen in the patch:
.headerNameBox, .headerValueBox { margin-bottom: 2px !important; }
.headerName { margin-bottom: 0 !important; }
Comment 25•18 years ago
|
||
Scott, I suggest the patch be changed from 5px to 2px
oops, retest because I made a mistake - discovered I installed 20060109 into 0130 directory :( haven't made that mistake in a long time.
yes TB is improved, but Mike's 2px is far better. for a 5 line header on my display the header height is 1-3/8", 1-1/4" and ~1" respectively for 5em, 5px, 2px. so a gain approaching 1/2" which is fantastic.
SM was tested on w2k, I don't have winXP results ATM.
Assignee | ||
Comment 26•18 years ago
|
||
My testing showed, 2px didn't leave enough spacing for normal sized fonts, things looked pretty compact. I think 5px seemed like a good compromise for both. yes it takes a bit more space than it did in 1.5.0.x for large fonts, but it's a lot less than it was in beta 2 and I thought it looked nice.
Assignee | ||
Comment 27•18 years ago
|
||
I'm running with 2px again at Normal fonts and it is pretty compact, but I could possible be talked into it. Mike, what do you think of 2px at normal fonts?
Comment 28•18 years ago
|
||
OK, I switched over to small fonts for testing [I can't *believe* that requires a Windows reboot]. Oddly, the 2px margin looks a little more cramped at 96dpi than at 125dpi -- does Mozilla scale "px" according to dpi? (I believe CSS calls for this, so I wouldn't be surprised if that's how it's done.)
The worst part is right along the bottom when the last header line is To: -- the underline of the address is almost flush with the border between envelope and message pane, makes that boundary look too busy. So, I'd say stick with the 5px for the default. This is an easy enough customization.
Comment 29•18 years ago
|
||
Comment 30•18 years ago
|
||
Comment 31•18 years ago
|
||
.headerNameBox, .headerValueBox, .headerName { margin-bottom: 0 !important; }
same result using the following:
.headerName, .headerValueBox { margin-bottom: 0 !important; }
Updated•18 years ago
|
Attachment #253128 -
Flags: review?(mnyromyr) → review+
Updated•18 years ago
|
Keywords: fixed-seamonkey1.1.1
You need to log in
before you can comment on or make changes to this bug.
Description
•