Closed
Bug 1052071
Opened 10 years ago
Closed 10 years ago
Style of quota indicator not as nice as it used to
Categories
(Thunderbird :: Theme, defect)
Tracking
(thunderbird32 fixed, thunderbird33 fixed, thunderbird34 fixed, thunderbird_esr3132+ fixed)
RESOLVED
FIXED
Thunderbird 34.0
People
(Reporter: Eduard.Braun2, Assigned: Paenglab)
Details
(Keywords: regression)
Attachments
(2 files)
1.72 KB,
image/png
|
Details | |
1.94 KB,
patch
|
jsbruner
:
review+
standard8
:
approval-comm-aurora+
standard8
:
approval-comm-beta+
standard8
:
approval-comm-esr31+
|
Details | Diff | Splinter Review |
While Thunderbird 24.7.0 showed a nice native-style progress indicator on Windows 7, the the style of the quota indicator in Thunderbird 31 seems broken (or at least it looks pretty ugly in case it was restyled on purpose). See the attached screenshot for a direct comparison: Left: Thunderbird 24.7.0 (correctly centered with native-style 3D effect) Right: Thunderbird 31.0 (shifted up, boring flat look, unnecessarily increased status-bar-height)
Comment 1•10 years ago
|
||
http://mxr.mozilla.org/comm-central/source/mail/themes/shared/mail/mailWindow1.css#90 vs. http://mxr.mozilla.org/comm-esr24/source/mail/themes/windows/mail/mailWindow1.css#374
Keywords: regression
Summary: Style of quota indicator is broken → Style of quota indicator not as nice as it used to
Assignee | ||
Comment 2•10 years ago
|
||
On moving to the shared files the -moz-appearance: none; slipped in for all platforms. This patch applies it only on Linux and OS X.
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8471080 -
Flags: review?(josiah)
Updated•10 years ago
|
Attachment #8471080 -
Flags: review?(josiah) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 3•10 years ago
|
||
https://hg.mozilla.org/comm-central/rev/43731259d967
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 34.0
Comment 4•10 years ago
|
||
Do we want to backport this to TB 31? If so, please request approval.
tracking-thunderbird_esr31:
--- → ?
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8471080 [details] [diff] [review] quotaMeter.patch [Approval Request Comment] User impact if declined: Not looking correct quota meter Testing completed (on c-c, etc.): in c-c Risk to taking this patch (and alternatives if risky): low, only cSS
Attachment #8471080 -
Flags: approval-comm-esr31?
Attachment #8471080 -
Flags: approval-comm-beta?
Attachment #8471080 -
Flags: approval-comm-aurora?
Updated•10 years ago
|
Attachment #8471080 -
Flags: approval-comm-esr31?
Attachment #8471080 -
Flags: approval-comm-esr31+
Attachment #8471080 -
Flags: approval-comm-beta?
Attachment #8471080 -
Flags: approval-comm-beta+
Attachment #8471080 -
Flags: approval-comm-aurora?
Attachment #8471080 -
Flags: approval-comm-aurora+
Comment 6•10 years ago
|
||
https://hg.mozilla.org/releases/comm-aurora/rev/1996c228ddd2
status-thunderbird33:
--- → fixed
Comment 7•10 years ago
|
||
https://hg.mozilla.org/releases/comm-beta/rev/775e3084c6dc
status-thunderbird32:
--- → fixed
Comment 8•10 years ago
|
||
https://hg.mozilla.org/releases/comm-esr31/rev/387081b6cde5
status-thunderbird_esr31:
--- → fixed
Updated•10 years ago
|
status-thunderbird34:
--- → fixed
Assignee | ||
Updated•9 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•