<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/104556/">http://git.reviewboard.kde.org/r/104556/</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;">Looks good to me! Thanks!</pre>
 <br />







<p>- Boudewijn</p>


<br />
<p>On April 12th, 2012, 2:21 p.m., Jean-Baptiste Mardelle 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 Calligra.</div>
<div>By Jean-Baptiste Mardelle.</div>


<p style="color: grey;"><i>Updated April 12, 2012, 2:21 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;">Currently in Krita's layer dialog, it is sometimes difficult to see which layer is the correct one. For example, check my screenshot: trying to select "Layer 104", it's hard to tell if it is the layer above or below the text label.

My patch adds the standard tree view hover effect in the layer dialog, which means that when moving mouse over a layer, it's much easier to find out which layer is where (see my second screenshot). My patch applies to all KoDocumentSectionDelegate in Calligra, which in my opinion improves the overall look and integration with other KDE applications.</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>libs/main/KoDocumentSectionDelegate.cpp <span style="color: grey">(3cb7b56)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/104556/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/104556/s/524/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/04/12/krita_layers_400x100.png" style="border: 1px black solid;" alt="Krita layers" /></a>

 <a href="http://git.reviewboard.kde.org/r/104556/s/525/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/04/12/krita_layers_hover_400x100.png" style="border: 1px black solid;" alt="Krita with hover effect" /></a>

</div>


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








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