<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/110367/">http://git.reviewboard.kde.org/r/110367/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This caused an unhandled switch warning in kdoctools/genshortcutents.cpp, which can be trivially fixed with:
diff --git a/kdoctools/genshortcutents.cpp b/kdoctools/genshortcutents.cpp
index b35b5a4..92bdea5 100644
--- a/kdoctools/genshortcutents.cpp
+++ b/kdoctools/genshortcutents.cpp
@@ -300,6 +300,9 @@ static QString entityForAccel( KStandardShortcut::StandardShortcut accel )
case KStandardShortcut::AboutKDE:
markup += "AboutKDE";
break;
+ case KStandardShortcut::JoinTheGame:
+ markup += "JoinTheGame";
+ break;
case KStandardShortcut::StandardShortcutCount:
break;
}
however I'm not sure if there is anything else needed for the doc tools. This additional patch allowed kdelibs docs to build just fine so I don't think there are additional XML schemas to modify, however it might be prudent to check.
-----------
As a separate point though, I must admit to being nervous about the idea of integrating this into *every* KDE-using app's UI, including applications that have no affiliation with KDE whatsoever other than using kdelibs.
How would we as devs/GNU-users react if the default bash prompt contained a notice on how to fund the FSF?
Additionally even if we assume that users of KDE software wish to fund contributor meetings, necessary infrastructure, and the future security of KDE software, it doesn't necessarily follow that they wish to e.g. raise awareness for Free Culture.
In any event I would recommend avoiding "baking in" a donation amount or method in the U/I, it may change in the future.
------------
I don't want to splash cold water on the idea because I think this kind of promo activity can be very useful, but I don't think this is the right format. I would recommend a separate app or Plasmoid that can be run with a way to link it from the Plasma desktop (which *does* belong to KDE), preferably not quite as obtrusively.
We're currently riding a relative wave of positive karma with FOSS users for the first time in years and I don't want to see KDE exposed to the same reaction that we've seen with Ubuntu's Amazon gem or earlier with KDE 4.0. I've grown fond of not having flamewars about cashews or JRTs and I wish to see that continue if possible. :)</pre>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/110367/diff/3/?file=142857#file142857line76" style="color: black; font-weight: bold; text-decoration: underline;">kdeui/dialogs/kjointhegamedialog_p.cpp</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">76</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="s">"<li>Regular first hand reports about KDE's activities do so</li></ul> "</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">"activities do so" is wrong, I'm not sure what exactly is meant though.</pre>
</div>
<br />
<p>- Michael</p>
<br />
<p>On May 9th, 2013, 11:05 p.m. UTC, Albert Astals Cid 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 kdelibs.</div>
<div>By Albert Astals Cid.</div>
<p style="color: grey;"><i>Updated May 9, 2013, 11:05 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;">Patch by Pau ages ago, Lydia wants to push for it again.
Note that i still find it ugly, but i'm not the kdeui maintainer, so i'll leave it to someone else to discuss.
Not sure if the StandardAction enum addition should be at the end in case someone is storing those enums in a file or something.</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;">Ran okular, saw the new menu entry, it did stuff.
Ran ksnapshot (that doesn't use xmlgui), saw the new menu entry, it did the same stuff.</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>kdecore/doc/README.kiosk <span style="color: grey">(b95002d)</span></li>
<li>kdeui/CMakeLists.txt <span style="color: grey">(b439e04)</span></li>
<li>kdeui/actions/kstandardaction.h <span style="color: grey">(3064301)</span></li>
<li>kdeui/actions/kstandardaction.cpp <span style="color: grey">(7de0c6f)</span></li>
<li>kdeui/actions/kstandardaction_p.h <span style="color: grey">(b8f8df1)</span></li>
<li>kdeui/dialogs/jointhegame.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>kdeui/dialogs/kaboutkdedialog_p.cpp <span style="color: grey">(b9728bf)</span></li>
<li>kdeui/dialogs/kjointhegamedialog_p.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kdeui/dialogs/kjointhegamedialog_p.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>kdeui/shortcuts/kstandardshortcut.h <span style="color: grey">(b31e45c)</span></li>
<li>kdeui/shortcuts/kstandardshortcut.cpp <span style="color: grey">(669f750)</span></li>
<li>kdeui/widgets/khelpmenu.h <span style="color: grey">(3389068)</span></li>
<li>kdeui/widgets/khelpmenu.cpp <span style="color: grey">(f547c46)</span></li>
<li>kdeui/xmlgui/ui_standards.rc <span style="color: grey">(a0f5bed)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110367/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>