<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/111815/">http://git.reviewboard.kde.org/r/111815/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On July 31st, 2013, 9:18 p.m. UTC, <b>Milian Wolff</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">No, I'm afraid I don't like this change.

a) it introduces flickering when the magic limit is reached and the scroll bar is hidden.
b) hiding the scrollbar when there is something to scroll is bad.

I agree that making it un-scrollable is also a bad choice, but I think its better than the above. Imo, the proper fix would be benchmark this and see whats going on and whether one could improve the situation that way instead. I'll have a look at that now.</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I've fixed this properly now by caching the QPixmaps for the icon names. This greatly improves the speed and we can simply enable the scrollbar always again.

Thanks for bringing this up! I'll close this review request now as its not required anymore.

Cheers</pre>
<br />










<p>- Milian</p>


<br />
<p>On July 31st, 2013, 2:07 p.m. UTC, Vlas Puhov wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 KDevelop.</div>
<div>By Vlas Puhov.</div>


<p style="color: grey;"><i>Updated July 31, 2013, 2:07 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;">When there is too many items in the list the scrollbar is disabled. It looks very odd(I've never seen a disabled scrollbar before). Furthermore it's not so obvious that it's disabled(I figured it out only from the third try).</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>plugins/quickopen/quickopenplugin.h <span style="color: grey">(6de4bd0)</span></li>

 <li>plugins/quickopen/quickopenplugin.cpp <span style="color: grey">(3fe08af)</span></li>

</ul>

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







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








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