From 054c81664abdc380c5384b132ad36ce4af8fdc06 Mon Sep 17 00:00:00 2001 From: looncraz Date: Sat, 9 Jan 2016 16:08:42 -0600 Subject: [PATCH] MediaPlayer PositionToolTip Color Previously the layout would crush the default colors of BStringView preventing BStringView from calling AdoptParentColors() on its own, so we must call it manually. In addition, the default tooltip view should fully adopt tooltip colors so that any colors will default to the desired foreground color (which is the same as the tooltip text color). Fixes #12573. Signed-off-by: Jessica Hamilton --- src/apps/mediaplayer/interface/PositionToolTip.cpp | 1 + src/kits/interface/ToolTipManager.cpp | 1 + 2 files changed, 2 insertions(+) diff --git a/src/apps/mediaplayer/interface/PositionToolTip.cpp b/src/apps/mediaplayer/interface/PositionToolTip.cpp index 667baae679..698727ba70 100644 --- a/src/apps/mediaplayer/interface/PositionToolTip.cpp +++ b/src/apps/mediaplayer/interface/PositionToolTip.cpp @@ -30,6 +30,7 @@ public: virtual void AttachedToWindow() { BStringView::AttachedToWindow(); + AdoptParentColors(); Update(-1, -1); } diff --git a/src/kits/interface/ToolTipManager.cpp b/src/kits/interface/ToolTipManager.cpp index 7f4d8a5da9..ad0f8024f8 100644 --- a/src/kits/interface/ToolTipManager.cpp +++ b/src/kits/interface/ToolTipManager.cpp @@ -67,6 +67,7 @@ ToolTipView::ToolTipView(BToolTip* tip) { fToolTip->AcquireReference(); SetViewUIColor(B_TOOL_TIP_BACKGROUND_COLOR); + SetHighUIColor(B_TOOL_TIP_TEXT_COLOR); BGroupLayout* layout = new BGroupLayout(B_VERTICAL); layout->SetInsets(5, 5, 5, 5);