Opened 8 years ago

Closed 8 years ago

#12596 closed bug (fixed)

[Deskbar] TimeView has white background

Reported by: diver Owned by: axeld
Priority: normal Milestone: R1
Component: Applications/Deskbar Version: R1/Development
Keywords: Cc: looncraz
Blocked By: Blocking:
Platform: All

Description

Attachments (2)

deskbar.png (36.2 KB ) - added by diver 8 years ago.
Deskbar-TimeViewText.patch (957 bytes ) - added by looncraz 8 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 by looncraz, 8 years ago

Screenshot?

Because all this results in a proper background AFAICT...

comment:2 by diver, 8 years ago

To make it a bit more visible I made Menu item text color lighter.

by diver, 8 years ago

Attachment: deskbar.png added

comment:3 by looncraz, 8 years ago

All I see is font-hinting on menu text that seems light (odd since the low color should match the view color). It would almost suggest that the low color is not being set, but I don't see that issue here at all, no matter what I do.

Try changing the menu text color, making sure you are completely up to date, and playing with anti-aliasing settings... otherwise the issue would seem like it would be more visible on dark backgrounds with dark text....

in reply to:  3 comment:4 by looncraz, 8 years ago

Replying to looncraz:

All I see is font-hinting on menu text that seems light (odd since the low color should match the view color). It would almost suggest that the low color is not being set, but I don't see that issue here at all, no matter what I do.

Try changing the menu text color, making sure you are completely up to date, and playing with anti-aliasing settings... otherwise the issue would seem like it would be more visible on dark backgrounds with dark text....

Wait, I get it, LOL! The LowUIColor() isn't using the tint value from the view, which is making it slightly lighter. This effect is really only noticeable with Glyph hinting off, it seems. I submit a patch tomorrow that should remedy that.

by looncraz, 8 years ago

Attachment: Deskbar-TimeViewText.patch added

comment:5 by looncraz, 8 years ago

patch: 01

comment:6 by waddlesplash, 8 years ago

Resolution: fixed
Status: newclosed

Applied in hrev50057. Thanks!

Note: See TracTickets for help on using tickets.