]> git.sesse.net Git - kdenlive/commitdiff
Add delay to status bar messages
authorSimon A. Eugster <simon.eu@gmail.com>
Wed, 15 Feb 2012 11:12:06 +0000 (12:12 +0100)
committerSimon A. Eugster <simon.eu@gmail.com>
Wed, 15 Feb 2012 11:17:26 +0000 (12:17 +0100)
Simplified the status bar code and updated it to queue messages;
Consecutive error messages will not overwrite each other anymore.

Resolves: http://www.kdenlive.org/mantis/view.php?id=2519

.gitignore
src/CMakeLists.txt
src/lib/CMakeLists.txt [new file with mode: 0644]
src/lib/qtimerWithTime.cpp [new file with mode: 0644]
src/lib/qtimerWithTime.h [new file with mode: 0644]
src/statusbarmessagelabel.cpp
src/statusbarmessagelabel.h

index 378eac25d311703f3f2cd456d8036da525cd0366..bdc5af0409bb506232f5fa7f1b4e87d2bf3fda5b 100644 (file)
@@ -1 +1,2 @@
+*~
 build
index 8fdcd8cc8fb452854947e710ccd05d900195086f..5ccf034e01fe73b082b7035d23b15affe26b9cdb 100644 (file)
@@ -80,6 +80,7 @@ add_subdirectory(colorscopes)
 add_subdirectory(commands)
 add_subdirectory(databackup)
 add_subdirectory(kiss_fft)
+add_subdirectory(lib)
 add_subdirectory(mimetypes)
 add_subdirectory(onmonitoritems)
 add_subdirectory(simplekeyframes)
diff --git a/src/lib/CMakeLists.txt b/src/lib/CMakeLists.txt
new file mode 100644 (file)
index 0000000..05dd09f
--- /dev/null
@@ -0,0 +1,5 @@
+set(kdenlive_SRCS
+  ${kdenlive_SRCS}
+  lib/qtimerWithTime.cpp
+  PARENT_SCOPE
+)
\ No newline at end of file
diff --git a/src/lib/qtimerWithTime.cpp b/src/lib/qtimerWithTime.cpp
new file mode 100644 (file)
index 0000000..cd40710
--- /dev/null
@@ -0,0 +1,12 @@
+#include "qtimerWithTime.h"
+
+void QTimerWithTime::start(int msec)
+{
+    QTimer::start(msec);
+    m_time.start();
+}
+
+int QTimerWithTime::elapsed() const
+{
+    return m_time.elapsed();
+}
diff --git a/src/lib/qtimerWithTime.h b/src/lib/qtimerWithTime.h
new file mode 100644 (file)
index 0000000..1a6e53b
--- /dev/null
@@ -0,0 +1,13 @@
+
+#include <QTimer>
+#include <QTime>
+
+class QTimerWithTime : public QTimer
+{
+    Q_OBJECT
+public:
+    virtual void start(int msec);
+    int elapsed() const;
+ private:
+    QTime m_time;
+};
index 42eef766691fb9ad5cc08f75b4a83faa8bc2c0c1..a50b5db6ba01e03d8a902355e6ffedf856a1752b 100644 (file)
@@ -1,5 +1,6 @@
 /***************************************************************************
  *   Copyright (C) 2006 by Peter Penz                                      *
+ *                 2012    Simon A. Eugster <simon.eu@gmail.com>           *
  *   peter.penz@gmx.at                                                     *
  *   Code borrowed from Dolphin, adapted (2008) to Kdenlive by             *
  *   Jean-Baptiste Mardelle, jb@kdenlive.org                               *
 
 
 StatusBarMessageLabel::StatusBarMessageLabel(QWidget* parent) :
-        QWidget(parent),
-        m_type(DefaultMessage),
-        m_state(Default),
-        m_illumination(-64),
-        m_minTextHeight(-1),
-        m_closeButton(0)
+    QWidget(parent),
+    m_state(Default),
+    m_illumination(-64),
+    m_minTextHeight(-1),
+    m_queueSemaphore(1),
+    m_closeButton(0)
 {
     setMinimumHeight(KIconLoader::SizeSmall);
     QPalette palette;
     palette.setColor(QPalette::Background, Qt::transparent);
     setPalette(palette);
-    m_hidetimer.setSingleShot(true);
-    m_hidetimer.setInterval(5000);
-    connect(&m_timer, SIGNAL(timeout()), this, SLOT(timerDone()));
-    connect(&m_hidetimer, SIGNAL(timeout()), this, SLOT(closeErrorMessage()));
 
-    m_closeButton = new QPushButton(i18nc("@action:button", "Close"), this);
+    m_closeButton = new QPushButton(i18nc("@action:button", "Confirm"), this);
     m_closeButton->hide();
-    connect(m_closeButton, SIGNAL(clicked()), this, SLOT(closeErrorMessage()));
+
+    m_queueTimer.setSingleShot(true);
+
+    bool b = true;
+    b &= connect(&m_queueTimer, SIGNAL(timeout()), this, SLOT(slotMessageTimeout()));
+
+    b &= connect(m_closeButton, SIGNAL(clicked()), this, SLOT(confirmErrorMessage()));
+    b &= connect(&m_timer, SIGNAL(timeout()), this, SLOT(timerDone()));
+    Q_ASSERT(b);
 }
 
 StatusBarMessageLabel::~StatusBarMessageLabel()
@@ -63,44 +68,93 @@ StatusBarMessageLabel::~StatusBarMessageLabel()
 }
 
 void StatusBarMessageLabel::setMessage(const QString& text,
-                                       MessageType type)
+                                       MessageType type, int timeoutMS)
+{
+    StatusBarMessageItem item(text, type, timeoutMS);
+
+    if (item.type == ErrorMessage || item.type == MltError) {
+        KNotification::event("ErrorMessage", item.text);
+    }
+
+    m_queueSemaphore.acquire();
+    if (!m_messageQueue.contains(item)) {
+        if (item.type == ErrorMessage || item.type == MltError) {
+            qDebug() << item.text;
+
+            // Put the new errror message at first place and immediately show it
+            if (item.timeoutMillis < 2000) {
+                item.timeoutMillis = 2000;
+            }
+            m_messageQueue.push_front(item);
+
+            // In case we are already displaying an error message, add a little delay
+            int delay = 800 * (m_currentMessage.type == ErrorMessage || m_currentMessage.type == MltError);
+            m_queueTimer.start(delay);
+
+        } else {
+
+            // Message with normal priority
+            m_messageQueue.push_back(item);
+            if (!m_queueTimer.elapsed() >= m_currentMessage.timeoutMillis) {
+                m_queueTimer.start(0);
+            }
+
+        }
+    }
+
+    m_queueSemaphore.release();
+}
+
+bool StatusBarMessageLabel::slotMessageTimeout()
 {
-    if ((text == m_text) && (type == m_type)) {
-        if (type == ErrorMessage) KNotification::event("ErrorMessage", m_text);
-        return;
+    m_queueSemaphore.acquire();
+
+    bool newMessage = false;
+
+    // Get the next message from the queue, unless the current one needs to be confirmed
+    if (m_messageQueue.size() > 0) {
+
+        if (!m_currentMessage.needsConfirmation()) {
+
+            m_currentMessage = m_messageQueue.at(0);
+            m_messageQueue.removeFirst();
+            newMessage = true;
+
+        }
+    }
+
+    // If the queue is empty, add a default (empty) message
+    if (m_messageQueue.size() == 0 && m_currentMessage.type != DefaultMessage) {
+        m_messageQueue.push_back(StatusBarMessageItem());
     }
 
-    /*if (m_type == ErrorMessage) {
-        if (type == ErrorMessage) {
-            m_pendingMessages.insert(0, m_text);
-        } else if ((m_state != Default) || !m_pendingMessages.isEmpty()) {
-            // a non-error message should not be shown, as there
-            // are other pending error messages in the queue
-            return;
+    // Start a new timer, unless the current message still needs to be confirmed
+    if (m_messageQueue.size() > 0) {
+
+        if (!m_currentMessage.needsConfirmation()) {
+
+            // If we only have the default message left to show in the queue,
+            // keep the current one for a little longer.
+            m_queueTimer.start(m_currentMessage.timeoutMillis + 2000*(m_messageQueue.at(0).type == DefaultMessage));
+
         }
-    }*/
+    }
 
-    m_text = text;
-    m_type = type;
 
     m_illumination = -64;
     m_state = Default;
     m_timer.stop();
 
     const char* iconName = 0;
-    QPixmap pixmap;
-    switch (type) {
+    switch (m_currentMessage.type) {
     case OperationCompletedMessage:
         iconName = "dialog-ok";
-        // "ok" icon should probably be "dialog-success", but we don't have that icon in KDE 4.0
         m_closeButton->hide();
-        m_hidetimer.stop();
         break;
 
     case InformationMessage:
         iconName = "dialog-information";
         m_closeButton->hide();
-        m_hidetimer.start();
         break;
 
     case ErrorMessage:
@@ -108,8 +162,6 @@ void StatusBarMessageLabel::setMessage(const QString& text,
         m_timer.start(100);
         m_state = Illuminate;
         m_closeButton->hide();
-        KNotification::event("ErrorMessage", m_text);
-        m_hidetimer.stop();
         break;
 
     case MltError:
@@ -118,24 +170,26 @@ void StatusBarMessageLabel::setMessage(const QString& text,
         m_state = Illuminate;
         updateCloseButtonPosition();
         m_closeButton->show();
-        m_hidetimer.stop();
         break;
 
     case DefaultMessage:
     default:
         m_closeButton->hide();
-        m_hidetimer.stop();
         break;
     }
 
     m_pixmap = (iconName == 0) ? QPixmap() : SmallIcon(iconName);
 
-    /*QFontMetrics fontMetrics(font());
-    setMaximumWidth(fontMetrics.boundingRect(m_text).width() + m_pixmap.width() + (BorderGap * 4));
-    updateGeometry();*/
+    m_queueSemaphore.release();
 
-    //QTimer::singleShot(GeometryTimeout, this, SLOT(assureVisibleText()));
     update();
+    return newMessage;
+}
+
+void StatusBarMessageLabel::confirmErrorMessage()
+{
+    m_currentMessage.confirmed = true;
+    m_queueTimer.start(0);
 }
 
 void StatusBarMessageLabel::setMinimumTextHeight(int min)
@@ -149,7 +203,7 @@ void StatusBarMessageLabel::setMinimumTextHeight(int min)
     }
 }
 
-void StatusBarMessageLabel::paintEvent(QPaintEvent* /* event */)
+void StatusBarMessageLabel::paintEvent(QPaintEvent*)
 {
     QPainter painter(this);
 
@@ -179,7 +233,7 @@ void StatusBarMessageLabel::paintEvent(QPaintEvent* /* event */)
     if (height() > m_minTextHeight) {
         flags = flags | Qt::TextWordWrap;
     }
-    painter.drawText(QRect(x, 0, availableTextWidth(), height()), flags, m_text);
+    painter.drawText(QRect(x, 0, availableTextWidth(), height()), flags, m_currentMessage.text);
     painter.end();
 }
 
@@ -187,7 +241,6 @@ void StatusBarMessageLabel::resizeEvent(QResizeEvent* event)
 {
     QWidget::resizeEvent(event);
     updateCloseButtonPosition();
-    //QTimer::singleShot(GeometryTimeout, this, SLOT(assureVisibleText()));
 }
 
 void StatusBarMessageLabel::timerDone()
@@ -211,7 +264,7 @@ void StatusBarMessageLabel::timerDone()
 
     case Illuminated: {
         // start desaturation
-        if (m_type != MltError) {
+        if (m_currentMessage.type != MltError) {
             m_state = Desaturate;
             m_timer.start(80);
         }
@@ -237,53 +290,6 @@ void StatusBarMessageLabel::timerDone()
     }
 }
 
-void StatusBarMessageLabel::assureVisibleText()
-{
-    if (m_text.isEmpty()) {
-        return;
-    }
-
-    int requiredHeight = m_minTextHeight;
-    if (m_type != DefaultMessage) {
-        // Calculate the required height of the widget thats
-        // needed for having a fully visible text. Note that for the default
-        // statusbar type (e. g. hover information) increasing the text height
-        // is not wanted, as this might rearrange the layout of items.
-
-        QFontMetrics fontMetrics(font());
-        const QRect bounds(fontMetrics.boundingRect(0, 0, availableTextWidth(), height(),
-                           Qt::AlignVCenter | Qt::TextWordWrap, m_text));
-        requiredHeight = bounds.height();
-        if (requiredHeight < m_minTextHeight) {
-            requiredHeight = m_minTextHeight;
-        }
-    }
-
-    // Increase/decrease the current height of the widget to the
-    // required height. The increasing/decreasing is done in several
-    // steps to have an animation if the height is modified
-    // (see StatusBarMessageLabel::resizeEvent())
-    const int gap = m_minTextHeight / 2;
-    int minHeight = minimumHeight();
-    if (minHeight < requiredHeight) {
-        minHeight += gap;
-        if (minHeight > requiredHeight) {
-            minHeight = requiredHeight;
-        }
-        setMinimumHeight(minHeight);
-        updateGeometry();
-    } else if (minHeight > requiredHeight) {
-        minHeight -= gap;
-        if (minHeight < requiredHeight) {
-            minHeight = requiredHeight;
-        }
-        setMinimumHeight(minHeight);
-        updateGeometry();
-    }
-
-    updateCloseButtonPosition();
-}
-
 int StatusBarMessageLabel::availableTextWidth() const
 {
     const int buttonWidth = 0; /*(m_type == ErrorMessage) ?
@@ -298,21 +304,5 @@ void StatusBarMessageLabel::updateCloseButtonPosition()
     m_closeButton->move(x, y);
 }
 
-void StatusBarMessageLabel::closeErrorMessage()
-{
-    if (!showPendingMessage()) {
-        setMessage(QString(), DefaultMessage);
-    }
-}
-
-bool StatusBarMessageLabel::showPendingMessage()
-{
-    if (!m_pendingMessages.isEmpty()) {
-        setMessage(m_pendingMessages.takeFirst(), ErrorMessage);
-        return true;
-    }
-    return false;
-}
-
 
 #include "statusbarmessagelabel.moc"
index a0f491df6994f8e7d912f02455713b9d116a46f3..2b7f7c5d156d0eb178b8af3a0dc80a20346b71dd 100644 (file)
@@ -1,5 +1,6 @@
 /***************************************************************************
  *   Copyright (C) 2006 by Peter Penz                                      *
+ *                 2012    Simon A. Eugster <simon.eu@gmail.com>           *
  *   peter.penz@gmx.at                                                     *
  *   Code borrowed from Dolphin, adapted (2008) to Kdenlive by             *
  *   Jean-Baptiste Mardelle, jb@kdenlive.org                               *
 #include <QPixmap>
 #include <QWidget>
 #include <QTimer>
+#include <QSemaphore>
 
 #include <definitions.h>
 
+#include "lib/qtimerWithTime.h"
+
 class QPaintEvent;
 class QResizeEvent;
 class QPushButton;
 
+
+/**
+  Queue-able message item holding all important information
+  */
+struct StatusBarMessageItem {
+
+    QString text;
+    MessageType type;
+    int timeoutMillis;
+    bool confirmed; ///< MLT errors need to be confirmed.
+
+    /// \return true if the error still needs to be confirmed
+    bool needsConfirmation() const
+    {
+        return type == MltError && !confirmed;
+    }
+
+    StatusBarMessageItem(const QString& text = QString(), MessageType type = DefaultMessage, int timeoutMS = 0) :
+        text(text), type(type), timeoutMillis(timeoutMS), confirmed(false) {}
+
+    bool operator ==(const StatusBarMessageItem &other)
+    {
+        return type == other.type && text == other.text;
+    }
+};
+
 /**
  * @brief Represents a message text label as part of the status bar.
  *
@@ -51,10 +81,6 @@ public:
     explicit StatusBarMessageLabel(QWidget* parent);
     virtual ~StatusBarMessageLabel();
 
-    MessageType type() const;
-
-    const QString& text() const;
-
     // TODO: maybe a better approach is possible with the size hint
     void setMinimumTextHeight(int min);
     int minimumTextHeight() const;
@@ -67,17 +93,11 @@ protected:
     virtual void resizeEvent(QResizeEvent* event);
 
 public slots:
-    void setMessage(const QString& text, MessageType type);
+    void setMessage(const QString& text, MessageType type, int timeoutMS = 0);
 
 private slots:
     void timerDone();
 
-    /**
-     * Increases the height of the message label so that
-     * the given text fits into given area.
-     */
-    void assureVisibleText();
-
     /**
      * Returns the available width in pixels for the text.
      */
@@ -93,21 +113,13 @@ private slots:
      * Closes the currently shown error message and replaces it
      * by the next pending message.
      */
-    void closeErrorMessage();
+    void confirmErrorMessage();
 
-private:
     /**
      * Shows the next pending error message. If no pending message
      * was in the queue, false is returned.
      */
-    bool showPendingMessage();
-
-    /**
-     * Resets the message label properties. This is useful when the
-     * result of invoking StatusBarMessageLabel::setMessage() should
-     * not rely on previous states.
-     */
-    void reset();
+    bool slotMessageTimeout();
 
 private:
     enum State {
@@ -120,31 +132,24 @@ private:
     enum { GeometryTimeout = 100 };
     enum { BorderGap = 2 };
 
-    MessageType m_type;
     State m_state;
     int m_illumination;
     int m_minTextHeight;
     QTimer m_timer;
-    QTimer m_hidetimer;
-    QString m_text;
-    QList<QString> m_pendingMessages;
+
+    QTimerWithTime m_queueTimer;
+    QSemaphore m_queueSemaphore;
+    QList<StatusBarMessageItem> m_messageQueue;
+    StatusBarMessageItem m_currentMessage;
+
     QPixmap m_pixmap;
     QPushButton* m_closeButton;
 };
 
-inline MessageType StatusBarMessageLabel::type() const
-{
-    return m_type;
-}
-
-inline const QString& StatusBarMessageLabel::text() const
-{
-    return m_text;
-}
-
 inline int StatusBarMessageLabel::minimumTextHeight() const
 {
     return m_minTextHeight;
 }
 
+
 #endif