<table><tr><td style="">ngraham requested changes to this revision.<br />ngraham added a comment.<br />This revision now requires changes to proceed.
</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/D24265">View Revision</a></tr></table><br /><div><div><p>Hmm, I have some UX concerns here after trying it out:</p>

<p>If I invoke Present Windows to see the desktop so that I can right-click on it to enter edit mode, when I exit the mode, all the windows return again unexpectedly. Exiting edit mode should only restore the windows if the Present Windows effect was invoked by entering edit more, not if it was already invoked manually by the user</p>

<p>While in Edit mode, if I click on the desktop, it exits edit mode. This feels very unnatural and I find myself accidentally exiting the mode a lot. I think it makes sense to automatically exit the mode when another application is focused, but not the Plasma desktop itself.</p>

<p>If we're eventually going to hide the "Lock Widgets" functionality from the context menu, we need to remove it from the toolbox as well, or else you can enter edit mode, lock widgets, and be unable to re-enter edit mode because the menu item to do it is disabled when widgets are locked (maybe this should be in another patch though?)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24265">https://phabricator.kde.org/D24265</a></div></div><br /><div><strong>To: </strong>mart, Plasma, ngraham<br /><strong>Cc: </strong>ngraham, GB_2, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>