Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#12576 closed bug (fixed)

Static colour of legend of ActivityMonitor

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

Description

This is hrev50010.

After the big SetColour merge, the text of the legend of ActivityMonitor does no longer change with the Desktop background colour. It used to work like with the icon labels: white on a darker background, black on a brighter background. Now it stays black. See:

Legend always black

Attachments (3)

ActivityMonitor.png (123.0 KB ) - added by humdinger 9 years ago.
Legend always black
Tracker-Desktop-Icons.patch (1021 bytes ) - added by looncraz 9 years ago.
ControlLook-DrawLabel.patch (1.3 KB ) - added by looncraz 9 years ago.
CORRECT PATCH

Download all attachments as: .zip

Change History (12)

by humdinger, 9 years ago

Attachment: ActivityMonitor.png added

Legend always black

comment:1 by diver, 9 years ago

Cc: looncraz added
Milestone: UnscheduledR1

comment:2 by looncraz, 9 years ago

I'll fix this in a few hours, I know exactly what I did here :p

by looncraz, 9 years ago

Attachment: Tracker-Desktop-Icons.patch added

comment:3 by looncraz, 9 years ago

patch: 01

by looncraz, 9 years ago

Attachment: ControlLook-DrawLabel.patch added

CORRECT PATCH

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

Replying to looncraz:

Sorry, this is the wrong patch :-/

Second time I've done that...

comment:5 by jessicah, 9 years ago

With these patches, it's still not sensitive to the background image, from what I can tell. One actually has to change the colour sliders to effect a change in the text labels. Will see what an older install does though...

comment:6 by jessicah, 9 years ago

Both patches applied in hrev50023. At the very least, these don't make the situation worse :p One can manually change the background colour sliders to change the text colour on the Desktop.

in reply to:  5 comment:7 by looncraz, 9 years ago

Replying to jessicah:

With these patches, it's still not sensitive to the background image, from what I can tell. One actually has to change the colour sliders to effect a change in the text labels. Will see what an older install does though...

I don't think it was ever sensitive to the background image itself, that would be a rather complex capability involving image sampling while also taking scaling/position of the background image into account... or just using some alpha :p

comment:8 by jessicah, 9 years ago

Resolution: fixed
Status: newclosed

Alright, I'll close the ticket. If humdinger thinks more work is required, then can revisit ;-)

comment:9 by humdinger, 9 years ago

Thanks for the fix! That's the same behaviour as before, i.e. it depends on the workspace's background colour.

Note: See TracTickets for help on using tickets.