<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/129039/">https://git.reviewboard.kde.org/r/129039/</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Weight changes - I think we can improve on that, and it's good that you are. +1</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Moving KCMS around again - I'm massively against.
We've only just moved them round 24 months ago, and whilst it's fine to itterate and bug fix, this is just purely subjective changes with absolutely no rationale given.
Everyone has a different opinion where they should be, neither is more wrong, and neither is more right. At which point moving causes more problems than it solves.</p></pre>
<br />
<p>- David Edmundson</p>
<br />
<p>On September 26th, 2016, 9:38 p.m. UTC, Andreas Kainz wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Andreas Kainz.</div>
<p style="color: grey;"><i>Updated Sept. 26, 2016, 9:38 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-desktop
</div>
<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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">new startup and shutdown system setting group will look like this</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Startup and Shutdown
- Login Screen (SDDM)
- Splash Screen
- Autostart
- Background Services
- Lock Screen
- Logout and Session</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">there are some patches to rearrange the existing kcm's.
- https://git.reviewboard.kde.org/r/129037/
- https://git.reviewboard.kde.org/r/129038/
- https://git.reviewboard.kde.org/r/129039/</p></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>kcms/autostart/autostart.desktop <span style="color: grey">(fdd7233)</span></li>
<li>kcms/kded/kcmkded.desktop <span style="color: grey">(4d9aa28)</span></li>
<li>kcms/ksmserver/kcmsmserver.desktop <span style="color: grey">(c92a4b3)</span></li>
<li>kcms/ksplash/kcm_splashscreen.desktop <span style="color: grey">(b13691f)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/129039/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>