[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    [mailcommon] src/folder: Make sure that "View columns" menu entry is not show when we don't have
From:       Laurent Montel <null () kde ! org>
Date:       2018-09-11 19:02:49
Message-ID: E1fznw1-00075V-33 () code ! kde ! org
[Download RAW message or body]

Git commit 18c67f3e1069b2ab01551c63623fbb5a39a16bcb by Laurent Montel.
Committed on 11/09/2018 at 19:02.
Pushed by mlaurent into branch 'master'.

Make sure that "View columns" menu entry is not show when we don't have

columns

M  +9    -7    src/folder/foldertreeview.cpp

https://commits.kde.org/mailcommon/18c67f3e1069b2ab01551c63623fbb5a39a16bcb

diff --git a/src/folder/foldertreeview.cpp b/src/folder/foldertreeview.cpp
index cc2d134..748df2c 100644
--- a/src/folder/foldertreeview.cpp
+++ b/src/folder/foldertreeview.cpp
@@ -145,14 +145,16 @@ void FolderTreeView::slotHeaderContextMenuRequested(const QPoint &pnt)
     // the menu for the columns
     QMenu menu;
     QAction *act = nullptr;
-    menu.addSection(i18n("View Columns"));
     const int nbColumn = header()->count();
-    for (int i = 1; i < nbColumn; ++i) {
-        act = menu.addAction(model()->headerData(i, Qt::Horizontal).toString());
-        act->setCheckable(true);
-        act->setChecked(!header()->isSectionHidden(i));
-        act->setData(QVariant(i));
-        connect(act, &QAction::triggered, this, &FolderTreeView::slotHeaderContextMenuChangeHeader);
+    if (nbColumn > 1) {
+        menu.addSection(i18n("View Columns"));
+        for (int i = 1; i < nbColumn; ++i) {
+            act = menu.addAction(model()->headerData(i, Qt::Horizontal).toString());
+            act->setCheckable(true);
+            act->setChecked(!header()->isSectionHidden(i));
+            act->setData(QVariant(i));
+            connect(act, &QAction::triggered, this, &FolderTreeView::slotHeaderContextMenuChangeHeader);
+        }
     }
 
     menu.addSection(i18n("Icon Size"));
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic