Fixed|libappfw|PopupMenuWidget: Inverted popup menu items colors

skyjake · Aug 25, 2015 · f3d27f9 · f3d27f9
1 parent f3e5d06
commit f3d27f9
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/doomsday/sdk/libappfw/src/widgets/popupmenuwidget.cpp b/doomsday/sdk/libappfw/src/widgets/popupmenuwidget.cpp
@@ -205,8 +205,9 @@ DENG_GUI_PIMPL(PopupMenuWidget)
 
     void setButtonColors(ButtonWidget &button)
     {
-        button.setTextColor(!infoStyle? "text" : "inverted.text");
-        button.setHoverTextColor(!infoStyle? "inverted.text" : "text",
+        bool const hovering = (hover == &button);
+        button.setTextColor(!hovering ^ infoStyle? "text" : "inverted.text");
+        button.setHoverTextColor(!hovering ^ infoStyle? "inverted.text" : "text",
                                  ButtonWidget::ReplaceColor);
     }
 
@@ -463,9 +464,10 @@ void PopupMenuWidget::panelClosing()
 
     if(d->hover)
     {
-        d->hover->setTextModulationColorf(style().colors().colorf(!d->infoStyle? "text" : "inverted.text"));
-        d->hover->setImageColor(style().colors().colorf(!d->infoStyle? "text" : "inverted.text"));
+        auto &btn = *d->hover;
         d->hover = 0;
+        d->setButtonColors(btn);
+        btn.setImageColor(style().colors().colorf(!d->infoStyle? "text" : "inverted.text"));
         requestGeometry();
     }