# HG changeset patch
# User Chris Cannam
# Date 1222700803 0
# Node ID 374b72f5b4bbb0a95a7f6cbd814c6313c3abc0c5
# Parent  5a55d0683ee5708c4512edcd533d2b88b8be7b81
* More SelectableLabel and TransformFinder tweaking

diff -r 5a55d0683ee5 -r 374b72f5b4bb widgets/SelectableLabel.cpp
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/widgets/SelectableLabel.cpp	Mon Sep 29 15:06:43 2008 +0000
@@ -0,0 +1,135 @@
+/* -*- c-basic-offset: 4 indent-tabs-mode: nil -*-  vi:set ts=8 sts=4 sw=4: */
+
+/*
+    Sonic Visualiser
+    An audio file viewer and annotation editor.
+    Centre for Digital Music, Queen Mary, University of London.
+    This file copyright 2008 QMUL.
+    
+    This program is free software; you can redistribute it and/or
+    modify it under the terms of the GNU General Public License as
+    published by the Free Software Foundation; either version 2 of the
+    License, or (at your option) any later version.  See the file
+    COPYING included with this distribution for more information.
+*/
+
+#include "SelectableLabel.h"
+
+#include <iostream>
+#include <QApplication>
+
+SelectableLabel::SelectableLabel(QWidget *p) :
+    QLabel(p),
+    m_selected(false)
+{
+    setTextFormat(Qt::RichText);
+//    setLineWidth(2);
+//    setFixedWidth(480);
+    setupStyle();
+}
+
+SelectableLabel::~SelectableLabel()
+{
+}
+
+void
+SelectableLabel::setUnselectedText(QString text)
+{
+    m_unselectedText = text;
+    if (!m_selected) {
+        setText(m_unselectedText);
+        resize(sizeHint());
+    }
+}
+
+void
+SelectableLabel::setSelectedText(QString text)
+{
+    m_selectedText = text;
+    if (m_selected) {
+        setText(m_selectedText);
+        resize(sizeHint());
+    }
+}
+
+void
+SelectableLabel::setupStyle()
+{
+    QPalette palette = QApplication::palette();
+
+    if (m_selected) {
+        setWordWrap(true);
+        setStyleSheet
+            (QString("QLabel:hover { background: %1; color: %3; } "
+                     "QLabel:!hover { background: %2; color: %3 } "
+                     "QLabel { padding: 7px }")
+             .arg(palette.button().color().name())
+             .arg(palette.mid().color().light().name())
+             .arg(palette.text().color().name()));
+    } else {
+        setWordWrap(false);
+        setStyleSheet
+            (QString("QLabel:hover { background: %1; color: %3; } "
+                     "QLabel:!hover { background: %2; color: %3 } "
+                     "QLabel { padding: 7px }")
+             .arg(palette.button().color().name())
+             .arg(palette.light().color().name())
+             .arg(palette.text().color().name()));
+
+//        setStyleSheet("QLabel:hover { background: #e0e0e0; color: black; } QLabel:!hover { background: white; color: black } QLabel { padding: 7px }");
+    }
+}    
+
+void
+SelectableLabel::setSelected(bool s)
+{
+    if (m_selected == s) return;
+    m_selected = s;
+    if (m_selected) {
+        setText(m_selectedText);
+    } else {
+        setText(m_unselectedText);
+    }
+    setupStyle();
+    parentWidget()->resize(parentWidget()->sizeHint());
+}
+
+void
+SelectableLabel::toggle()
+{
+    setSelected(!m_selected);
+}
+
+void
+SelectableLabel::mousePressEvent(QMouseEvent *e)
+{
+    setSelected(true);
+    emit selectionChanged();
+}
+
+void
+SelectableLabel::mouseDoubleClickEvent(QMouseEvent *e)
+{
+    std::cerr << "mouseDoubleClickEvent" << std::endl;
+    emit doubleClicked();
+}
+
+void
+SelectableLabel::enterEvent(QEvent *)
+{
+//    std::cerr << "enterEvent" << std::endl;
+//    QPalette palette = QApplication::palette();
+//    palette.setColor(QPalette::Window, Qt::gray);
+//    setStyleSheet("background: gray");
+//    setPalette(palette);
+}
+
+void
+SelectableLabel::leaveEvent(QEvent *)
+{
+//    std::cerr << "leaveEvent" << std::endl;
+//    setStyleSheet("background: white");
+//    QPalette palette = QApplication::palette();
+//    palette.setColor(QPalette::Window, Qt::gray);
+//    setPalette(palette);
+}
diff -r 5a55d0683ee5 -r 374b72f5b4bb widgets/SelectableLabel.h
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/widgets/SelectableLabel.h	Mon Sep 29 15:06:43 2008 +0000
@@ -0,0 +1,53 @@
+/* -*- c-basic-offset: 4 indent-tabs-mode: nil -*-  vi:set ts=8 sts=4 sw=4: */
+
+/*
+    Sonic Visualiser
+    An audio file viewer and annotation editor.
+    Centre for Digital Music, Queen Mary, University of London.
+    This file copyright 2008 QMUL.
+    
+    This program is free software; you can redistribute it and/or
+    modify it under the terms of the GNU General Public License as
+    published by the Free Software Foundation; either version 2 of the
+    License, or (at your option) any later version.  See the file
+    COPYING included with this distribution for more information.
+*/
+
+#ifndef _SELECTABLE_LABEL_H_
+#define _SELECTABLE_LABEL_H_
+
+#include <QLabel>
+
+class SelectableLabel : public QLabel
+{
+    Q_OBJECT
+
+public:
+    SelectableLabel(QWidget *parent = 0);
+    virtual ~SelectableLabel();
+
+    void setSelectedText(QString);
+    void setUnselectedText(QString);
+
+    bool isSelected() const { return m_selected; }
+
+signals:
+    void selectionChanged();
+    void doubleClicked();
+
+public slots:
+    void setSelected(bool);
+    void toggle();
+
+protected:
+    virtual void mousePressEvent(QMouseEvent *e);
+    virtual void mouseDoubleClickEvent(QMouseEvent *e);
+    virtual void enterEvent(QEvent *);
+    virtual void leaveEvent(QEvent *);
+    void setupStyle();
+    QString m_selectedText;
+    QString m_unselectedText;
+    bool m_selected;
+};
+
+#endif
diff -r 5a55d0683ee5 -r 374b72f5b4bb widgets/TransformFinder.cpp
--- a/widgets/TransformFinder.cpp	Mon Sep 29 11:11:42 2008 +0000
+++ b/widgets/TransformFinder.cpp	Mon Sep 29 15:06:43 2008 +0000
@@ -15,116 +15,19 @@
 
 #include "TransformFinder.h"
 
+#include "base/XmlExportable.h"
 #include "transform/TransformFactory.h"
+#include "SelectableLabel.h"
 
 #include <QVBoxLayout>
 #include <QGridLayout>
 #include <QLineEdit>
 #include <QLabel>
-//#include <SelectableLabel>
 #include <QDialogButtonBox>
 #include <QScrollArea>
 #include <QApplication>
 #include <QDesktopWidget>
-
-SelectableLabel::SelectableLabel(QWidget *p) :
-    QLabel(p),
-    m_selected(false)
-{
-    setTextFormat(Qt::RichText);
-//    setLineWidth(2);
-//    setFixedWidth(480);
-    setupStyle();
-}
-
-void
-SelectableLabel::setUnselectedText(QString text)
-{
-    m_unselectedText = text;
-    if (!m_selected) {
-        setText(m_unselectedText);
-        resize(sizeHint());
-    }
-}
-
-void
-SelectableLabel::setSelectedText(QString text)
-{
-    m_selectedText = text;
-    if (m_selected) {
-        setText(m_selectedText);
-        resize(sizeHint());
-    }
-}
-
-void
-SelectableLabel::setupStyle()
-{
-    if (m_selected) {
-        setWordWrap(true);
-        setStyleSheet("QLabel:hover { background: #e0e0e0; color: black; } QLabel:!hover { background: #f0f0f0; color: black } QLabel { padding: 7px }");
-
-//        setFrameStyle(QFrame::Box | QFrame::Plain);
-    } else {
-        setWordWrap(false);
-        setStyleSheet("QLabel:hover { background: #e0e0e0; color: black; } QLabel:!hover { background: white; color: black } QLabel { padding: 7px }");
-
-//        setFrameStyle(QFrame::NoFrame);
-    }
-}    
-
-void
-SelectableLabel::setSelected(bool s)
-{
-    if (m_selected == s) return;
-    m_selected = s;
-    if (m_selected) {
-        setText(m_selectedText);
-    } else {
-        setText(m_unselectedText);
-    }
-    setupStyle();
-    parentWidget()->resize(parentWidget()->sizeHint());
-}
-
-void
-SelectableLabel::toggle()
-{
-    setSelected(!m_selected);
-}
-
-void
-SelectableLabel::mousePressEvent(QMouseEvent *e)
-{
-    setSelected(true);
-    emit selectionChanged();
-}
-
-void
-SelectableLabel::mouseDoubleClickEvent(QMouseEvent *e)
-{
-    std::cerr << "mouseDoubleClickEvent" << std::endl;
-}
-
-void
-SelectableLabel::enterEvent(QEvent *)
-{
-//    std::cerr << "enterEvent" << std::endl;
-//    QPalette palette = QApplication::palette();
-//    palette.setColor(QPalette::Window, Qt::gray);
-//    setStyleSheet("background: gray");
-//    setPalette(palette);
-}
-
-void
-SelectableLabel::leaveEvent(QEvent *)
-{
-//    std::cerr << "leaveEvent" << std::endl;
-//    setStyleSheet("background: white");
-//    QPalette palette = QApplication::palette();
-//    palette.setColor(QPalette::Window, Qt::gray);
-//    setPalette(palette);
-}
+#include <QTimer>
 
 TransformFinder::TransformFinder(QWidget *parent) :
     QDialog(parent),
@@ -161,7 +64,6 @@
         palette.setColor(QPalette::Window, palette.color(QPalette::Base));
         m_resultsFrame->setPalette(palette);
         m_resultsScroll->setPalette(palette);
-//        resultsFrame->setFrameStyle(QFrame::Sunken | QFrame::Box);
         m_resultsLayout = new QVBoxLayout;
         m_resultsLayout->setSpacing(0);
         m_resultsLayout->setContentsMargins(0, 0, 0, 0);
@@ -184,6 +86,11 @@
 
     resize(width, height);
     raise();
+
+    m_upToDateCount = 0;
+    m_timer = new QTimer(this);
+    connect(m_timer, SIGNAL(timeout()), this, SLOT(timeout()));
+    m_timer->start(0);
 }
 
 TransformFinder::~TransformFinder()
@@ -194,82 +101,98 @@
 TransformFinder::searchTextChanged(const QString &text)
 {
     std::cerr << "text is " << text.toStdString() << std::endl;
+    m_newSearchText = text;
+}
 
-    QStringList keywords = text.split(' ', QString::SkipEmptyParts);
-    TransformFactory::SearchResults results =
-        TransformFactory::getInstance()->search(keywords);
+void
+TransformFinder::timeout()
+{
+    int maxResults = 40;
     
-    std::cerr << results.size() << " result(s)..." << std::endl;
+    if (m_newSearchText != "") {
 
-    std::set<TransformFactory::Match> sorted;
-    for (TransformFactory::SearchResults::const_iterator j = results.begin();
-         j != results.end(); ++j) {
-        sorted.insert(j->second);
+        QString text = m_newSearchText;
+        m_newSearchText = "";
+
+        QStringList keywords = text.split(' ', QString::SkipEmptyParts);
+        TransformFactory::SearchResults results =
+            TransformFactory::getInstance()->search(keywords);
+        
+        std::cerr << results.size() << " result(s)..." << std::endl;
+        
+        std::set<TransformFactory::Match> sorted;
+        sorted.clear();
+        for (TransformFactory::SearchResults::const_iterator j = results.begin();
+             j != results.end(); ++j) {
+            sorted.insert(j->second);
+        }
+
+        m_sortedResults.clear();
+        for (std::set<TransformFactory::Match>::const_iterator j = sorted.end();
+             j != sorted.begin(); ) {
+            --j;
+            m_sortedResults.push_back(*j);
+            if (m_sortedResults.size() == maxResults) break;
+        }
+
+        if (m_sortedResults.empty()) m_selectedTransform = "";
+        else m_selectedTransform = m_sortedResults.begin()->transform;
+
+        m_upToDateCount = 0;
+
+        for (int j = m_labels.size(); j > m_sortedResults.size(); ) {
+            m_labels[--j]->hide();
+        }
+
+        return;
     }
 
-    int i = 0;
-/*
-    for (std::set<TransformFactory::Match>::const_iterator j = sorted.begin();
-         j != sorted.end(); ++j) {
-        std::cerr << i++ << ": " << j->transform.toStdString() << ": ";
-        for (TransformFactory::Match::FragmentMap::const_iterator k =
-                 j->fragments.begin();
-             k != j->fragments.end(); ++k) {
-            std::cerr << k->first.toStdString() << ": "
-                      << k->second.toStdString() << " ";
-        }
-        std::cerr << "(" << j->score << ")" << std::endl;
-    }
-*/
+    if (m_upToDateCount < m_sortedResults.size()) {
 
-    i = 0;
-    int maxResults = 40;
-    int height = 0;
-    int width = 0;
+        int i = m_upToDateCount;
 
-    if (sorted.empty()) m_selectedTransform = "";
-
-    for (std::set<TransformFactory::Match>::const_iterator j = sorted.end();
-         j != sorted.begin(); ) {
-        --j;
+        std::cerr << "sorted size = " << m_sortedResults.size() << std::endl;
 
         TransformDescription desc =
-            TransformFactory::getInstance()->getTransformDescription(j->transform);
+            TransformFactory::getInstance()->getTransformDescription
+            (m_sortedResults[i].transform);
 
         QString labelText;
-        labelText += tr("%2<br><small>").arg(desc.name);
+        labelText += tr("%1: %2<br><small>")
+            .arg(m_sortedResults[i].score)
+            .arg(XmlExportable::encodeEntities(desc.name));
+
         labelText += "...";
         for (TransformFactory::Match::FragmentMap::const_iterator k =
-                 j->fragments.begin();
-             k != j->fragments.end(); ++k) {
+                 m_sortedResults[i].fragments.begin();
+             k != m_sortedResults[i].fragments.end(); ++k) {
             labelText += k->second;
             labelText += "... ";
         }
         labelText += tr("</small>");
 
         QString selectedText;
-        selectedText += tr("<b>%1</b><br>").arg(desc.name);
-        selectedText += tr("<small>%1</small>").arg(desc.longDescription);
-/*
-        for (TransformFactory::Match::FragmentMap::const_iterator k =
-                 j->fragments.begin();
-             k != j->fragments.end(); ++k) {
-            selectedText += tr("<br><small>%1: %2</small>").arg(k->first).arg(k->second);
-        }
-*/
+        selectedText += tr("<b>%1</b><br>")
+            .arg(XmlExportable::encodeEntities(desc.name));
+        selectedText += tr("<small>%1</small>")
+            .arg(XmlExportable::encodeEntities(desc.longDescription));
 
         selectedText += tr("<ul><small>");
-        selectedText += tr("<li>Plugin type: %1</li>").arg(desc.type);
-        selectedText += tr("<li>Category: %1</li>").arg(desc.category);
-        selectedText += tr("<li>System identifier: %1</li>").arg(desc.identifier);
+        selectedText += tr("<li>Plugin type: %1</li>")
+            .arg(XmlExportable::encodeEntities(desc.type));
+        selectedText += tr("<li>Category: %1</li>")
+            .arg(XmlExportable::encodeEntities(desc.category));
+        selectedText += tr("<li>System identifier: %1</li>")
+            .arg(XmlExportable::encodeEntities(desc.identifier));
         selectedText += tr("</small></ul>");
 
         if (i >= m_labels.size()) {
             SelectableLabel *label = new SelectableLabel(m_resultsFrame);
-//            m_resultsLayout->addWidget(label, i, 0);
             m_resultsLayout->addWidget(label);
             connect(label, SIGNAL(selectionChanged()), this,
                     SLOT(selectedLabelChanged()));
+            connect(label, SIGNAL(doubleClicked()), this,
+                    SLOT(accept()));
             QPalette palette = label->palette();
             label->setPalette(palette);
             m_labels.push_back(label);
@@ -280,35 +203,15 @@
         m_labels[i]->setUnselectedText(labelText);
         m_labels[i]->setSelectedText(selectedText);
 
-        /*
-        m_labels[i]->setSelected(false);
-        m_selectedTransform = "";
-        */
-
-        m_labels[i]->setSelected(i == 0);
-        if (i == 0) {
-            m_selectedTransform = desc.identifier;
-        }
-
-/*
-        QSize sh = m_labels[i]->sizeHint();
-        std::cerr << "size hint for text \"" << labelText.toStdString() << "\" has height " << sh.height() << std::endl;
-        height += sh.height();
-        if (sh.width() > width) width = sh.width();
-*/
-//        m_labels[i]->resize(m_labels[i]->sizeHint());
-//        m_labels[i]->updateGeometry();
+        m_labels[i]->setSelected(m_selectedTransform == desc.identifier);
         m_labels[i]->show();
 
-        if (++i == maxResults) break;
+        ++m_upToDateCount;
+
+//        if (m_upToDateCount == m_sortedResults.size()) {
+            m_resultsFrame->resize(m_resultsFrame->sizeHint());
+//        }
     }
-
-    std::cerr << "m_labels.size() = " << m_labels.size() << ", i = " << i << ", height = " << height << std::endl;
-
-    for (int j = m_labels.size(); j > i; ) m_labels[--j]->hide();
-
-    m_resultsFrame->resize(m_resultsFrame->sizeHint());
-//    m_resultsFrame->resize(height, width);
 }
 
 void
@@ -335,6 +238,6 @@
 TransformId
 TransformFinder::getTransform() const
 {
-    return "";
+    return m_selectedTransform;
 }
 
diff -r 5a55d0683ee5 -r 374b72f5b4bb widgets/TransformFinder.h
--- a/widgets/TransformFinder.h	Mon Sep 29 11:11:42 2008 +0000
+++ b/widgets/TransformFinder.h	Mon Sep 29 15:06:43 2008 +0000
@@ -21,45 +21,14 @@
 #include <vector>
 
 #include "transform/Transform.h"
+#include "transform/TransformFactory.h"
 
 class QVBoxLayout;
 class QScrollArea;
 class QLabel;
 class SelectableLabel;
 class QWidget;
-
-
-#include <QLabel>
-class SelectableLabel : public QLabel
-{
-    Q_OBJECT
-
-public:
-    SelectableLabel(QWidget *parent = 0);
-    virtual ~SelectableLabel() { }
-
-    void setSelectedText(QString);
-    void setUnselectedText(QString);
-
-    bool isSelected() const { return m_selected; }
-
-signals:
-    void selectionChanged();
-
-public slots:
-    void setSelected(bool);
-    void toggle();
-
-protected:
-    virtual void mousePressEvent(QMouseEvent *e);
-    virtual void mouseDoubleClickEvent(QMouseEvent *e);
-    virtual void enterEvent(QEvent *);
-    virtual void leaveEvent(QEvent *);
-    void setupStyle();
-    QString m_selectedText;
-    QString m_unselectedText;
-    bool m_selected;
-};
+class QTimer;
 
 class TransformFinder : public QDialog
 {
@@ -74,6 +43,7 @@
 protected slots:
     void searchTextChanged(const QString &);
     void selectedLabelChanged();
+    void timeout();
 
 protected:
     QScrollArea *m_resultsScroll;
@@ -81,6 +51,12 @@
     QVBoxLayout *m_resultsLayout;
     std::vector<SelectableLabel *> m_labels;
     TransformId m_selectedTransform;
+    QTimer *m_timer;
+
+    QString m_newSearchText;
+    typedef std::vector<TransformFactory::Match> SortedResults;
+    SortedResults m_sortedResults;
+    int m_upToDateCount;
 };
 
 #endif
diff -r 5a55d0683ee5 -r 374b72f5b4bb widgets/widgets.pro
--- a/widgets/widgets.pro	Mon Sep 29 11:11:42 2008 +0000
+++ b/widgets/widgets.pro	Mon Sep 29 15:06:43 2008 +0000
@@ -42,6 +42,7 @@
            PropertyBox.h \
            PropertyStack.h \
            RangeInputDialog.h \
+           SelectableLabel.h \
            SubdividingMenu.h \
            TextAbbrev.h \
            Thumbwheel.h \
@@ -77,6 +78,7 @@
            PropertyBox.cpp \
            PropertyStack.cpp \
            RangeInputDialog.cpp \
+           SelectableLabel.cpp \
            SubdividingMenu.cpp \
            TextAbbrev.cpp \
            Thumbwheel.cpp \