Closed
Bug 846551
Opened 11 years ago
Closed 11 years ago
The unread count on the chat toolbar button is ugly and unreadable on a Mac retina screen
Categories
(Thunderbird :: Instant Messaging, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 23.0
People
(Reporter: florian, Assigned: Paenglab)
Details
Attachments
(4 files, 2 obsolete files)
9.43 KB,
image/png
|
Details | |
4.66 KB,
image/png
|
Details | |
8.76 KB,
image/png
|
Details | |
3.71 KB,
patch
|
mconley
:
review+
mconley
:
ui-review+
|
Details | Diff | Splinter Review |
See attached screenshot. I'm not completely sure if the red circle is too large or the font size too small (or even both): - I think the circle is too big because I think I'm supposed to still be able to see the icon - I also think the font size is too small because the number is completely unreadable at such a small size (it's half the size in both width and height when actually displayed on the retina screen).
Assignee | ||
Comment 1•11 years ago
|
||
The badge is also on LoDPI to big. I've tweaked some values to let the badge be always 11px * 11px. Under OSX I made the font 0.7em instead of 0.6em. Florian, I gave you f? to check how it looks under HiDPI.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #720310 -
Flags: review?(mconley)
Attachment #720310 -
Flags: feedback?(florian)
Comment 2•11 years ago
|
||
This problem isn't limited to Macs either. The look on Windows (XP), shame on me;) is terrible too! See the Windows screenshot.
Comment 3•11 years ago
|
||
Paenglab - can you please post screenshots of the badges for each platform?
Updated•11 years ago
|
Flags: needinfo?(richard.marti)
Updated•11 years ago
|
Attachment #720310 -
Flags: ui-review?(mconley)
Assignee | ||
Comment 4•11 years ago
|
||
I saw on various Linux with different font height also different badge height because of the setting of 0.6em (it varied on the different platforms between 7.2px and 7.8px). I set now a fixed height of 8px. The other changes are needed to be correct with the fixed height.
Attachment #720310 -
Attachment is obsolete: true
Attachment #720310 -
Flags: ui-review?(mconley)
Attachment #720310 -
Flags: review?(mconley)
Attachment #720310 -
Flags: feedback?(florian)
Attachment #729139 -
Flags: ui-review?(mconley)
Attachment #729139 -
Flags: review?(mconley)
Flags: needinfo?(richard.marti)
Assignee | ||
Comment 5•11 years ago
|
||
Assignee | ||
Comment 6•11 years ago
|
||
Patch unbitrotted after landing of Bug 844599.
Attachment #729139 -
Attachment is obsolete: true
Attachment #729139 -
Flags: ui-review?(mconley)
Attachment #729139 -
Flags: review?(mconley)
Attachment #729773 -
Flags: ui-review?(mconley)
Attachment #729773 -
Flags: review?(mconley)
Comment 7•11 years ago
|
||
Comment on attachment 729773 [details] [diff] [review] patch v2 unbitrotted Review of attachment 729773 [details] [diff] [review]: ----------------------------------------------------------------- Yes, I think this looks far far better. Thank you, Richard!
Attachment #729773 -
Flags: ui-review?(mconley)
Attachment #729773 -
Flags: ui-review+
Attachment #729773 -
Flags: review?(mconley)
Attachment #729773 -
Flags: review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 8•11 years ago
|
||
https://hg.mozilla.org/comm-central/rev/f601c38e6fe7
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 23.0
You need to log in
before you can comment on or make changes to this bug.
Description
•