On 10/08/12 20:32, Thomas Lübking wrote:
@@ -101,7 +102,10 @@ void MessageListWidget::slotApplySearch()
  void MessageListWidget::slotAutoEnableDisableSearch()
  {
      bool isEnabled;
-    if (tree&&  tree->model()) {
+    if (!m_quickSearchText->text().isEmpty()) { // rowCount might be 0 for 
active filter
+        isEnabled = true;
+    }
+    else if (tree&&  tree->model()) {
          isEnabled = tree->model()->rowCount();
      } else {
          isEnabled = false;

What is this doing -- I assume this is not related to focus on wheel?

diff --git a/src/Gui/Window.cpp b/src/Gui/Window.cpp
index 510d767..513907a 100644
--- a/src/Gui/Window.cpp
+++ b/src/Gui/Window.cpp
@@ -801,6 +801,7 @@ void MainWindow::msgListDoubleClicked(const 
QModelIndex&index)
      newView->setMessage(index);

      QScrollArea *widget = new QScrollArea();
+    setFocusPolicy(Qt::StrongFocus); // not by the wheel

Should be widget->setFocusPolicy(), right? But what is it for, anyway -- just for the consistency?

diff --git a/src/Gui/MessageListWidget.cpp b/src/Gui/MessageListWidget.cpp
index cb3e1b2..0eb39c5 100644
--- a/src/Gui/MessageListWidget.cpp
+++ b/src/Gui/MessageListWidget.cpp
@@ -35,6 +35,7 @@ MessageListWidget::MessageListWidget(QWidget *parent) :
 {
     tree = new MsgListView(this);
     tree->setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOn);
+    tree->setFocusPolicy(Qt::StrongFocus); // not by the wheel

In future, please rebase your tree on the recent master from gitorious; the line "tree->setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOn)" is not there.

The patch is in, apart from the first two modifications. Thanks for that.

Cheers,
Jan
--
Trojita, a fast e-mail client -- http://trojita.flaska.net/

Reply via email to