<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/113700/">http://git.reviewboard.kde.org/r/113700/</a>

     </td>

    </tr>

   </table>

   <br />

 <p>Ship it!</p>

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">good catch.. doesn't look easy to catch ;)</pre>

 <br />

<p>- Marco Martin</p>

<br />

<p>On November 7th, 2013, 8:59 a.m. UTC, Martin Gräßlin wrote:</p>

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

 <tr>

  <td>

<div>Review request for Plasma.</div>

<div>By Martin Gräßlin.</div>

<p style="color: grey;"><i>Updated Nov. 7, 2013, 8:59 a.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

plasma-framework

</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;">Calling DialogShadows::self()->addWindow(...) will create the low level window.

If this is called before the property windowFlags is evaluated the window flag

X11BypassWindowManagerHint has no effect as that one must be present when the

window is created through the low level xcb call.

This was breaking declarative KWin scripts as KWin's windows need to have the

X11BypassWindowManagerHint.</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/declarativeimports/core/dialog.cpp <span style="color: grey">(ba43404)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>