From: Peter Penz Date: Sun, 10 Oct 2010 17:11:55 +0000 (+0000) Subject: Update the statusbar also during loading the items, not only after the loading has... X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/4ad4d4785cb1ad88351711de0ec0c1f0e4e5185a Update the statusbar also during loading the items, not only after the loading has been completed. This fixes the issue that no current item-count is shown when doing a search. svn path=/trunk/KDE/kdebase/apps/; revision=1184520 --- diff --git a/src/dolphinviewcontainer.cpp b/src/dolphinviewcontainer.cpp index aa20409ba..da7466947 100644 --- a/src/dolphinviewcontainer.cpp +++ b/src/dolphinviewcontainer.cpp @@ -74,6 +74,7 @@ DolphinViewContainer::DolphinViewContainer(const KUrl& url, QWidget* parent) : m_filterBar(0), m_statusBar(0), m_statusBarTimer(0), + m_statusBarTimestamp(), m_dirLister(0), m_proxyModel(0) { @@ -125,6 +126,8 @@ DolphinViewContainer::DolphinViewContainer(const KUrl& url, QWidget* parent) : this, SLOT(updateProgress(int))); connect(m_dirLister, SIGNAL(itemsDeleted(const KFileItemList&)), this, SLOT(delayedStatusBarUpdate())); + connect(m_dirLister, SIGNAL(newItems(KFileItemList)), + this, SLOT(delayedStatusBarUpdate())); connect(m_dirLister, SIGNAL(completed()), this, SLOT(slotDirListerCompleted())); connect(m_dirLister, SIGNAL(infoMessage(const QString&)), @@ -280,32 +283,39 @@ void DolphinViewContainer::showFilterBar(bool show) void DolphinViewContainer::delayedStatusBarUpdate() { - // Invoke updateStatusBar() with a small delay. This assures that - // when a lot of delayedStatusBarUpdates() are done in a short time, - // no bottleneck is given. - m_statusBarTimer->start(); + if (m_statusBarTimer->isActive() && (m_statusBarTimestamp.elapsed() > 2000)) { + // No update of the statusbar has been done during the last 2 seconds, + // although an update has been requested. Trigger an immediate update. + m_statusBarTimer->stop(); + updateStatusBar(); + } else { + // Invoke updateStatusBar() with a small delay. This assures that + // when a lot of delayedStatusBarUpdates() are done in a short time, + // no bottleneck is given. + m_statusBarTimer->start(); + } } void DolphinViewContainer::updateStatusBar() { + m_statusBarTimestamp.start(); + // As the item count information is less important // in comparison with other messages, it should only // be shown if: // - the status bar is empty or // - shows already the item count information or // - shows only a not very important information - // - if any progress is given don't show the item count info at all - const QString msg = m_statusBar->message(); - const bool updateStatusBarMsg = (msg.isEmpty() - || (msg == m_statusBar->defaultText()) - || (m_statusBar->type() == DolphinStatusBar::Information)) - && (m_statusBar->progress() == 100); + const QString newMessage = m_view->statusBarText(); + const QString currentMessage = m_statusBar->message(); + const bool updateStatusBarMsg = currentMessage.isEmpty() + || (currentMessage == m_statusBar->defaultText()) + || (m_statusBar->type() == DolphinStatusBar::Information); - const QString text = m_view->statusBarText(); - m_statusBar->setDefaultText(text); + m_statusBar->setDefaultText(newMessage); if (updateStatusBarMsg) { - m_statusBar->setMessage(text, DolphinStatusBar::Default); + m_statusBar->setMessage(newMessage, DolphinStatusBar::Default); } } diff --git a/src/dolphinviewcontainer.h b/src/dolphinviewcontainer.h index fe1ad8d6e..aa43e8e66 100644 --- a/src/dolphinviewcontainer.h +++ b/src/dolphinviewcontainer.h @@ -28,11 +28,8 @@ #include -#include -#include -#include -#include -#include +#include +#include #include @@ -257,7 +254,8 @@ private: FilterBar* m_filterBar; DolphinStatusBar* m_statusBar; - QTimer* m_statusBarTimer; + QTimer* m_statusBarTimer; // Triggers a delayed update + QElapsedTimer m_statusBarTimestamp; // Time in ms since last update DolphinModel* m_dolphinModel; DolphinDirLister* m_dirLister;