<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/104103/">http://git.reviewboard.kde.org/r/104103/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 28th, 2012, 2:01 p.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;">The windows style handles the tabs display. I agree with some styles, it is hard to distinguish which tab you're on.
I'm not sure making the text bold is the way to go. However, go ahead and put up more screenshots of what you like.</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;">I think about the only way to handle this would be to add options to the profile Tab page. Perhaps a color/style/etc area where the user and ignore or pick how they want the selected tab to appear.
If this is done, we might also use it for the 'monitor activity' tab display.
Doing something automatically will cause issues since each KDE style has different way of handling selected tab styles.</pre>
<br />
<p>- Kurt</p>
<br />
<p>On February 28th, 2012, 12:31 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 Feb. 28, 2012, 12:31 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 idea is to make the current tab more outstanding so that users can tell which is the current/active tab with just one glance.
The patch is just another one-line hack from me :)
If the idea is good, I guess I should spend more time on reading all those stylesheet documents to make the results more polished. Simply using bold font looks a little rude and primitive.
Maybe we can steal some good stuff from Kate.
</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/ViewContainer.cpp <span style="color: grey">(0a81dbb)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104103/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/104103/s/443/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/02/28/before_400x100.png" style="border: 1px black solid;" alt="before" /></a>
<a href="http://git.reviewboard.kde.org/r/104103/s/444/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/02/28/after_400x100.png" style="border: 1px black solid;" alt="after" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>