Closed
Bug 1272469
Opened 8 years ago
Closed 8 years ago
font colors from installed theme used inappropriately on the toolbars
Categories
(Thunderbird :: Theme, defect)
Tracking
(thunderbird46 wontfix, thunderbird47 fixed, thunderbird48 fixed, thunderbird49 fixed, thunderbird_esr4546+ fixed)
RESOLVED
FIXED
Thunderbird 49.0
People
(Reporter: aceman, Assigned: Paenglab)
Details
(Keywords: regression)
Attachments
(4 files)
257.78 KB,
image/png
|
Details | |
2.46 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
2.45 KB,
patch
|
Paenglab
:
review+
rkent
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
2.46 KB,
patch
|
Paenglab
:
review+
rkent
:
approval-comm-beta+
rkent
:
approval-comm-esr45+
|
Details | Diff | Splinter Review |
I attach a screenshot of my toolbar in TB45 that uses a lightweight theme. In the image I marked the problems. On the main toolbar, which now is more white than it used to be (I think this is Firefox's fault), the text of the buttons is not very readable. On the quick filter bar, the background image from the theme is not used, but the text color from theme is. That does not match with the grey default background and the text is not very readable. In this case, when default background is used, also default text color should be used. It is already done correctly on the pushed Tags button in the image. Can any of these problems be fixed? This was not a problem in TB38.
Assignee | ||
Comment 1•8 years ago
|
||
The problem is, the LW-theme has set a light text color (in my opinion too light for the theme background). For every text in the main toolbar it's the LW-theme fault. Can you give me the name of the LW-theme so I can test it? What I see as a TB bug is that the QFB toolbar uses the LW-theme text color.
(In reply to Richard Marti (:Paenglab) from comment #1) > The problem is, the LW-theme has set a light text color (in my opinion too > light for the theme background). For every text in the main toolbar it's the > LW-theme fault. Can you give me the name of the LW-theme so I can test it? https://addons.mozilla.org/en-Us/thunderbird/addon/dream-swept/ But the theme text was fine before Firefox started to artifically lighten the background on the toolbars. This crept into TB too now. You can see in TB on the main menu bar the image is darked and the text is better on it (well and the menu also has shadow under the strings). The lightening/whitening on toolbars in Firefox has ruined most themes in my opinion. They do not look good now (and with enough contrast). > What I see as a TB bug is that the QFB toolbar uses the LW-theme text color. Yes.
The save QFB problem also appears in the attachment block in message view (the one saying X attachments: <name> <size>"). Text color from theme, background image not.
Assignee | ||
Comment 4•8 years ago
|
||
This patch fixes the text color issues with LW-themes on QFB- and attachments toolbar.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8752576 -
Flags: review?(acelists)
Comment on attachment 8752576 [details] [diff] [review] Bug1272469.patch Review of attachment 8752576 [details] [diff] [review]: ----------------------------------------------------------------- Works for me on Linux, thanks!
Attachment #8752576 -
Flags: review?(acelists) → review+
status-thunderbird_esr45:
--- → affected
tracking-thunderbird_esr45:
--- → ?
Keywords: checkin-needed
Assignee | ||
Comment 6•8 years ago
|
||
https://hg.mozilla.org/comm-central/rev/d69dbb3c065d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 49.0
Assignee | ||
Comment 7•8 years ago
|
||
Patch that applies on aurora [Approval Request Comment] User impact if declined: unreadable buttons in QFB. anf attachments toolbars with LW-themes enabled Testing completed (on c-c, etc.): on c-c Risk to taking this patch (and alternatives if risky): low
Attachment #8752716 -
Flags: review+
Attachment #8752716 -
Flags: approval-comm-aurora?
Assignee | ||
Comment 8•8 years ago
|
||
Patch that applies on esr45 [Approval Request Comment] User impact if declined: unreadable buttons in QFB. anf attachments toolbars with LW-themes enabled Testing completed (on c-c, etc.): on c-c Risk to taking this patch (and alternatives if risky): low
Attachment #8752718 -
Flags: review+
Attachment #8752718 -
Flags: approval-comm-esr45?
Assignee | ||
Comment 9•8 years ago
|
||
I'm not asking for beta because I don't know if it's still based on 45 or higher. Depending on on where it bases you can use either patches.
Comment 10•8 years ago
|
||
Even though this is really late, it seems low risk so let's push it to get it into our release candidate on the beta channel.
Comment 11•8 years ago
|
||
Comment on attachment 8752718 [details] [diff] [review] Bug1272469-esr45.patch [Triage Comment] http://hg.mozilla.org/releases/comm-beta/rev/03e9226caa21 http://hg.mozilla.org/releases/comm-esr45/rev/8ff3cb312f47 Also pushed to beta, which will be for TB 47.
Attachment #8752718 -
Flags: approval-comm-esr45?
Attachment #8752718 -
Flags: approval-comm-esr45+
Attachment #8752718 -
Flags: approval-comm-beta+
Comment 12•8 years ago
|
||
Comment on attachment 8752716 [details] [diff] [review] Bug1272469-aurora.patch https://hg.mozilla.org/releases/comm-aurora/rev/b1304880a600
Attachment #8752716 -
Flags: approval-comm-aurora? → approval-comm-aurora+
Updated•8 years ago
|
status-thunderbird46:
--- → wontfix
status-thunderbird47:
--- → fixed
status-thunderbird48:
--- → fixed
status-thunderbird49:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•