<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/102638/">http://git.reviewboard.kde.org/r/102638/</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;">I'm sorry to reject your work, but we don't wish to support tabs on the right for now, as we don't want to support configuration options that we feel would fall short of a certain aesthetic standard. The nickname list is currently not relocatable either, and with most people likely choosing to have it visible, having two list views bunched up on the right of the window would feel unbalanced. Further, the proposed patch doesn't extend the list delegate painting code to handle the right position properly (by moving the tab-like foundation arcs to the left-hand side), but even if it did, this would probably feel awkward in combination with left-aligned content.</pre>
<br />
<p>- Eike</p>
<br />
<p>On September 16th, 2011, 8:25 p.m. UTC, Javier Llorente wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Konversation.</div>
<div>By Javier Llorente.</div>
<p style="color: grey;"><i>Updated Sept. 16, 2011, 8:25 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;">Adds right tab support to Konversation</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/config/konversation.kcfg <span style="color: grey">(14ccc75b16066aedd5e0a5470e9a67e7ad6c0e5c)</span></li>
<li>src/config/tabs_configui.ui <span style="color: grey">(75b8308b8d7f417644ff2fa84139a5a175be3611)</span></li>
<li>src/viewer/viewcontainer.cpp <span style="color: grey">(e51c7536cd907ff31fa5a4c036072b6a206d3d9d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102638/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>