<table><tr><td style="">hein added a comment.
</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/D13274">View Revision</a></tr></table><br /><div><div><p>Agreed on master-only.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The fact that TM entries are draggable outside their panel in the first place is a bit odd in many contexts because most of the time you're going to be looking at apps, not your desktop, and there's nothing you would actually want to or be able to drag them to.</p></blockquote>

<p>I don't think this is odd at all. It's how desktop UIs work. I can also drag tabs from Firefox outside of Firefox, but I can't just drop them everywhere. Drags are globally modal in some sense, as are, say, context menus. There's no concept of "you can only drag within this area", generally, unless you go for cursor confinement, which has a host of other downsides.</p>

<p>It works the same in other desktop operating systems, e.g. in the Mac OS dock.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I kind of preferred one of the original patches from way back yonder that only made TM entries draggable outside their panel when widgets were unlocked (<a href="https://phabricator.kde.org/D8564" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D8564</a>: Disallow drop of task manager icons outside of plasmoid when widgets are locked).</p></blockquote>

<p>This is not what the patch you like actually did. That patch disallowed <em>dropping</em> things outside the panel when it's locked, but it still, of course, allowed dragging them outside of the panel. It's basically the same as this, except that you need to unlock the panel to drop things on the desktop. I find that unnecessarily complicated and this works better.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13274">https://phabricator.kde.org/D13274</a></div></div><br /><div><strong>To: </strong>hein, davidedmundson, Zren<br /><strong>Cc: </strong>ngraham, zzag, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>