<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/103529/">http://git.reviewboard.kde.org/r/103529/</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 4d5474054353341fb35fac81ca441c246515792e by Nikita Churaev to branch KDE/4.8.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On December 24th, 2011, 5:35 p.m., Nikita Churaev 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 kdelibs and Plasma.</div>
<div>By Nikita Churaev.</div>


<p style="color: grey;"><i>Updated Dec. 24, 2011, 5:35 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;">Contrast bug: KGlobalSettings::contrast() returns different default value than KGlobalSettings::contrastF(). Oxygen uses contrastF() while KDE color control module uses contrast(), so when the user first uses color settings, contrast appears to change.

Plasma popup bug: Right-aligned popups are one pixel away from right edge of the screen and top-aligned popups (when panel is on top) are one pixel inside panel. This is because the bug in QRect, where right() and bottom() value return value that is less than the true value by one. This is a feature-bug that Qt developers aren't going to fix because of compatiability reasons. My patch just applies + 1 compensation to all expressions that use QRect::right() and QRect::bottom().</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>kdeui/kernel/kglobalsettings.cpp <span style="color: grey">(819b314)</span></li>

 <li>plasma/corona.cpp <span style="color: grey">(366a9df)</span></li>

</ul>

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




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








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