From 8c6d549022e6da7ee267e3b4e5e230d446aef5ad Mon Sep 17 00:00:00 2001 From: John Scipione Date: Mon, 25 Nov 2013 23:37:38 -0500 Subject: [PATCH] BStringItem: Set the low color to the list view color instead of the low color, practically this should make little difference, just a cleanup. --- src/kits/interface/StringItem.cpp | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/src/kits/interface/StringItem.cpp b/src/kits/interface/StringItem.cpp index 6902f827b6..73fa7d8bf6 100644 --- a/src/kits/interface/StringItem.cpp +++ b/src/kits/interface/StringItem.cpp @@ -77,14 +77,17 @@ BStringItem::DrawItem(BView *owner, BRect frame, bool complete) rgb_color lowColor = owner->LowColor(); if (IsSelected() || complete) { - if (IsSelected()) { - owner->SetHighColor(ui_color(B_LIST_SELECTED_BACKGROUND_COLOR)); - owner->SetLowColor(owner->HighColor()); - } else - owner->SetHighColor(lowColor); + rgb_color color; + if (IsSelected()) + color = ui_color(B_LIST_SELECTED_BACKGROUND_COLOR); + else + color = owner->ViewColor(); + owner->SetLowColor(color); + owner->SetHighColor(color); owner->FillRect(frame); - } + } else + owner->SetLowColor(owner->ViewColor()); owner->MovePenTo(frame.left + be_control_look->DefaultLabelSpacing(), frame.top + fBaselineOffset);