<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/101475/">http://git.reviewboard.kde.org/r/101475/</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 7fd2209d8c110c9fb08d383a12cbf957a85daf53 by Burkhard Lück.</pre>
<br />
<p>- Commit</p>
<br />
<p>On May 30th, 2011, 8:26 p.m., Burkhard Lück 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, Dawit Alemayehu and David Faure.</div>
<div>By Burkhard Lück.</div>
<p style="color: grey;"><i>Updated May 30, 2011, 8:26 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 the add search engine window is left aligned to the left side of the search bar. E. g. using locales with long translations of the window title "Set URI Shortcuts" (or other strings in the window) like eg fr part of the window to add the search engine is outside the screen, because the translated string is longer than the search bar.
The patch makes the window right aligned at the end of the search bar, than no matter how long the translated string is the whole window will be always inside konquerors main window.
Screenshot in locale fr before the patch showing the bug: http://bugsfiles.kde.org/attachment.cgi?id=52424
Screenshot in locale fr with the patch (one with / one without throbber): http://bugsfiles.kde.org/attachment.cgi?id=60484</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=253879">253879</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>konq-plugins/searchbar/searchbar.cpp <span style="color: grey">(c7aa301)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101475/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>