On 13/01/2014 23:30, John Scipione wrote: > On Mon, Jan 13, 2014 at 4:52 PM, François Revol <revol@xxxxxxx> wrote: >> I noticed BStringView checks for the parent color and uses ui_color() as >> a fallback. >> I propose we also set the HighColor to ui_color(B_PANEL_TEXT_COLOR), but >> I'm not sure in which cases. >> I'd say if (color == ui_color(B_PANEL_BACKGROUND_COLOR)) (either it >> wasn't set, or the parent's is panel bg). >> >> Comments? > > What exactly is the problem because the default high and low colors > set in BView and are black and white respectively. You only want to > use the B_PANEL_TEXT_COLOR and B_PANEL_BACKGROUND_COLOR if you are > drawing a panel, which you can't say by default. > Well, some themes do set B_PANEL_BACKGROUND_COLOR to black... François.