<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/103621/">http://git.reviewboard.kde.org/r/103621/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On February 6th, 2012, 9:38 p.m., <b>Alexander Neundorf</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;">Good from my POV (cmake stuff).</pre>
</blockquote>
<p>On February 6th, 2012, 11:13 p.m., <b>Christoph Feck</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;">UI-wise looks also fine. Was there anything else we needed to do? If not, merge to master. Thanks, you rock!</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;">Alex, we need this FindKdeclarative.cmake in kdelibs, and not in this repository since it is used by almost every single KDE Mobile Applications in theory (one common use case is the localization)... Are you fine with this "quick" solution, if I push it against KDE/4.8 ? I do not personally have time for learning this *Config.cmake, and we would not like to hard code "kdeclarative" into the target_link_libraries either. If someone felt like volunteering with a proper config file, I would be happy. :-)</pre>
<br />
<p>- Laszlo</p>
<br />
<p>On February 4th, 2012, 2:04 p.m., Lamarque Vieira Souza 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 Base Apps and KDE Runtime.</div>
<div>By Lamarque Vieira Souza.</div>
<p style="color: grey;"><i>Updated Feb. 4, 2012, 2:04 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;">Port shutdown dialog to QML. Two QML themes are included: default, which mimics the current shutdown dialog look & feel, and contour, which is used in Plasma Active.</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;">Works in Plasma Active Two using MeeGo image and KDE SC 4.8. It does not work in 4.7.x because the default theme requires kde-runtime 4.8's declarative imports.
TODO:
. test right to left language support.</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=216853">216853</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=216853">216853</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>ksmserver/CMakeLists.txt <span style="color: grey">(295b96e)</span></li>
<li>ksmserver/Copyright.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/FindKDeclarative.cmake <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/Messages.sh <span style="color: grey">(0aa8bab)</span></li>
<li>ksmserver/shutdown.cpp <span style="color: grey">(7fd1e11)</span></li>
<li>ksmserver/shutdowndlg.h <span style="color: grey">(e5f0942)</span></li>
<li>ksmserver/shutdowndlg.cpp <span style="color: grey">(a09a1a7)</span></li>
<li>ksmserver/themes/contour/ContourButton.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/contour/main.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/contour/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/contour/screenshot.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/ContextMenu.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/KSMButton.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/MenuItem.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/helper.js <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/main.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>ksmserver/themes/default/screenshot.png <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103621/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://git.reviewboard.kde.org/r/103621/s/400/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/01/03/qml-shutdowndlg_rebootmenu_400x100.jpeg" style="border: 1px black solid;" alt="" /></a>
<a href="http://git.reviewboard.kde.org/r/103621/s/407/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2012/01/05/qml-shutdowndlg_rebootmenu_label_accelerators_400x100.jpeg" style="border: 1px black solid;" alt="New version with label accelerator working" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>