<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="https://git.reviewboard.kde.org/r/115393/">https://git.reviewboard.kde.org/r/115393/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit d7a3a1023828e16db0f3bc1168b3a5fdd4707c5d by David Edmundson to branch master.</pre>

 <br />

<p>- Commit Hook</p>

<br />

<p>On January 30th, 2014, 1:32 p.m. UTC, David Edmundson wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 David Edmundson.</div>

<p style="color: grey;"><i>Updated Jan. 30, 2014, 1:32 p.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;">

For the WidgetExplorer we want to show the tooltips to the right of the

scrollbar in the view, not to the right of the delegate the tooltip is

for.

This patch adds a boolean flag which shifts it to the nearest point to

the visualParent which is outside the visualParent's window - in this

case outside the scrollbar.

I have also introduced a manual test for Dialog Positioning, and

includied some fixes in the original logic.

</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/tests/manual/dialog_positioning.qml <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/declarativeimports/core/dialog.cpp <span style="color: grey">(0248596)</span></li>

 <li>src/declarativeimports/core/dialog.h <span style="color: grey">(dc2b688)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>