<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/125626/">https://git.reviewboard.kde.org/r/125626/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 13th, 2015, 6:49 p.m. UTC, <b>Thomas Pfeiffer</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Great improvement overall!
Two things:
1. The semantic of the button for stoping an Activity (close) does not correspond with "stop". Either they should be renamed to "Closed", or the icon should be changed to the one for stopping playback.
2. Why is the button for creating a new Activity so far away from the list of Activities? I'd place it right below the list of running Activities so users won't miss it.</p></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;"><ol style="padding: 0;text-rendering: inherit;margin: 0 0 0 2em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">If I get the icon for this, I'll use it (I don't think that the current stop icon in breeze would be suitable for this - a simple rectangle...). The 'delete' is also quite problematic.</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">It is in the same place it used to be in. It will probably changed in the near future.</li>
</ol></pre>
<br />
<p>- Ivan</p>
<br />
<p>On October 13th, 2015, 5:32 p.m. UTC, Ivan Čukić 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, David Edmundson and Marco Martin.</div>
<div>By Ivan Čukić.</div>
<p style="color: grey;"><i>Updated Oct. 13, 2015, 5:32 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;">This is based on the VDG (Andrew's) mockup.</p>
<ul style="padding: 0;text-rendering: inherit;margin: 0 0 0 1em;line-height: inherit;white-space: normal;">
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Activities have more contextual information</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Configuration and activity creation interface is the same as in activities KCM</li>
<li style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">Fixed the issue of plasma locking when Qt decides will-I-wount-I (the relayout loop when it decides whether to show a scroll-bar or not)</li>
</ul>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">It requires the latest KActivities master.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><img alt="Shoot" src="http://ibin.co/2IvS4TSHzgKE" style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" /></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>desktoppackage/contents/activitymanager/ActivityDeletionDialog.qml <span style="color: grey">(523aba6)</span></li>
<li>desktoppackage/contents/activitymanager/ActivityCreationDialog.qml <span style="color: grey">(9317b0e)</span></li>
<li>desktoppackage/contents/activitymanager/ActivityItem.qml <span style="color: grey">(526716c)</span></li>
<li>desktoppackage/contents/activitymanager/ActivityList.qml <span style="color: grey">(e8bf5f0)</span></li>
<li>desktoppackage/contents/activitymanager/ActivityManager.qml <span style="color: grey">(ff5cdc1)</span></li>
<li>desktoppackage/contents/activitymanager/ControlButton.qml <span style="color: grey">(1e9854f)</span></li>
<li>desktoppackage/contents/activitymanager/Heading.qml <span style="color: grey">(be95b9e)</span></li>
<li>desktoppackage/contents/activitymanager/StoppedActivityItem.qml <span style="color: grey">(c1a417f)</span></li>
<li>desktoppackage/contents/activitymanager/static.js <span style="color: grey">(0eaab96)</span></li>
<li>imports/activitymanager/switcherbackend.h <span style="color: grey">(846862e)</span></li>
<li>imports/activitymanager/switcherbackend.cpp <span style="color: grey">(736fe67)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/125626/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>