Under Windows Classic Theme Thunderbird 17 title bar is same colour as window background

RESOLVED FIXED in Thunderbird 20.0

Status

Thunderbird
Theme
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: Virgo Pärna, Assigned: Paenglab)

Tracking

17 Branch
Thunderbird 20.0
All
Windows XP

Thunderbird Tracking Flags

(thunderbird18 fixed, thunderbird19 fixed, thunderbird20 fixed, thunderbird-esr1718+ fixed)

Details

(Whiteboard: [workaround in comment 1 set mail.tabs.drawInTitlebar to false][GS], URL)

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

5 years ago
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/17.0 Firefox/17.0
Build ID: 20121119183901

Steps to reproduce:

Updated Thunderbird to version 17.


Actual results:

Under Windows XP with classic Windows theme Thunderbirds window title bar does not have correct colour (blue when active, dark gray when inactive).



Expected results:

Window title bar should be blue, when active and dark gray, when not inactive.
Component: Untriaged → Theme
(Assignee)

Comment 1

5 years ago
Please can you provide a screenshot? Have you disabled the tabs? With hidden menu bar the tabs would draw in the title bar, then the title bar color change would make the tabs hard readable. To become the color changing title bar back you can do this: Go to Tools/Options.../Advanced/General and click on Config Editor... Then say yes to the warning and search for mail.tabs.drawInTitlebar. Double click it to change to false.
(Reporter)

Comment 2

5 years ago
Created attachment 683679 [details]
Screenshot

Here is the requested screenshot. Tabs are not disabled, but they are below the menu bar (isn't it default on Windows XP?). And menu bar is wisible. Setting mail.tabs.drawInTitlebar to false helped, but it should be default, when tabs are below menu bar.

Comment 3

5 years ago
Maybe also reported here :
https://getsatisfaction.com/mozilla_messaging/topics/thunderbird_17_0_horrible_bright_blue_menu_bar
and
http://forums.mozfr.org/viewtopic.php?f=4&t=109356

Updated

5 years ago

Comment 4

5 years ago
For info I'm seeing exactly the same effect as reported in comment 2 on Windows 7 64-bit.

Setting mail.tabs.drawInTitlebar to false makes the title bar background correct and contain the correct text.

Setting mail.tabs.drawInTitlebar to true makes the title bar background incorrect and contain no text (so you can't see which account you are looking at).

Comment 5

5 years ago
Actually, after reading 

https://developer.mozilla.org/en-US/docs/Mozilla/Preferences/Preference_reference/mail.tabs.drawInTitlebar 

it's clear that the intended "functionality" of this option (to draw the tabs in the title bar when set to "true") isn't working either.
(Assignee)

Comment 6

5 years ago
When you hide the menu bar the tabs are in title bar. You can then access the menu bar either by pressing the ALT key or with the mouse on the new button with the three bars, which should be shown on the right of the main tool bar, and then use the Application menu.

Comment 7

5 years ago
As I recall, windows classic theme acted like this bugs complaint.
I don't have that laptop availablr to check now, but the theme that worked correctly had "XP" in the theme name.

Comment 8

5 years ago
I see this for a long time but I thought it was already filed :)
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 9

5 years ago
The same (analogously) is true even with Classic View set in the Windows XP UI.  The point is that since this gratuitous UI change, we can no longer tell whether Thunderbird is the active app by looking at its title bar; in effect it no longer HAS a title bar.

Please provide a preference to turn off this change (and all future UI changes, preferably before releasing them!!)
(Assignee)

Comment 10

5 years ago
(In reply to John David Galt from comment #9)
> Please provide a preference to turn off this change (and all future UI
> changes, preferably before releasing them!!)

Please check comment 1.
John:

Did flipping mail.tabs.drawInTitlebar to false fix the problem?

-Mike

Comment 12

5 years ago
Yes, thanks, that has fixed it for me.
I just did some tests, I can confirm this affects users of the Windows Classic theme on Windows XP and Windows 7.
tracking-thunderbird-esr17: --- → ?
Hardware: x86 → All
Summary: Under Windows XP Thunderbird 17 title bar is same colour as window background instead of blue, when active and dark gray, when not active. → Under Windows Classic Theme Thunderbird 17 title bar is same colour as window background
Whiteboard: [GS] → [workaround in comment 1 set mail.tabs.drawInTitlebar to false][GS]
(Assignee)

Comment 14

5 years ago
Created attachment 686201 [details] [diff] [review]
proposed fix

This patch adds only on inactive window the inactivecaption color. I added also a rule which makes the menu text visible on inactive background color.

This is only a part of the fix. Maybe we should have a logic where the draw in title bar is on false when the menu is not on auto hide. Aero and Win8 default should stay as they are now because the menu bar is drawn below the tabs. But this would be out of my skills.

Or when draw in title bar is on true then the menu bar is on all this systems below the tabs. Draw in title bar is no system standard then we could also break the menu position standard.
Attachment #686201 - Flags: ui-review?(bwinton)
Attachment #686201 - Flags: review?(bwinton)
(Assignee)

Comment 15

5 years ago
Created attachment 686204 [details]
screenshot with patch in action (on top active on bottom inactive)
Comment on attachment 686201 [details] [diff] [review]
proposed fix

>+++ b/mail/themes/qute/mail/mailWindow1.css
>@@ -402,24 +402,29 @@
>+#messengerWindow[tabsintitlebar]:-moz-system-metric(windows-classic)
>+menubar > menu:-moz-window-inactive {

Since this is a continuation of the previous line and not a new selector, I think we want to indent it a little.  (I'll let you pick whether 2 or 4 spaces looks better.)

Aside from that, it looks good, so ui-r=me, and r=me!

Thanks,
Blake.
Attachment #686201 - Flags: ui-review?(bwinton)
Attachment #686201 - Flags: ui-review+
Attachment #686201 - Flags: review?(bwinton)
Attachment #686201 - Flags: review+
(Assignee)

Comment 17

5 years ago
Created attachment 690450 [details] [diff] [review]
patch for check-in

I've chosen 2px indentation.

[Approval Request Comment]
This could help to differentiate between active and inactive window under Classic also for ESR17.
Attachment #686201 - Attachment is obsolete: true
Attachment #690450 - Flags: approval-comm-esr17?
Attachment #690450 - Flags: approval-comm-beta?
Attachment #690450 - Flags: approval-comm-aurora?
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
Attachment #690450 - Flags: approval-comm-esr17?
Attachment #690450 - Flags: approval-comm-esr17+
Attachment #690450 - Flags: approval-comm-beta?
Attachment #690450 - Flags: approval-comm-beta+
Attachment #690450 - Flags: approval-comm-aurora?
Attachment #690450 - Flags: approval-comm-aurora+
tracking-thunderbird-esr17: ? → 18+
https://hg.mozilla.org/comm-central/rev/5c47002e482b
https://hg.mozilla.org/releases/comm-aurora/rev/65e60d5672e0
https://hg.mozilla.org/releases/comm-beta/rev/1bdd7893efd9
https://hg.mozilla.org/releases/comm-esr17/rev/359ef8f9ace5
Assignee: nobody → richard.marti
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 20.0
status-thunderbird18: --- → fixed
status-thunderbird19: --- → fixed
status-thunderbird20: --- → fixed
status-thunderbird-esr17: --- → fixed

Updated

5 years ago
Duplicate of this bug: 821645

Comment 20

4 years ago
I've just updated thunderbird from 10.0.12esr to 17.0.2esr and the title bar
was wrong until I set mail.tabs.drawInTitlebar to false. By wrong I mean
a)  colour didn't change when window changes between active/inactive
b)  no title text
c)  no thunderbird icon at the left
This is despite the tracking flags for this bug stating status-thunderbird-esr17: Fixed

I'm running 32-bit Windows XP pro using the classic theme.

Comment 21

3 years ago
For some reason this got fixed for me only now with latest nightly of 2014-04-29. It works regardless of draw_in_titlebar value.
(Assignee)

Comment 22

3 years ago
The reason is bug 953204.

Comment 23

3 years ago
That's weird, bug 953204 seems to have broken it for me.  The menubar and tabbar background are not supposed to be the same.
(Assignee)

Comment 24

3 years ago
This bug shouldn't be used for this issue.
John, please can you file a new bug for your problem, if it's on Daily or latest 31.0a2? Please also include screenshots.
You need to log in before you can comment on or make changes to this bug.