<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://git.reviewboard.kde.org/r/106722/">http://git.reviewboard.kde.org/r/106722/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On October 4th, 2012, 1:13 p.m., <b>Thomas Lübking</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;">Special case solution?

This will (by assumed behavior, didn't look into details) for quite sure trigger bug reports ("can't switch desktops", "present windows no longer works with fullscreen <browser_here>"), so here's a warning about that ;-)

If some client wants to ensure unobscured input it can grab the pointer, what would also work with other WMs then...</pre>
 </blockquote>




 <p>On October 4th, 2012, 1:24 p.m., <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;">actually it's not a special case solution. Looking at it, I see:
* games which grab mouse
* applications which have additional UI hidden at the top

>From the apps I sometimes use in fullscreen, these are:
* Gwenview
* Okular
* Browsers
* VMs
* VNC Viewer

So it's actually a quite common feature. In fact I do not know any application not doing it.

Concerning the possible complaints: I cannot remember a bug report about games preventing present windows.

But it's not a big deal to add a config option to it.</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;">you do not want to drop out of an ego shooter and be dead when you return, so they grab the input.
this is different with other applications.

reg. the other apps, this does rather imply edges (so most of the time desktop switching, iff) and not the corners.

reg. the option, there's still bug #182987 and (was) #293011

imo the major aspect here is however, that kwin fixes a problem of those clients which will remain for other WMs

that way, we could also just introduce KDE_NETWM_OCCUPY_EDGE or so</pre>
<br />








<p>- Thomas</p>


<br />
<p>On October 4th, 2012, 1:02 p.m., Martin Gräßlin wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 kwin, Plasma and Aurélien Gâteau.</div>
<div>By Martin Gräßlin.</div>


<p style="color: grey;"><i>Updated Oct. 4, 2012, 1:02 p.m.</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;">Block screen edge activation for fullscreen windows

If the active client is a fullscreen window, the screenedges on the
screen the window is on get's blocked for activation.

We have to assume that the window is managing the screenedge itself to
offer there additional functionality (e.g. a panel) and KWin's screen
edge functionality would conflict.</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>kwin/screenedge.cpp <span style="color: grey">(a1507c3543877477e564bba368feaa31bc44655e)</span></li>

</ul>

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




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








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