<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://reviewboard.kde.org/r/5035/">http://reviewboard.kde.org/r/5035/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On August 28th, 2010, 2:56 p.m., <b>Todd</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;">Would it be possible to also do it with middle-click?  This is unused for that widget as far as I can tell, and would avoid needing to use the keyboard entirely.</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;">that would be even more easy to trigger accidentally and raise the possibility of this being found not on purpose. imagine what someone who accidentally middle clicks and triggers this behaviour is going to think? most people won&#39;t connect &quot;I just middle clicked on the representation of a window&quot; with the present windows effect being triggered, and if they do it will likely come across as slightly odd. middle clicking doesn&#39;t have a &quot;bring forward and show&quot; semantic anywhere else afaik, ergo the unexpectedness.

so, no. :)

(tangentially, the same reason i don&#39;t like this feature is the same reason i veto&#39;d the concept of scroll wheel over the battery plasmoid adjusting backlight brightness.)</pre>
<br />








<p>- Aaron</p>


<br />
<p>On August 28th, 2010, 7:49 a.m., Martin Gräßlin wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Plasma, Aaron Seigo and Marco Martin.</div>
<div>By Martin Gräßlin.</div>


<p style="color: grey;"><i>Updated 2010-08-28 07:49:57</i></p>




<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;">This patch adds support to pager to trigger the present windows effect for the desktop the user clicks on when holding ctrl. This is similar to what we have when clicking on a tasks group when ctrl is hold.

I&#39;m not sure if this is a too hidden feature, but I consider it as could be useful and consistent with the tasks applet.</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>trunk/KDE/kdebase/workspace/plasma/desktop/applets/pager/pager.cpp <span style="color: grey">(1163977)</span></li>

</ul>

<p><a href="http://reviewboard.kde.org/r/5035/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>