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