<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/102523/">http://git.reviewboard.kde.org/r/102523/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On September 6th, 2011, 9:13 a.m., <b>Sebastian Trueg</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 weird thing here is that this widget can also be opened from the KCM. Thus, the KCM could start itself. Not a great idea. Maybe we should stick to the action in the context menu of the system tray icon for now. Or you make the button optional.</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 was wondering why the hell this dialog is in the KCM’s code and not nepomukcontroller or somewhere but I couldn’t figure out how to open it.
Ah okay, so there is a tiny link “Details†that brings up that dialog.
Should be possible to show the button only if it is launched from the tray, not when launched from the KCM itself. I’ll look into it.</pre>
<br />
<p>- Kai Uwe</p>
<br />
<p>On September 3rd, 2011, 4:22 p.m., Kai Uwe Broulik 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 KDE Runtime and Nepomuk.</div>
<div>By Kai Uwe Broulik.</div>
<p style="color: grey;"><i>Updated Sept. 3, 2011, 4:22 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;">This little patch adds a "Configure Desktop Search" button to the nepomukcontroller statuswidget (the little status dialog that appears if you left-click on the nepomuk tray icon).
I know that you can access strigi configuration via the tray icon's context menu but I often found myself opening the status dialog and then wanting to get to the config dialog from there. Can't harm, can it? :)
I also added a pause/resume icon to the suspend/resume button.</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>nepomuk/kcm/statuswidget.h <span style="color: grey">(11f20b6)</span></li>
<li>nepomuk/kcm/statuswidget.cpp <span style="color: grey">(f3074dd)</span></li>
<li>nepomuk/kcm/statuswidget.ui <span style="color: grey">(ab210c8)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102523/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>