<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://reviewboard.kde.org/r/4541/">http://reviewboard.kde.org/r/4541/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Plasma.</div>
<div>By Octavian Voicu.</div>
<p style="color: grey;"><i>Updated 2010-07-08 21:06:50.086050</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description (updated)</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0;">Possible fix for a bug in KDE system tray component.
I'm not up-to-date with recent plasma changes, but I'm guessing that at some
point the `Message Indicator' widget changed its plugin name from `notifier' to
`message-indicator'.
The system tray applet, when invoked for the first time, tries to make sure
essential widgets are included, and adds `notifier' (the old "fake" message
indicator), when it should probably add `message-indicator'.
Excerpt from ~/.kde/share/config/plasma-desktop-appletsrc:
[Containments][4][Applets][10][Configuration][Applets][2]
plugin=message-indicator
...
[Containments][4][Applets][10][Configuration][Applets][3]
plugin=notifier
...
I attached a fix for this, which adds `message-indicator' instead of
`notifier'. An alternative fix would be to delete the offending lines entirely,
but I'm guessing we want the message indicator widget to show up by default.
For screenshot and more info see https://bugs.kde.org/show_bug.cgi?id=242637
</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>/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/ui/applet.cpp <span style="color: grey">(1144396)</span></li>
</ul>
<p><a href="http://reviewboard.kde.org/r/4541/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>