<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/108340/">http://git.reviewboard.kde.org/r/108340/</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 458757543a962c57f8e2836aa484e99d0061845f by Weng Xuetian on behalf of Yichao Yu to branch KDE/4.10.</pre>
 <br />







<p>- Commit</p>


<br />
<p>On January 11th, 2013, 1:27 a.m., Yichao Yu 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 kde-workspace and Luís Gabriel Lima.</div>
<div>By Yichao Yu.</div>


<p style="color: grey;"><i>Updated Jan. 11, 2013, 1:27 a.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;">The size of window icon in the pager applet was always the same and is too small to if the applet is placed on the desktop. This patch increase the icon size if the window rectangle is big enough.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Compiled and see screen shots.
</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>plasma/desktop/applets/pager/package/contents/ui/main.qml <span style="color: grey">(d3203b9)</span></li>

 <li>plasma/desktop/applets/pager/pager.cpp <span style="color: grey">(70d1b55)</span></li>

</ul>

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



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>

<div>

 <a href="http://git.reviewboard.kde.org/r/108340/s/1024/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2013/01/11/before_400x100.png" style="border: 1px black solid;" alt="Before" /></a>

 <a href="http://git.reviewboard.kde.org/r/108340/s/1025/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2013/01/11/after_400x100.png" style="border: 1px black solid;" alt="After" /></a>

</div>


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








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