<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/128362/">https://git.reviewboard.kde.org/r/128362/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 5th, 2016, 2:32 p.m. UTC, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I do not understand why this review was discarded. Care to elaborate?</p></pre>
</blockquote>
<p>On July 5th, 2016, 4:24 p.m. UTC, <b>Eike Hein</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Based on IRC discussion neither you or I are are available to write the favored implementation so I decided to abandon the proposed code.</p></pre>
</blockquote>
<p>On July 6th, 2016, 6:18 a.m. UTC, <b>Martin Gräßlin</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">But then the stringly based might be better. I don't see why you discarded the review.</p></pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I think your suggestion probably makes sense (considering the reason the actions broke seems to be that they used "org.kde.kwin", which I fixed up to "org.kde.KWin" ...), but I don't know how to do it so the time/effort required outweighed the need I see for the code.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I think Unclutter and Cascade are probably not meaningfully useful anymore in an era of things like the Present Windows effect or Alt+Tab with thumbnails or the majority of users who prefer maximized windows now:</p>
<ol style="padding: 0;text-rendering: inherit;margin: 0 0 0 2em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Unclutter: This is probably mostly to un-obscure window contents, but Present Windows does this much better, without rearranging windows in an essentially unpredictable manner. Plus the implementation isn't very nice - it ends up actually moving the panel popup dialog as well.</p>
</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Cascade: I think this is to arrange windows to have the /title bar/ unobscured so they can be subsequently handled more easily on an individual basis. I'm not sure how many users actually do this anymore, though. Even if they do, Present Windows (which is on a hot corner by default) offers a convenient way to get a handle of an obscured window.</p>
</li>
</ol>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I was OK with fixing/keeping the actions given minimal required effort, but it enters a territory of diminished returns very quickly given the above. And the applet might well be better for it: Why should there be buttons just in this one applet for Unclutter and Cascade? Why not Present Windows or Show Desktop or Minimize All? Or even actions that operate on the selected item? Basically the inclusion of them seems somewhat arbitrary, and just to continue a legacy. Which already got interrupted when they were broken for many releases without leading to any bug reports.</p></pre>
<br />
<p>- Eike</p>
<br />
<p>On July 5th, 2016, 2:13 p.m. UTC, Eike Hein wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Eike Hein.</div>
<p style="color: grey;"><i>Updated July 5, 2016, 2:13 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdeclarative
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is the remaining useful code from the Tasks engine, which is now scheduled for removal. It will be used by the Window List plasmoid.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">As discussed on IRC.</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/qmlcontrols/kwindowsystemplugin/CMakeLists.txt <span style="color: grey">(ce0ea74)</span></li>
<li>src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.h <span style="color: grey">(a9db965)</span></li>
<li>src/qmlcontrols/kwindowsystemplugin/kwindowsystemproxy.cpp <span style="color: grey">(4850011)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128362/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>