<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="https://git.reviewboard.kde.org/r/118061/">https://git.reviewboard.kde.org/r/118061/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 10th, 2014, 12:15 a.m. UTC, <b>Aleix Pol Gonzalez</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 be better to let the Desktop settle down a bit before we start to fork things out? Actually we should find ways to share code and not having to actually fork these, which is really counter-productive.</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;">That's the whole point of having shell packages. One thing that may be improved in the future is introducing a fallback mechanism between packages to not have to copy all, but i'm quite on the fence to that, and i wouldn't do it if not after it has to be provn really, really necessary.
And without attempts now I'll never really know for sure what is missing in the whole mechanism.
Furthermore, the gsoc on active is *now* and not in a few months time, the mediacenter gscoc is *now* and not in a few months time.</pre>
<br />
<p>- Marco</p>
<br />
<p>On May 9th, 2014, 2:05 p.m. UTC, Antonis Tsiapaliokas wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Plasma.</div>
<div>By Antonis Tsiapaliokas.</div>
<p style="color: grey;"><i>Updated May 9, 2014, 2:05 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-mobile
</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;">Add a copy of desktop shell package but remove the stuff that we
don't need. Like the contextmenu, panelconfiguration and the toolbox
</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>qmlpackages/shell/package/metadata.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/views/Panel.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/views/Desktop.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/layout.js <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/loader.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/explorer/WidgetExplorer.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/explorer/Tooltip.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/explorer/AppletDelegate.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/defaults <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/MouseEventInputButton.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/ContainmentConfiguration.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/ConfigurationShortcuts.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/ConfigurationContainmentAppearance.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/ConfigurationContainmentActions.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/ConfigCategoryDelegate.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/configuration/AppletConfiguration.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/applet/DefaultCompactRepresentation.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/applet/CompactApplet.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/applet/AppletError.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/WindowPreview.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/StoppedActivityItem.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/Heading.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ControlButton.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ActivityManager.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ActivityList.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ActivityItem.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ActivityDeletionDialog.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ActivityCreationDialog.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/shell/package/contents/activitymanager/ActivityBrowser.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>qmlpackages/CMakeLists.txt <span style="color: grey">(d277441)</span></li>
<li>CMakeLists.txt <span style="color: grey">(c7e3797)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118061/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>