<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/104071/">http://git.reviewboard.kde.org/r/104071/</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 48e804dbe4dd4bebc362e722c69df6e797605437 by Dawit Alemayehu to branch KDE/4.8.</pre>
<br />
<p>- Commit</p>
<br />
<p>On February 29th, 2012, 5:04 p.m., Dawit Alemayehu 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 Base Apps and David Faure.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated Feb. 29, 2012, 5:04 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;">The attached patch fixes the issue of the status bar becoming huge when, for whatever reason, the requested icon could not be found and a default icon is returned instead. In this case the code that was requesting the Konqueror status bar indicator icons was using the convenience function UserIcon from KIconLoader.
The problem with using UserIcon is that the size of the icon returned is completely dependent upon the actual size of the requested icon on disk. If the icon is large, then a large icon is returned. As such when the icon cache is utterly corrupted for whatever reason and returns some default icon with its default (actual) size, Konqueror's status bar balloons to one quarter of the main window size. Not what we want. To avoid this, the patch converts the code to use KIconLoader::loadIcon with the icon size set to KIconLoader::SizeSmall instead of the related convenience function.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=294321">294321</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>konqueror/src/konqframestatusbar.cpp <span style="color: grey">(0142bd0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104071/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>