The default bug view has changed. See this FAQ.

Using lightweight themes causes the tab bar to move down 16px.

RESOLVED FIXED in Thunderbird 25.0

Status

Thunderbird
Mail Window Front End
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: JosiahOne, Assigned: JosiahOne)

Tracking

Trunk
Thunderbird 25.0
All
Mac OS X

Thunderbird Tracking Flags

(thunderbird24+ fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Using lightweight themes causes the tab bar to be pushed down 16px. I know the cause and will have a patch up shortly. Also going to test if this is an issue on all platforms, or just OS X.
This also happens with TB 24. This is onlyvisible by applying the LW-theme. After restart with the LW-theme the margin-bottom is correctly on -6px.

I suppose at TB start without LW-theme the titlebar is hidden and the margin calculation in http://mxr.mozilla.org/comm-central/source/mail/base/content/msgMail3PaneWindow.js#1666 can't calculate the correct margin. Applying then a LW-theme bring then the incorrect margin-bottom. After restart with LW-theme the titlebar is not hidden and the calculation can work correctly.
tracking-thunderbird24: --- → ?
Created attachment 771684 [details] [diff] [review]
Fix.

Richard pretty much had it right, except we want the margin-bottom to be 0px, not -6px.

In fact, why this whole margin-manipulation was ever added is beyond me. That is for drawing the tabs in the titlebar, which we aren't doing yet. Probably some mistake when draw in titlebar was brought over to TB for OS X.

Anyway, this patch fixes the issue by completely removing this margin-bottom modification. Mike, could you take a few minutes to review this?
Attachment #771684 - Flags: review?(mconley)
Status: NEW → ASSIGNED
Comment on attachment 771684 [details] [diff] [review]
Fix.

So, umm. Mike's on vacation. So I'll let Blake do the review instead. ;)
Attachment #771684 - Flags: review?(mconley) → review?(bwinton)
Comment on attachment 771684 [details] [diff] [review]
Fix.

This will break on Windows draw in titlebar. The gap above the tabs has to be 16px (for this is the calculation). Maybe it's better to set in messenger.css on #titlebar a margin-bottom: 0px !important. When draw in titlebar comes to OS X then this can be removed and the calculation is working correct also under OS X.
Attachment #771684 - Flags: review?(bwinton)
Created attachment 771709 [details] [diff] [review]
Fix.

Well, I really would have preferred to use #ifdefs to avoid CSS overrides, but it seems #ifdefs can't be nested in js files. Therefore, I went ahead and just did the CSS override.

Richard, now you can review this. :)
Attachment #771684 - Attachment is obsolete: true
Attachment #771709 - Flags: review?(richard.marti)
Comment on attachment 771709 [details] [diff] [review]
Fix.

Looks good to me. Please add a linefeed in comment to make the line not longer than 80 chars.
Attachment #771709 - Flags: review?(richard.marti) → review+
Created attachment 771784 [details] [diff] [review]
Fix.
Attachment #771709 - Attachment is obsolete: true
Attachment #771784 - Flags: review+
Requesting checkin, though I think our tree will be closed for awhile...
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/490d830d4e1a
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 25.0
tracking-thunderbird24: ? → +
Comment on attachment 771784 [details] [diff] [review]
Fix.

[Triage Comment]
Taking to 24 as the comments say we need it there as well, and it is low risk
Attachment #771784 - Flags: approval-comm-aurora+
https://hg.mozilla.org/releases/comm-aurora/rev/776a4660e156
status-thunderbird24: --- → fixed
You need to log in before you can comment on or make changes to this bug.