<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/101931/">http://git.reviewboard.kde.org/r/101931/</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;">Have you run kdialog_unittest?</pre>
<br />
<p>- Albert</p>
<br />
<p>On July 12th, 2011, 4:01 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 kdelibs and Darío Andrés Rodríguez.</div>
<div>By Burkhard Lück.</div>
<p style="color: grey;"><i>Updated July 12, 2011, 4:01 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;">Quoting the BR:
"When using KMessageBox::detailedError() it sometimes doesn't resize properly to
show all the text"
See kgpg error dialog without kdelibs patch. The text "your keyringxx" is cut off.
Pressing Details button (kgpg error dialog without kdelibs patch after clicking details)
resizes the dialog properly and all text is visible.
Last screenshot (kgpg error dialog with kdelibs patch) shows the dialog after the start
and before clicking the Deatils button, everything is properly readable.
I commit this review request on behalf of Dario Andres (https://bugs.kde.org/show_bug.cgi?id=193097#c9)
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Tested with kgpg error dialog, see third screenshot.</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=193097">193097</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>kdeui/dialogs/kdialog.cpp <span style="color: grey">(32f5df0)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101931/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/101931/s/200/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/12/kgpg-error-dialog-without-kdelibs-patch_1_400x100.png" style="border: 1px black solid;" alt="kgpg error dialog without kdelibs patch" /></a>
<a href="http://git.reviewboard.kde.org/r/101931/s/201/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/12/kgpg-error-dialog-without-kdelibs-patch-details_400x100.png" style="border: 1px black solid;" alt="kgpg error dialog without kdelibs patch after clicking details" /></a>
<a href="http://git.reviewboard.kde.org/r/101931/s/203/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/07/12/kgpg-error-dialog-with-kdelibs-patch_400x100.png" style="border: 1px black solid;" alt="kgpg error dialog with kdelibs patch" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>