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

List:       kde-commits
Subject:    [plasma-desktop/Plasma/5.14] containments/desktop/plugins/folder: Add lock icon to desktop context m
From:       Nathaniel Graham <null () kde ! org>
Date:       2018-09-22 15:40:45
Message-ID: E1g3k1V-0000ui-RN () code ! kde ! org
[Download RAW message or body]

Git commit a5e45788fa3320bf4f3d439e1cb8cdff6de26e6b by Nathaniel Graham, on behalf of \
Alex Nemeth. Committed on 22/09/2018 at 15:40.
Pushed by ngraham into branch 'Plasma/5.14'.

Add lock icon to desktop context menu

Summary:
Having only one icon in the whole menu seemed a waste of space.
Adding a lock icon to the lock function greatly improves the scene.

Test Plan:
Before:
{F6277876}

After:
{F6277875}

Reviewers: ngraham, #vdg

Reviewed By: ngraham, #vdg

Subscribers: abetts, safaalfulaij, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D15683

M  +1    -1    containments/desktop/plugins/folder/viewpropertiesmenu.cpp

https://commits.kde.org/plasma-desktop/a5e45788fa3320bf4f3d439e1cb8cdff6de26e6b

diff --git a/containments/desktop/plugins/folder/viewpropertiesmenu.cpp \
b/containments/desktop/plugins/folder/viewpropertiesmenu.cpp index 000861e4..b89b2070 \
                100644
--- a/containments/desktop/plugins/folder/viewpropertiesmenu.cpp
+++ b/containments/desktop/plugins/folder/viewpropertiesmenu.cpp
@@ -97,7 +97,7 @@ ViewPropertiesMenu::ViewPropertiesMenu(QObject *parent) : \
                QObject(parent)
     m_previews = m_menu->addAction(QIcon::fromTheme(QStringLiteral("view-preview")), \
i18n("Show Previews"), this, &ViewPropertiesMenu::previewsChanged);  \
m_previews->setCheckable(true);  
-    m_locked = m_menu->addAction(i18n("Locked"), this, SIGNAL(lockedChanged()));
+    m_locked = m_menu->addAction(QIcon::fromTheme(QStringLiteral("lock")), \
i18n("Locked"), this, SIGNAL(lockedChanged()));  m_locked->setCheckable(true);
 }
 


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