<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/109611/">http://git.reviewboard.kde.org/r/109611/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 20th, 2013, 1:05 a.m. CEST, <b>Kai Uwe Broulik</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;">I'm not sure about adding a submenu for that. Imho it should highlight the categories with new applications and the applications themselves, so, after they might drop out of the recently installed menu, the user already knows where to find them.</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;">Well, I always thought that about the results of the search field. ;)
At least with a submenu it's clear for the user what this is about. Highlighting categories might be confusing...
Or do you mean to do both?
But, while I actually do like your proposal with highlighting, I don't really know at the moment how to implement that.
Whereas adding the submenu is quite easy to do.
Well, I may try to do it that way.
But it could be implemented first with the submenu and then changed later to highlights, maybe?</pre>
<br />
<p>- Wolfgang</p>
<br />
<p>On May 19th, 2013, 11:41 p.m. CEST, Wolfgang Bauer 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 kde-workspace.</div>
<div>By Wolfgang Bauer.</div>
<p style="color: grey;"><i>Updated May 19, 2013, 11:41 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 comes from a patch included in openSUSE)
This patch makes kickoff remember all the .desktop files it sees (in kickoffrc).
New ones are additionally shown in a seperate submenu named "Recently Installed" (for 3 days).
The seen entries are saved as "id=date" pairs in the "Seen Applications" group.
Entries older than 3 days get their date set to empty, which means "don't show this entry anymore".
Also if there have no entries been saved yet, the date of all found entries is set to empty as well. This prevents that all menu entries are shown in the "Recently Installed" submenu on first start.
The "Recently Installed" submenu can be toggled on and off in the plasmoid's settings.</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;">Created a new .desktop file in ~/.local/share/applications, ran kbuildsycoca4 and the menu "Recently Installed" appeared with this entry.
After logout/login this is still present.
Deleted the .desktop file again, ran kbuildsycoca4 and the menu "Recently Installed" disappeared again.
I have been using the (vanilla KDE) kickoff applet with this patch for about a week now.
Also this patch is already part of openSUSE for several years...</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=316916">316916</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>plasma/desktop/applets/kickoff/applet/applet.cpp <span style="color: grey">(a6f7379)</span></li>
<li>plasma/desktop/applets/kickoff/applet/kickoffConfig.ui <span style="color: grey">(8664ac8)</span></li>
<li>plasma/desktop/applets/kickoff/core/applicationmodel.h <span style="color: grey">(f0f8872)</span></li>
<li>plasma/desktop/applets/kickoff/core/applicationmodel.cpp <span style="color: grey">(57b6ba5)</span></li>
<li>plasma/desktop/applets/kickoff/simpleapplet/simpleapplet.cpp <span style="color: grey">(28fba18)</span></li>
<li>plasma/desktop/applets/kickoff/ui/launcher.h <span style="color: grey">(2a234c3)</span></li>
<li>plasma/desktop/applets/kickoff/ui/launcher.cpp <span style="color: grey">(4425bcc)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109611/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/04/02/kickoff-settings.png">Settings dialog (kickoff style) with patch</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/04/02/kickoff.png">Plasmoid (kickoff style) showing the "Recently Installed" submenu</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/04/02/classic-settings.png">Settings dialog (classic style) with patch</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/04/02/classic.png">Plasmoid (classic style) showing the "Recently Installed" submenu</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/19/kickoffrc">sample kickoffrc written by this patch</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>