<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 />





 <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 df9a20b4747e5126adb1c526d827a621938028c0 by Jean-Baptiste Mardelle to branch master.</pre>
 <br />







<p>- Commit</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>