Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#14587 closed bug (fixed)

[HaikuDepot] wrong background color for email and webpage icons

Reported by: diver Owned by: stippi
Priority: normal Milestone: R1/beta2
Component: Applications/HaikuDepot Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

Attachments (1)

haikudepot.png (42.6 KB ) - added by diver 6 years ago.

Download all attachments as: .zip

Change History (5)

by diver, 6 years ago

Attachment: haikudepot.png added

comment:1 by apl-haiku, 5 years ago

I have investigated the HVIF data and the icons are fine; no shaded background there and the icon looks really good.

The view class that renders the icons is BitmapView and it draws a background of the low colour. This class is used in different places in HD, but in this case with PackageInfoView the parent window is using the B_PANEL_BACKGROUND_COLOR (1) as the background color with a tint of kContentTint.

BitmapView "adopts" the colours of the parent view in BitmapView::AllAttached but here the parent seems not to have the tint. I suspect that the actual parent is an instance of BGroupView. It seems that in PackageInfoView::AllAttached an attempt is made to relay the top-level colours down into the children views such that perhaps the BitmapView might pick it up?

In any case, when the BitmapView::DrawBackground is invoked it has the right UI colour for low, but is missing the tint which is why it renders with a slightly different saturation compared to the surrounding window.

I am a bit confused where the low UI colour might be being set elsewhere. I did override BitmapView::SetLowUIColor but this didn't get hit.

I haven't been able to track this one down yet.

in reply to:  1 comment:2 by X512, 5 years ago

Replying to apl-haiku:

BitmapView "adopts" the colours of the parent view in BitmapView::AllAttached but here the parent seems not to have the tint.

With recent changes Haiku supports transparent views, no adopting is needed. You need to set B_TRANSPARENT_BACKGROUND view flag and SetViewColor(B_TRANSPARENT_COLOR).

comment:3 by waddlesplash, 5 years ago

Resolution: fixed
Status: newclosed

Fixed in hrev54068.

comment:4 by nielx, 5 years ago

Milestone: UnscheduledR1/beta2

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

Note: See TracTickets for help on using tickets.