Status bar: Hide the space free bar when size is unknown

[dolphin.git] / src / statusbar / statusbarspaceinfo.cpp

diff --git a/src/statusbar/statusbarspaceinfo.cpp b/src/statusbar/statusbarspaceinfo.cpp
index ea8b4704c02f6a9ed9e14d15882081fadbefffe4..acffcf69dba126a8cbf423ad708f74be111cbe8d 100644 (file)
--- a/src/statusbar/statusbarspaceinfo.cpp
+++ b/src/statusbar/statusbarspaceinfo.cpp
@@ -22,67 +22,119 @@
 
 #include "spaceinfoobserver.h"
 
 
 #include "spaceinfoobserver.h"
 

-#include <KLocale>
-#include <KIO/Job>
+#include <KLocalizedString>
+#include <KNS3/KMoreToolsMenuFactory>

 
 

-#include <QKeyEvent>
+#include <QMouseEvent>

 
 StatusBarSpaceInfo::StatusBarSpaceInfo(QWidget* parent) :
     KCapacityBar(KCapacityBar::DrawTextInline, parent),
 
 StatusBarSpaceInfo::StatusBarSpaceInfo(QWidget* parent) :
     KCapacityBar(KCapacityBar::DrawTextInline, parent),

-    m_observer(0)
+    m_observer(nullptr)

 {
 {

+    setCursor(Qt::PointingHandCursor);

 }
 
 StatusBarSpaceInfo::~StatusBarSpaceInfo()
 {
 }
 
 }
 
 StatusBarSpaceInfo::~StatusBarSpaceInfo()
 {
 }
 

-void StatusBarSpaceInfo::setUrl(const KUrl& url)
+void StatusBarSpaceInfo::setShown(bool shown)
+{
+    m_shown = shown;
+    if (!m_shown) {
+        hide();
+        m_ready = false;
+    }
+}
+
+void StatusBarSpaceInfo::setUrl(const QUrl& url)

 {
     if (m_url != url) {
         m_url = url;
 {
     if (m_url != url) {
         m_url = url;

+        m_ready = false;

         if (m_observer) {
         if (m_observer) {

-            m_observer->setUrl(url);
+            m_observer.reset(new SpaceInfoObserver(m_url, this));
+            connect(m_observer.data(), &SpaceInfoObserver::valuesChanged, this, &StatusBarSpaceInfo::slotValuesChanged);

         }
     }
 }
 
         }
     }
 }
 

-KUrl StatusBarSpaceInfo::url() const
+QUrl StatusBarSpaceInfo::url() const

 {
     return m_url;
 }
 
 {
     return m_url;
 }
 

+void StatusBarSpaceInfo::update()
+{
+    if (m_observer) {
+        m_observer->update();
+    }
+}
+

 void StatusBarSpaceInfo::showEvent(QShowEvent* event)
 {
 void StatusBarSpaceInfo::showEvent(QShowEvent* event)
 {

-    KCapacityBar::showEvent(event);
-    m_observer.reset(new SpaceInfoObserver(m_url, this));
-    slotValuesChanged();
-    connect(m_observer.data(), SIGNAL(valuesChanged()), this, SLOT(slotValuesChanged()));
+    if (m_shown) {
+        if (m_ready) {
+            KCapacityBar::showEvent(event);
+        }
+
+        if (m_observer.isNull()) {
+            m_observer.reset(new SpaceInfoObserver(m_url, this));
+            connect(m_observer.data(), &SpaceInfoObserver::valuesChanged, this, &StatusBarSpaceInfo::slotValuesChanged);
+        }
+    }

 }
 
 void StatusBarSpaceInfo::hideEvent(QHideEvent* event)
 {
 }
 
 void StatusBarSpaceInfo::hideEvent(QHideEvent* event)
 {

-    m_observer.reset();
+    if (m_ready) {
+        m_observer.reset();
+        m_ready = false;
+    }

     KCapacityBar::hideEvent(event);
 }
 
     KCapacityBar::hideEvent(event);
 }
 

+void StatusBarSpaceInfo::mousePressEvent(QMouseEvent* event)
+{
+    if (event->button() == Qt::LeftButton) {
+        // Creates a menu with tools that help to find out more about free
+        // disk space for the given url.
+
+        // Note that this object must live long enough in case the user opens
+        // the "Configure..." dialog
+        KMoreToolsMenuFactory menuFactory(QStringLiteral("dolphin/statusbar-diskspace-menu"));
+        menuFactory.setParentWidget(this);
+        auto menu = menuFactory.createMenuFromGroupingNames(
+            { "disk-usage", "more:", "disk-partitions" }, m_url);
+
+        menu->exec(QCursor::pos());
+    }
+}
+

 void StatusBarSpaceInfo::slotValuesChanged()
 {
     Q_ASSERT(m_observer);
     const quint64 size = m_observer->size();
 void StatusBarSpaceInfo::slotValuesChanged()
 {
     Q_ASSERT(m_observer);
     const quint64 size = m_observer->size();

-    if (size == 0) {
-        setText(i18nc("@info:status", "Unknown size"));
-        setValue(0);
-        update();
+
+    if (!m_shown || size == 0) {
+        hide();
+        return;
+    }
+
+    m_ready = true;
+
+    const quint64 available = m_observer->available();
+    const quint64 used = size - available;
+    const int percentUsed = qRound(100.0 * qreal(used) / qreal(size));
+
+    setText(i18nc("@info:status Free disk space", "%1 free", KIO::convertSize(available)));
+    setUpdatesEnabled(false);
+    setValue(percentUsed);
+    setUpdatesEnabled(true);
+
+    if (!isVisible()) {
+        show();

     } else {
     } else {

-        const quint64 available = m_observer->available();
-        const quint64 used = size - available;
-        const int percentUsed = qRound(100.0 * qreal(used) / qreal(size));
-
-        setText(i18nc("@info:status Free disk space", "%1 free", KIO::convertSize(available)));
-        setUpdatesEnabled(false);
-        setValue(percentUsed);
-        setUpdatesEnabled(true);

         update();
     }
 }
         update();
     }
 }