<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: Plasma.<br />broulik set the repository for this revision to rPLASMAWORKSPACE Plasma Workspace.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2416" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Implements the FIXME there and removes the manual DBus code.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Started plasmashell, shortcuts would still show up.<br />
Removed a shortcut, restarted plasmashell (the previous code also didn't detect changes at runtime) and no shortcut was shown (hence the value(0) instead of first() as the latter requires the list to not be empty)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2416" rel="noreferrer">https://phabricator.kde.org/D2416</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>containmentactions/contextmenu/CMakeLists.txt<br />
containmentactions/contextmenu/menu.cpp<br />
containmentactions/contextmenu/menu.h</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>broulik, Plasma<br /><strong>Cc: </strong>plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>