<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://svn.reviewboard.kde.org/r/4264/">http://svn.reviewboard.kde.org/r/4264/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 Kate and Plasma.</div>
<div>By Yuen Hoe Lim.</div>
<p style="color: grey;"><i>Updated 2010-09-06 06:31:35.309919</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Saw Aaron's mail on configChanged() and attempted to fix the patch accordingly. Not too sure if I'm doing it right though.</pre>
</td>
</tr>
</table>
<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;">Allows the user to specify which items to show or hide in the Kate Sessions Plasmoid using a configuration interface. Two possible use-cases for this:
1) Allow users who just want quick access to their projects to remove the default three items (Start Kate, New Session etc), which can take up a lot of space since the applet is usually quite small.
2) Allow multiple applets displaying different lists, for example having applets in different activities each displaying only sessions related to that activity.
The Kate Sessions Plasmoid currently does not have a configuration interface so I thought this is probably no harm :) I have attached a screenshot of the configuration UI. I'm somewhat inexperienced with QtDesigner so feedback on how I could improve the patch code or the config UI are most welcome :)</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 on trunk on my computer. Works as far as I can tell.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/KDE/kdesdk/kate/plasma/session/CMakeLists.txt <span style="color: grey">(1171815)</span></li>
<li>/trunk/KDE/kdesdk/kate/plasma/session/katesessionConfig.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.h <span style="color: grey">(1171815)</span></li>
<li>/trunk/KDE/kdesdk/kate/plasma/session/katesessionapplet.cpp <span style="color: grey">(1171815)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/4264/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://svn.reviewboard.kde.org/r/4264/s/428/"><img src="http://svn.reviewboard.kde.org" style="border: 1px black solid;" alt="Config UI Screenshot" /></a>
<a href="http://svn.reviewboard.kde.org/r/4264/s/497/"><img src="http://svn.reviewboard.kde.org" style="border: 1px black solid;" alt="Updated config UI screenshot" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>