<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/102823/">http://git.reviewboard.kde.org/r/102823/</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;">KDE3 had "Seconds to detect silence: 10"; that seems better. Was there a reason you put it in the center? I'm not sure if it should be alphabetic or by likelihood of usage (on bottom if that).
You ui diff has some extraneous changes as well.
I haven't tested the code yet.</pre>
<br />
<p>- Kurt</p>
<br />
<p>On October 10th, 2011, 11:35 p.m., Jekyll Wu 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 Konsole.</div>
<div>By Jekyll Wu.</div>
<p style="color: grey;"><i>Updated Oct. 10, 2011, 11:35 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 title has said enough.
The main problem is : Is the UI clear enough? I found it not easy to express clearly what this option does in limited words.
</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=195978">195978</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>src/EditProfileDialog.h <span style="color: grey">(3f62baf)</span></li>
<li>src/EditProfileDialog.cpp <span style="color: grey">(7a23374)</span></li>
<li>src/EditProfileDialog.ui <span style="color: grey">(b5471d5)</span></li>
<li>src/Profile.h <span style="color: grey">(e06cba4)</span></li>
<li>src/Profile.cpp <span style="color: grey">(d9706d1)</span></li>
<li>src/SessionManager.cpp <span style="color: grey">(7855770)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102823/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/102823/s/287/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2011/10/10/xterm1_400x100.png" style="border: 1px black solid;" alt="" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>