<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/110413/">http://git.reviewboard.kde.org/r/110413/</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 13th, 2013, 11:18 a.m. UTC, <b>Eike Hein</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;">This isn't a call to reject this patch (which I think is a nice idea), but FWIW: The Task Manager widget is still intended to be replaced by a QML rewrite before feature freeze, and the QML bindings to Plasma's tooltip primitive currently don't allow for a lot in the way of layouting custom content, so I might not be able to recreate this too easily there, specifally having a line break between the two bits of info (unless the tooltip subtext prop accepts rich text). That said since I already had to fork the QML ToolTip component to get window preview thumbnails in there, I guess I could always fork my way further out of that dilemma.
In libplasma2 QML ToolTips are fortunately going to be much more flexible, ending the need for a fork.</pre>
</blockquote>
<p>On May 13th, 2013, 12:53 p.m. UTC, <b>Eike Hein</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;">Turns out that throwing a <br/> in there is no problem, so this is easy to sync up with after all.</pre>
</blockquote>
<p>On May 13th, 2013, 1 p.m. UTC, <b>José Millán Soto</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;">If that's the case, I suppose I should do a commit which includes the changes in TaskItem but not the changes in WindowTaskItem.
Am I right?</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;">Nah, I don't think you need to bother. I don't think the lib change is going to cause any merge conflicts when I rebase my branch onto master and the widget change doesn't affect me since the old C++ codebase just goes out the window. And it probably makes sense to commit the widget change even now, since it's useful testing by exercising the new lib API you add, or in case I get unexpectedly hit by a bus :).</pre>
<br />
<p>- Eike</p>
<br />
<p>On May 13th, 2013, 11:10 a.m. UTC, José Millán Soto wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 José Millán Soto.</div>
<p style="color: grey;"><i>Updated May 13, 2013, 11:10 a.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 adds information about the activities on which a task is available to the tooltip.
Two methods to obtain information about the activities a task is available, activities and activityNames, where added to TaskManager::TaskItem.
Depending whether the applet is configured to show tasks which are not available in the current activity or not, the information displayed will be different.
If the applet is configured to show only tasks in current activity, only the tasks which are also available in other activities will have this information in the tooltip.</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;">In order to test this patch, three activities (called "Activity 1", "Activity 2" and "Activity 3") were created.
Various instances of KDialog were created, all of them in desktop 1, and they were assigned to various activities (The title of the dialog was set to AxDi, where x are the activities on which the dialog will be available).
All screenshots were taken on activity 1.
Screenshots 1 to 3 were taken with the taskbar configured to display tasks in all activities but only in desktop 1. Screenshots 4 to 6 were taken with the configuration to show tasks only in current activity. Screenshot 7 was taken with the taskbar displaying all tasks from all desktops. Screenshot 8 was created with the taskbar configured to group elements by program (activity information is not yet available).</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=307163">307163</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>libs/taskmanager/taskitem.h <span style="color: grey">(35606e2)</span></li>
<li>libs/taskmanager/taskitem.cpp <span style="color: grey">(c9613c9)</span></li>
<li>plasma/desktop/applets/tasks/windowtaskitem.cpp <span style="color: grey">(f840076)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110413/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot1.png">Screenshot 1</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot2.png">Screenshot 2</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot3.png">Screenshot 3</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot4.png">Screenshot 4</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot5.png">Screenshot 5</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot6.png">Screenshot 6</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot7.png">Screenshot 7</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/05/13/snapshot8.png">Screenshot 8</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>