From 75c8f832f3a4d3fa7cfd5b40edf1eebe3eda6ae3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jaakko=20Ker=C3=A4nen?= <jaakko.keranen@iki.fi>
Date: Sat, 27 Feb 2016 19:56:21 +0200
Subject: [PATCH] UI|Home: Keep action buttons hidden until needed

---
 .../client/src/ui/home/homeitemwidget.cpp     | 36 ++++++++++++++-----
 1 file changed, 28 insertions(+), 8 deletions(-)

diff --git a/doomsday/apps/client/src/ui/home/homeitemwidget.cpp b/doomsday/apps/client/src/ui/home/homeitemwidget.cpp
index ad27aeb12f..45ad7cbcb2 100644
--- a/doomsday/apps/client/src/ui/home/homeitemwidget.cpp
+++ b/doomsday/apps/client/src/ui/home/homeitemwidget.cpp
@@ -19,6 +19,7 @@
 #include "ui/home/homeitemwidget.h"
 
 #include <de/SequentialLayout>
+#include <QTimer>
 
 using namespace de;
 
@@ -66,6 +67,7 @@ DENG_GUI_PIMPL(HomeItemWidget)
     DotPath selectedBgColor   { "background" };
     DotPath textColor         { "text" };
     DotPath selectedTextColor { "text" };
+    QTimer buttonHideTimer;
 
     Instance(Public *i) : Base(i)
     {
@@ -82,6 +84,12 @@ DENG_GUI_PIMPL(HomeItemWidget)
 
         background->setBehavior(Focusable);
         background->addEventHandler(new ClickHandler(self));
+
+        buttonHideTimer.setSingleShot(true);
+        QObject::connect(&buttonHideTimer, &QTimer::timeout, [this] ()
+        {
+            for(auto *btn : buttons) btn->hide();
+        });
     }
 
     ~Instance()
@@ -106,6 +114,12 @@ DENG_GUI_PIMPL(HomeItemWidget)
     {
         if(!buttonsWidth) return;
 
+        if(show)
+        {
+            buttonHideTimer.stop();
+            for(auto *button : buttons) button->show();
+        }
+
         TimeDelta const SPAN = .5;
         if(show)
         {
@@ -114,6 +128,8 @@ DENG_GUI_PIMPL(HomeItemWidget)
         else
         {
             labelRightMargin->set(-rule("halfunit"), SPAN);
+            buttonHideTimer.setInterval(SPAN.asMilliSeconds());
+            buttonHideTimer.start();
         }
     }
 
@@ -169,16 +185,19 @@ LabelWidget &HomeItemWidget::label()
 
 void HomeItemWidget::setSelected(bool selected)
 {
-    d->selected = selected;
-    if(selected)
-    {
-        d->showButtons(true);
-    }
-    else
+    if(d->selected != selected)
     {
-        d->showButtons(false);
+        d->selected = selected;
+        if(selected)
+        {
+            d->showButtons(true);
+        }
+        else
+        {
+            d->showButtons(false);
+        }
+        d->updateColors();
     }
-    d->updateColors();
 }
 
 bool HomeItemWidget::isSelected() const
@@ -211,5 +230,6 @@ void HomeItemWidget::addButton(ButtonWidget *button)
 
     d->buttons << button;
     d->label->add(button);
+    button->hide();
     d->updateButtonLayout();
 }