<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/105761/">http://git.reviewboard.kde.org/r/105761/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 7th, 2013, 10:59 a.m., <b>Aaron J. Seigo</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;">wouldn't it make more sense to simply use the Theme QML element from PlasmaCore components?</pre>
</blockquote>
<p>On January 7th, 2013, 11:03 a.m., <b>Marco Martin</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;">yes, either that or if org.kde.plasma.core is imported, a global "theme" element is already registered, so there isn't even risk of instanciating redundant objects</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;">perfect. so i'll discard this review request with the understanding that using org.kde.plasma.core should be the approach.</pre>
<br />
<p>- Aaron J.</p>
<br />
<p>On December 19th, 2012, 6:30 p.m., Nuno Bento 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 Plasma and Ivan Čukić.</div>
<div>By Nuno Bento.</div>
<p style="color: grey;"><i>Updated Dec. 19, 2012, 6:30 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 patch modifies KSplashQML allowing it to exposes the default Plasma theme to QML splash screens.
The idea was taken from the work done by David Edmundson and Aurélien Gâteau in LightDM-KDE.
Giving QML splash themes access to the default Plasma theme makes it possible to create themes that transition seamlessly to the desktop (for anyone using the default wallpaper). It also makes it possible to recreate the default KSplash theme in QML with 2 major benefits over the existing one: 1) the wallpaper image used is appropriate to the user's resolution and 2) the theme doesn't need to be modified between releases since it always uses the wallpaper from the default Plasma theme.
I've already implemented and tested a recreation of the Default theme in QML and will post the patch if this one gets accepted.</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;">1. Applied path to new kde:kde-workspace clone
2. Compiled and installed ksplashqml
3. Modified Minimalistic theme to use Image as background with source set to plasmaTheme.wallpaperPath(screenSize)
4. Created another QML based theme similar to the Default one
5. Tested login with 2 different users using both themes</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>ksplash/ksplashqml/CMakeLists.txt <span style="color: grey">(c2065d9)</span></li>
<li>ksplash/ksplashqml/SplashWindow.cpp <span style="color: grey">(4417643)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105761/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>