Rework the context menu action handling in BufferView
[quassel.git] / src / uisupport / bufferview.cpp
index c0e85ad..6da7535 100644 (file)
 
 #include "bufferview.h"
 
+#include "action.h"
 #include "buffermodel.h"
 #include "bufferviewfilter.h"
+#include "buffersettings.h"
 #include "buffersyncer.h"
 #include "client.h"
+#include "iconloader.h"
 #include "mappedselectionmodel.h"
 #include "network.h"
 #include "networkmodel.h"
 
 #include "uisettings.h"
 
-#include "global.h"
-
 #include <QAction>
 #include <QFlags>
 #include <QHeaderView>
 *****************************************/
 // Please be carefull when reimplementing methods which are used to inform the view about changes to the data
 // to be on the safe side: call QTreeView's method aswell
-BufferView::BufferView(QWidget *parent) : QTreeView(parent) {
-  setContextMenuPolicy(Qt::CustomContextMenu);
+BufferView::BufferView(QWidget *parent)
+  : QTreeView(parent),
+    showChannelList(tr("Show Channel List"), this),
+    _menuActions(this)
+{
+  setupMenuActions();
+  showChannelList.setIcon(SmallIcon("format-list-unordered"));
 
-  connect(this, SIGNAL(customContextMenuRequested(const QPoint &)),
-          this, SLOT(showContextMenu(const QPoint &)));
+  connect(this, SIGNAL(collapsed(const QModelIndex &)), SLOT(on_collapse(const QModelIndex &)));
+  connect(this, SIGNAL(expanded(const QModelIndex &)), SLOT(on_expand(const QModelIndex &)));
 
   setSelectionMode(QAbstractItemView::ExtendedSelection);
 }
@@ -72,21 +78,39 @@ void BufferView::init() {
 
   setSortingEnabled(true);
   sortByColumn(0, Qt::AscendingOrder);
-#ifndef Q_WS_QWS
-  // this is a workaround to not join channels automatically... we need a saner way to navigate for qtopia anyway though,
-  // such as mark first, activate at second click...
-  connect(this, SIGNAL(activated(QModelIndex)), this, SLOT(joinChannel(QModelIndex)));
+
+  // activated() fails on X11 and Qtopia at least
+#if defined Q_WS_QWS or defined Q_WS_X11
+  connect(this, SIGNAL(doubleClicked(QModelIndex)), SLOT(joinChannel(QModelIndex)));
 #else
-  connect(this, SIGNAL(doubleClicked(QModelIndex)), this, SLOT(joinChannel(QModelIndex)));  // Qtopia uses single click for activation
+  // afaik this is better on Mac and Windows
+  connect(this, SIGNAL(activated(QModelIndex)), SLOT(joinChannel(QModelIndex)));
 #endif
 }
 
+void BufferView::setupMenuActions() {
+  _menuActions.addAction("ConnectNet", new Action(SmallIcon("network-connect"), tr("Connect"), this));
+  _menuActions.addAction("DisconnectNet", new Action(SmallIcon("network-disconnect"), tr("Disconnect"), this));
+  _menuActions.addAction("JoinChannel", new Action(tr("Join Channel"), this));
+  _menuActions.addAction("JoinBuffer", new Action(tr("Join"), this));
+  _menuActions.addAction("PartBuffer", new Action(tr("Part"), this));
+  _menuActions.addAction("HideBuffersTemporarily", new Action(tr("Hide Buffers"), this));
+  _menuActions.addAction("HideBuffersPermanently", new Action(tr("Hide Buffers Permanently"), this));
+  _menuActions.addAction("RemoveBuffer", new Action(tr("Delete Buffer"), this));
+  _menuActions.addAction("IgnoreList", new Action(tr("Ignore List"), this));
+  _menuActions.addAction("HideJoin", new Action(tr("Joins"), this))->setCheckable(true);
+  _menuActions.addAction("HidePart", new Action(tr("Parts"), this))->setCheckable(true);
+  _menuActions.addAction("HideQuit", new Action(tr("Quits"), this))->setCheckable(true);
+  _menuActions.addAction("HideNick", new Action(tr("Nick Changes"), this))->setCheckable(true);
+  _menuActions.addAction("HideMode", new Action(tr("Mode Changes"), this))->setCheckable(true);
+  _menuActions.addAction("HideDayChange", new Action(tr("Day Change"), this))->setCheckable(true);
+
+  connect(&_menuActions, SIGNAL(actionTriggered(QAction *)), SLOT(actionTriggered(QAction *)));
+}
+
 void BufferView::setModel(QAbstractItemModel *model) {
   delete selectionModel();
-  if(QTreeView::model()) {
-    disconnect(QTreeView::model(), SIGNAL(layoutChanged()), this, SLOT(layoutChanged()));
-  }
-  
+
   QTreeView::setModel(model);
   init();
   // remove old Actions
@@ -99,8 +123,6 @@ void BufferView::setModel(QAbstractItemModel *model) {
   if(!model)
     return;
 
-  connect(model, SIGNAL(layoutChanged()), this, SLOT(layoutChanged()));
-  
   QString sectionName;
   QAction *showSection;
   for(int i = 1; i < model->columnCount(); i++) {
@@ -112,7 +134,7 @@ void BufferView::setModel(QAbstractItemModel *model) {
     connect(showSection, SIGNAL(toggled(bool)), this, SLOT(toggleHeader(bool)));
     header()->addAction(showSection);
   }
-  
+
 }
 
 void BufferView::setFilteredModel(QAbstractItemModel *model_, BufferViewConfig *config) {
@@ -125,6 +147,7 @@ void BufferView::setFilteredModel(QAbstractItemModel *model_, BufferViewConfig *
 
   if(model()) {
     disconnect(this, 0, model(), 0);
+    disconnect(model(), 0, this, 0);
   }
 
   if(!model_) {
@@ -132,8 +155,7 @@ void BufferView::setFilteredModel(QAbstractItemModel *model_, BufferViewConfig *
   } else {
     BufferViewFilter *filter = new BufferViewFilter(model_, config);
     setModel(filter);
-    connect(this, SIGNAL(removeBuffer(const QModelIndex &)),
-           filter, SLOT(removeBuffer(const QModelIndex &)));
+    connect(filter, SIGNAL(configChanged()), this, SLOT(on_configChanged()));
   }
   setConfig(config);
 }
@@ -141,20 +163,20 @@ void BufferView::setFilteredModel(QAbstractItemModel *model_, BufferViewConfig *
 void BufferView::setSelectionModel(QItemSelectionModel *selectionModel) {
   if(QTreeView::selectionModel())
     disconnect(selectionModel, SIGNAL(currentChanged(QModelIndex, QModelIndex)),
-              model(), SIGNAL(checkPreviousCurrentForRemoval(QModelIndex, QModelIndex)));
-    
+               model(), SIGNAL(checkPreviousCurrentForRemoval(QModelIndex, QModelIndex)));
+
   QTreeView::setSelectionModel(selectionModel);
   BufferViewFilter *filter = qobject_cast<BufferViewFilter *>(model());
   if(filter) {
     connect(selectionModel, SIGNAL(currentChanged(QModelIndex, QModelIndex)),
-           filter, SLOT(checkPreviousCurrentForRemoval(QModelIndex, QModelIndex)));
+            filter, SLOT(checkPreviousCurrentForRemoval(QModelIndex, QModelIndex)));
   }
 }
 
 void BufferView::setConfig(BufferViewConfig *config) {
   if(_config == config)
     return;
-  
+
   if(_config) {
     disconnect(_config, 0, this, 0);
   }
@@ -177,7 +199,7 @@ void BufferView::setRootIndexForNetworkId(const NetworkId &networkId) {
     for(int i = 0; i < networkCount; i++) {
       child = model()->index(i, 0);
       if(networkId == model()->data(child, NetworkModel::NetworkIdRole).value<NetworkId>())
-       setRootIndex(child);
+        setRootIndex(child);
     }
   }
 }
@@ -189,7 +211,7 @@ void BufferView::joinChannel(const QModelIndex &index) {
     return;
 
   BufferInfo bufferInfo = index.data(NetworkModel::BufferInfoRole).value<BufferInfo>();
-  
+
   Client::userInput(bufferInfo, QString("/JOIN %1").arg(bufferInfo.bufferName()));
 }
 
@@ -201,13 +223,26 @@ void BufferView::keyPressEvent(QKeyEvent *event) {
   QTreeView::keyPressEvent(event);
 }
 
-void BufferView::removeSelectedBuffers() {
-  QSet<int> removedRows;
+void BufferView::removeSelectedBuffers(bool permanently) {
+  if(!config())
+    return;
+
+  BufferId bufferId;
+  QSet<BufferId> removedRows;
   foreach(QModelIndex index, selectionModel()->selectedIndexes()) {
-    if(index.data(NetworkModel::ItemTypeRole) == NetworkModel::BufferItemType && !removedRows.contains(index.row())) {
-      removedRows << index.row();
-      emit removeBuffer(index);
-    }
+    if(index.data(NetworkModel::ItemTypeRole) != NetworkModel::BufferItemType)
+      continue;
+
+    bufferId = index.data(NetworkModel::BufferIdRole).value<BufferId>();
+    if(removedRows.contains(bufferId))
+      continue;
+
+    removedRows << bufferId;
+
+    if(permanently)
+      config()->requestRemoveBufferPermanently(bufferId);
+    else
+      config()->requestRemoveBuffer(bufferId);
   }
 }
 
@@ -217,7 +252,7 @@ void BufferView::rowsInserted(const QModelIndex & parent, int start, int end) {
   // ensure that newly inserted network nodes are expanded per default
   if(parent.data(NetworkModel::ItemTypeRole) != NetworkModel::NetworkItemType)
     return;
-  
+
   if(model()->rowCount(parent) == 1 && parent.data(NetworkModel::ItemActiveRole) == true) {
     // without updating the parent the expand will have no effect... Qt Bug?
     update(parent);
@@ -225,19 +260,33 @@ void BufferView::rowsInserted(const QModelIndex & parent, int start, int end) {
   }
 }
 
-void BufferView::layoutChanged() {
+void BufferView::on_configChanged() {
   Q_ASSERT(model());
 
-  // expand all active networks
+  // expand all active networks... collapse inactive ones... unless manually changed
   QModelIndex networkIdx;
+  NetworkId networkId;
   for(int row = 0; row < model()->rowCount(); row++) {
     networkIdx = model()->index(row, 0);
+    if(model()->rowCount(networkIdx) ==  0)
+      continue;
+
+    networkId = model()->data(networkIdx, NetworkModel::NetworkIdRole).value<NetworkId>();
+    if(!networkId.isValid())
+      continue;
+
     update(networkIdx);
-    if(model()->rowCount(networkIdx) > 0 && model()->data(networkIdx, NetworkModel::ItemActiveRole) == true) {
+
+    bool expandNetwork = false;
+    if(_expandedState.contains(networkId))
+      expandNetwork = _expandedState[networkId];
+    else
+      expandNetwork = model()->data(networkIdx, NetworkModel::ItemActiveRole).toBool();
+
+    if(expandNetwork)
       expand(networkIdx);
-    } else {
+    else
       collapse(networkIdx);
-    }
   }
 
   // update selection to current one
@@ -249,146 +298,260 @@ void BufferView::layoutChanged() {
   mappedSelectionModel->mappedSelect(Client::bufferModel()->standardSelectionModel()->selection(), QItemSelectionModel::ClearAndSelect | QItemSelectionModel::Rows);
 }
 
+void BufferView::on_collapse(const QModelIndex &index) {
+  storeExpandedState(index.data(NetworkModel::NetworkIdRole).value<NetworkId>(), false);
+}
+
+void BufferView::on_expand(const QModelIndex &index) {
+  storeExpandedState(index.data(NetworkModel::NetworkIdRole).value<NetworkId>(), true);
+}
+
+void BufferView::storeExpandedState(NetworkId networkId, bool expanded) {
+  _expandedState[networkId] = expanded;
+}
+
 void BufferView::dataChanged(const QModelIndex &topLeft, const QModelIndex &bottomRight) {
   QTreeView::dataChanged(topLeft, bottomRight);
-  
+
   // determine how many items have been changed and if any of them is a networkitem
   // which just swichted from active to inactive or vice versa
   if(topLeft.data(NetworkModel::ItemTypeRole) != NetworkModel::NetworkItemType)
     return;
 
   for(int i = topLeft.row(); i <= bottomRight.row(); i++) {
-    QModelIndex networkIdx = topLeft.sibling(topLeft.row(), 0);
+    QModelIndex networkIdx = topLeft.sibling(i, 0);
     if(model()->rowCount(networkIdx) == 0)
       continue;
 
     bool isActive = networkIdx.data(NetworkModel::ItemActiveRole).toBool();
+#ifdef SPUTDEV
+    if(isExpanded(networkIdx) != isActive) setExpanded(networkIdx, true);
+#else
     if(isExpanded(networkIdx) != isActive) setExpanded(networkIdx, isActive);
+#endif
   }
 }
 
-
 void BufferView::toggleHeader(bool checked) {
   QAction *action = qobject_cast<QAction *>(sender());
   header()->setSectionHidden((action->property("column")).toInt(), !checked);
 }
 
-void BufferView::showContextMenu(const QPoint &pos) {
-  QModelIndex index = indexAt(pos);
-  if(!index.isValid()) return;
-  QMenu contextMenu(this);
-  QAction *connectNetAction = contextMenu.addAction(tr("Connect"));
-  QAction *disconnectNetAction = contextMenu.addAction(tr("Disconnect"));
-  QAction *joinChannelAction = contextMenu.addAction(tr("Join Channel"));
-
-  QAction *joinBufferAction = contextMenu.addAction(tr("Join"));
-  QAction *partBufferAction = contextMenu.addAction(tr("Part"));
-  QAction *hideBufferAction = contextMenu.addAction(tr("Remove buffers"));
-  hideBufferAction->setToolTip(tr("Removes the selected buffers from a custom view but leaves the buffer itself untouched"));
-  QAction *removeBufferAction = contextMenu.addAction(tr("Delete buffer"));
-
-  QMenu *hideEventsMenu = contextMenu.addMenu(tr("Hide Events"));
-  QAction *hideJoinAction = hideEventsMenu->addAction(tr("Join Events"));
-  QAction *hidePartAction = hideEventsMenu->addAction(tr("Part Events"));
-  QAction *hideKillAction = hideEventsMenu->addAction(tr("Kill Events"));
-  QAction *hideQuitAction = hideEventsMenu->addAction(tr("Quit Events"));
-  QAction *hideModeAction = hideEventsMenu->addAction(tr("Mode Events"));
-  hideJoinAction->setCheckable(true);
-  hidePartAction->setCheckable(true);
-  hideKillAction->setCheckable(true);
-  hideQuitAction->setCheckable(true);
-  hideModeAction->setCheckable(true);
-  hideJoinAction->setEnabled(false);
-  hidePartAction->setEnabled(false);
-  hideKillAction->setEnabled(false);
-  hideQuitAction->setEnabled(false);
-  hideModeAction->setEnabled(false);
-
-  QAction *ignoreListAction = new QAction(tr("Ignore list"), this);
-  ignoreListAction->setEnabled(false);
-  QAction *whoBufferAction = new QAction(tr("WHO"), this);
-
-  if(index.data(NetworkModel::ItemTypeRole) == NetworkModel::NetworkItemType) {
-    if(index.data(NetworkModel::ItemActiveRole).toBool()) {
-      contextMenu.addAction(disconnectNetAction);
-      contextMenu.addSeparator();
-      contextMenu.addAction(joinChannelAction);
-    } else {
-      contextMenu.addAction(connectNetAction);
-    }
+bool BufferView::checkRequirements(const QModelIndex &index, ItemActiveStates requiredActiveState) {
+  if(!index.isValid())
+    return false;
+
+  ItemActiveStates isActive = index.data(NetworkModel::ItemActiveRole).toBool()
+    ? ActiveState
+    : InactiveState;
+
+  if(!(isActive & requiredActiveState))
+    return false;
+
+  return true;
+}
+
+void BufferView::addItemToMenu(QAction *action, QMenu *menu, const QModelIndex &index, ItemActiveStates requiredActiveState) {
+  addItemToMenu(action, menu, checkRequirements(index, requiredActiveState));
+}
+
+void BufferView::addItemToMenu(QAction *action, QMenu *menu, bool condition) {
+  if(condition) {
+    menu->addAction(action);
+    action->setVisible(true);
+  } else {
+    action->setVisible(false);
   }
+}
 
-  BufferInfo bufferInfo = index.data(NetworkModel::BufferInfoRole).value<BufferInfo>();
-  QString channelname = index.sibling(index.row(), 0).data().toString();
-
-  if(index.data(NetworkModel::ItemTypeRole) == NetworkModel::BufferItemType) {
-    if(bufferInfo.type() != BufferInfo::ChannelBuffer && bufferInfo.type() != BufferInfo::QueryBuffer) return;
-    contextMenu.addAction(joinBufferAction);
-    contextMenu.addAction(partBufferAction);
-    if(config())
-      contextMenu.addAction(hideBufferAction);
-    contextMenu.addAction(removeBufferAction);
-    contextMenu.addMenu(hideEventsMenu);
-    contextMenu.addAction(ignoreListAction);
-    contextMenu.addAction(whoBufferAction);
-
-    if(bufferInfo.type() == BufferInfo::ChannelBuffer) {
-      if(index.data(NetworkModel::ItemActiveRole).toBool()) {
-        removeBufferAction->setEnabled(false);
-        removeBufferAction->setToolTip("To delete the buffer, part the channel first.");
-        joinBufferAction->setVisible(false);
-        whoBufferAction->setVisible(false);
-      } else {
-        partBufferAction->setVisible(false);
+void BufferView::addItemToMenu(QMenu *subMenu, QMenu *menu, const QModelIndex &index, ItemActiveStates requiredActiveState) {
+  if(checkRequirements(index, requiredActiveState)) {
+    menu->addMenu(subMenu);
+    subMenu->setVisible(true);
+  } else {
+    subMenu->setVisible(false);
+  }
+}
+
+void BufferView::addSeparatorToMenu(QMenu *menu, const QModelIndex &index, ItemActiveStates requiredActiveState) {
+  if(checkRequirements(index, requiredActiveState)) {
+    menu->addSeparator();
+  }
+}
+
+QMenu *BufferView::createHideEventsSubMenu(QMenu *menu, BufferId bufferId) {
+  int filter = BufferSettings(bufferId).messageFilter();
+  _menuActions.action("HideJoin")->setChecked(filter & Message::Join);
+  _menuActions.action("HidePart")->setChecked(filter & Message::Part);
+  _menuActions.action("HideQuit")->setChecked(filter & Message::Quit);
+  _menuActions.action("HideNick")->setChecked(filter & Message::Nick);
+  _menuActions.action("HideMode")->setChecked(filter & Message::Mode);
+  _menuActions.action("HideDayChange")->setChecked(filter & Message::DayChange);
+
+  QMenu *hideEventsMenu = menu->addMenu(tr("Hide Events"));
+  hideEventsMenu->addAction(_menuActions.action("HideJoin"));
+  hideEventsMenu->addAction(_menuActions.action("HidePart"));
+  hideEventsMenu->addAction(_menuActions.action("HideQuit"));
+  hideEventsMenu->addAction(_menuActions.action("HideNick"));
+  hideEventsMenu->addAction(_menuActions.action("HideMode"));
+  hideEventsMenu->addAction(_menuActions.action("HideDayChange"));
+  return hideEventsMenu;
+}
+
+void BufferView::contextMenuEvent(QContextMenuEvent *event) {
+  _menuIndex = indexAt(event->pos());
+  if(!_menuIndex.isValid())
+    _menuIndex = rootIndex();
+  if(!_menuIndex.isValid())
+    return;
+
+  QMenu contextMenu(this);
+  addActionsToMenu(&contextMenu, _menuIndex);
+  if(!contextMenu.actions().isEmpty())
+    contextMenu.exec(QCursor::pos());
+
+  _menuIndex = QModelIndex();
+}
+
+void BufferView::addActionsToMenu(QMenu *contextMenu, const QModelIndex &index) {
+  const Network *network = Client::network(index.data(NetworkModel::NetworkIdRole).value<NetworkId>());
+  Q_CHECK_PTR(network);
+
+  NetworkModel::itemType itemType = static_cast<NetworkModel::itemType>(index.data(NetworkModel::ItemTypeRole).toInt());
+
+  switch(itemType) {
+  case NetworkModel::NetworkItemType:
+    showChannelList.setData(index.data(NetworkModel::NetworkIdRole));
+    addItemToMenu(&showChannelList, contextMenu, index, ActiveState);
+    addItemToMenu(_menuActions.action("DisconnectNet"), contextMenu, network->connectionState() != Network::Disconnected);
+    addItemToMenu(_menuActions.action("ConnectNet"), contextMenu, network->connectionState() == Network::Disconnected);
+    addSeparatorToMenu(contextMenu, index, ActiveState);
+    addItemToMenu(_menuActions.action("JoinChannel"), contextMenu, index, ActiveState);
+    break;
+  case NetworkModel::BufferItemType:
+    {
+      BufferInfo bufferInfo = index.data(NetworkModel::BufferInfoRole).value<BufferInfo>();
+      switch(bufferInfo.type()) {
+      case BufferInfo::ChannelBuffer:
+        addItemToMenu(_menuActions.action("JoinBuffer"), contextMenu, index, InactiveState);
+        addItemToMenu(_menuActions.action("PartBuffer"), contextMenu, index, ActiveState);
+        addItemToMenu(_menuActions.action("HideBuffersTemporarily"), contextMenu, (bool)config());
+        addItemToMenu(_menuActions.action("HideBuffersPermanently"), contextMenu, (bool)config());
+        addItemToMenu(_menuActions.action("RemoveBuffer"), contextMenu, index, InactiveState);
+        createHideEventsSubMenu(contextMenu, bufferInfo.bufferId());
+        addItemToMenu(_menuActions.action("IgnoreList"), contextMenu);
+        break;
+      case BufferInfo::QueryBuffer:
+        addItemToMenu(_menuActions.action("HideBuffersTemporarily"), contextMenu, (bool)config());
+        addItemToMenu(_menuActions.action("HideBuffersPermanently"), contextMenu, (bool)config());
+        addItemToMenu(_menuActions.action("RemoveBuffer"), contextMenu);
+        createHideEventsSubMenu(contextMenu, bufferInfo.bufferId());
+        break;
+      default:
+        addItemToMenu(_menuActions.action("HideBuffersTemporarily"), contextMenu, (bool)config());
+        addItemToMenu(_menuActions.action("HideBuffersPermanently"), contextMenu, (bool)config());
+        break;
       }
-    } else {
-      joinBufferAction->setVisible(false);
-      partBufferAction->setVisible(false);
     }
+    break;
+  default:
+    return;
   }
+}
+
+void BufferView::actionTriggered(QAction *result) {
+  const Network *network = Client::network(_menuIndex.data(NetworkModel::NetworkIdRole).value<NetworkId>());
+  Q_CHECK_PTR(network);
 
-  QAction *result = contextMenu.exec(QCursor::pos());
-  if(result == connectNetAction || result == disconnectNetAction) {
-    const Network *network = Client::network(index.data(NetworkModel::NetworkIdRole).value<NetworkId>());
-    if(!network) return;
-    if(network->connectionState() == Network::Disconnected) 
-      network->requestConnect();
-    else 
-      network->requestDisconnect();
-  } else
-  if(result == joinChannelAction) {
+  if(network && result == _menuActions.action("ConnectNet")) {
+    network->requestConnect();
+    return;
+  }
+
+  if(network && result == _menuActions.action("DisconnectNet")) {
+    network->requestDisconnect();
+    return;
+  }
+
+if(result == _menuActions.action("JoinChannel")) {
     // FIXME no QInputDialog in Qtopia
 #ifndef Q_WS_QWS
     bool ok;
-    QString channelName = QInputDialog::getText(this, tr("Join Channel"), 
-                                                tr("Input channel name:"),QLineEdit::Normal,
-                                                QDir::home().dirName(), &ok);
-
-    if (ok && !channelName.isEmpty()) {
-      BufferInfo bufferInfo = index.child(0,0).data(NetworkModel::BufferInfoRole).value<BufferInfo>();
-      if(bufferInfo.isValid()) {
-        Client::instance()->userInput(bufferInfo, QString("/J %1").arg(channelName));
-      }
+    QString channelName = QInputDialog::getText(this, tr("Join Channel"), tr("Input channel name:"), QLineEdit::Normal, QString(), &ok);
+    if(ok && !channelName.isEmpty()) {
+      Client::instance()->userInput(BufferInfo::fakeStatusBuffer(_menuIndex.data(NetworkModel::NetworkIdRole).value<NetworkId>()),
+                                     QString("/J %1").arg(channelName));
     }
 #endif
-  } else if(result == joinBufferAction) {
-    Client::instance()->userInput(bufferInfo, QString("/JOIN %1").arg(channelname));
-  } else if(result == partBufferAction) {
-    Client::instance()->userInput(bufferInfo, QString("/PART %1").arg(channelname));
-  } else if(result == hideBufferAction) {
+    return;
+  }
+
+  if(result == _menuActions.action("JoinBuffer")) {
+    BufferInfo bufferInfo = _menuIndex.data(NetworkModel::BufferInfoRole).value<BufferInfo>();
+    Client::instance()->userInput(bufferInfo, QString("/JOIN %1").arg(bufferInfo.bufferName()));
+    return;
+  }
+
+  if(result == _menuActions.action("PartBuffer")) {
+    BufferInfo bufferInfo = _menuIndex.data(NetworkModel::BufferInfoRole).value<BufferInfo>();
+    Client::instance()->userInput(bufferInfo, QString("/PART"));
+    return;
+  }
+
+  if(result == _menuActions.action("HideBuffersTemporarily")) {
     removeSelectedBuffers();
-  } else if(result == removeBufferAction) {
+    return;
+  }
+
+  if(result == _menuActions.action("HideBuffersPermanently")) {
+    removeSelectedBuffers(true);
+    return;
+  }
+
+  if(result == _menuActions.action("RemoveBuffer")) {
+    BufferInfo bufferInfo = _menuIndex.data(NetworkModel::BufferInfoRole).value<BufferInfo>();
     int res = QMessageBox::question(this, tr("Remove buffer permanently?"),
                                     tr("Do you want to delete the buffer \"%1\" permanently? This will delete all related data, including all backlog "
                                        "data, from the core's database!").arg(bufferInfo.bufferName()),
                                         QMessageBox::Yes|QMessageBox::No, QMessageBox::No);
     if(res == QMessageBox::Yes) {
       Client::removeBuffer(bufferInfo.bufferId());
-    } 
-  } else 
-  if(result == whoBufferAction) {
-    Client::instance()->userInput(bufferInfo, QString("/WHO %1").arg(channelname));
+    }
+    return;
+  }
+
+  if(result == _menuActions.action("HideJoin")) {
+    BufferId bufferId = _menuIndex.data(NetworkModel::BufferIdRole).value<BufferId>();
+    BufferSettings(bufferId).filterMessage(Message::Join, result->isChecked());
+    return;
   }
+  if(result == _menuActions.action("HidePart")) {
+    BufferId bufferId = _menuIndex.data(NetworkModel::BufferIdRole).value<BufferId>();
+    BufferSettings(bufferId).filterMessage(Message::Part, result->isChecked());
+    return;
+  }
+  if(result == _menuActions.action("HideQuit")) {
+    BufferId bufferId = _menuIndex.data(NetworkModel::BufferIdRole).value<BufferId>();
+    BufferSettings(bufferId).filterMessage(Message::Quit, result->isChecked());
+    return;
+  }
+  if(result == _menuActions.action("HideNick")) {
+    BufferId bufferId = _menuIndex.data(NetworkModel::BufferIdRole).value<BufferId>();
+    BufferSettings(bufferId).filterMessage(Message::Nick, result->isChecked());
+    return;
+  }
+  if(result == _menuActions.action("HideMode")) {
+    BufferId bufferId = _menuIndex.data(NetworkModel::BufferIdRole).value<BufferId>();
+    BufferSettings(bufferId).filterMessage(Message::Mode, result->isChecked());
+    return;
+  }
+  if(result == _menuActions.action("HideDayChange")) {
+    BufferId bufferId = _menuIndex.data(NetworkModel::BufferIdRole).value<BufferId>();
+    BufferSettings(bufferId).filterMessage(Message::DayChange, result->isChecked());
+    return;
+  }
+
+  qWarning() << "Unhandled menu action";
 }
 
 void BufferView::wheelEvent(QWheelEvent* event) {
@@ -415,13 +578,13 @@ void BufferView::wheelEvent(QWheelEvent* event) {
       }
   selectionModel()->setCurrentIndex( resultingIndex, QItemSelectionModel::ClearAndSelect | QItemSelectionModel::Rows );
   selectionModel()->select( resultingIndex, QItemSelectionModel::ClearAndSelect );
-  
+
 }
 
 
 QSize BufferView::sizeHint() const {
   return QTreeView::sizeHint();
-  
+
   if(!model())
     return QTreeView::sizeHint();