<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 11th, 2011, 3:25 a.m., <b>Kurt Hindenburg</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The reaseon I put 'Tab Monitoring' near 'Tab Title' are both per-tab, while the 'Tab Bar' is not. On second thought, th ordering by likelihood of usage seems better.</pre>
<br />
<p>- Jekyll</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>