From d6e10a5942442f8423385ff1301c9163b334e4ed Mon Sep 17 00:00:00 2001 From: Peter Penz Date: Wed, 14 Mar 2012 16:51:21 +0100 Subject: Use sub-menus for the "Sort By"- and "Additional Information"-menu This allows doing some grouping for the visible roles by e.g. Image, Document, Music or whatever. --- src/views/dolphinview.cpp | 28 ++++++++++++++++++++++------ 1 file changed, 22 insertions(+), 6 deletions(-) (limited to 'src/views/dolphinview.cpp') diff --git a/src/views/dolphinview.cpp b/src/views/dolphinview.cpp index 890a39d89..29a132184 100644 --- a/src/views/dolphinview.cpp +++ b/src/views/dolphinview.cpp @@ -779,17 +779,33 @@ void DolphinView::slotHeaderContextMenuRequested(const QPointF& pos) KItemListView* view = m_container->controller()->view(); const QSet visibleRolesSet = view->visibleRoles().toSet(); + QString groupName; + QMenu* groupMenu = 0; + // Add all roles to the menu that can be shown or hidden by the user const QList rolesInfo = KFileItemModel::rolesInformation(); foreach (const KFileItemModel::RoleInfo& info, rolesInfo) { - if (info.role != "name") { - const QString text = fileItemModel()->roleDescription(info.role); + if (info.role == "name") { + // It should not be possible to hide the "name" role + continue; + } + + const QString text = fileItemModel()->roleDescription(info.role); + QAction* action = 0; + if (info.group.isEmpty()) { + action = menu.data()->addAction(text); + } else { + if (!groupMenu || info.group != groupName) { + groupName = info.group; + groupMenu = menu.data()->addMenu(groupName); + } - QAction* action = menu.data()->addAction(text); - action->setCheckable(true); - action->setChecked(visibleRolesSet.contains(info.role)); - action->setData(info.role); + action = groupMenu->addAction(text); } + + action->setCheckable(true); + action->setChecked(visibleRolesSet.contains(info.role)); + action->setData(info.role); } QAction* action = menu.data()->exec(pos.toPoint()); -- cgit v1.3