[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-usability
Subject: Re: [KDE Usability] Review Request 126621: [Task Manager] Add support for Unity Launcher API and App
From: "Kai Uwe Broulik" <kde () privat ! broulik ! de>
Date: 2016-01-07 21:21:20
Message-ID: 20160107212120.6499.78623 () mimi ! kde ! org
[Download RAW message or body]
--===============1577370705329133613==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126621/
-----------------------------------------------------------
(Updated Jan. 7, 2016, 9:21 nachm.)
Review request for Plasma, KDE Usability, Craig Drummond, Eike Hein, and Lukáš \
Tinkl.
Changes
-------
* Handle incoming 64 bit count and discard it - it's utter rubbish for this to be 64 \
bit
* Indicate job count as badge for application jobs
* always re-populate launcher info when URL changes rather than just in init, in case \
delegate is re-used
Bugs: 343632
https://bugs.kde.org/show_bug.cgi?id=343632
Repository: plasma-desktop
Description
-------
This adds support for the Unity Launcher API [1] with which applications can show a \
progress indicator, a number badge as well as demand the user's attention. It also \
shows application progress (such as copying a file) with the respective application.
This functionality has been present in the Icon Tasks applet [2] in Plasma 4 times. \
libunity must be present for most applications to actually use this.
I did not want to pollute libtaskmanager with this, so I made it a plugin in the Task \
Manager applet, which should also be fairly trivial to update to the new Launcher API \
that Unity 8 is going to use [3], or if we come up with our own solution at some \
point, maybe.
The primary focus is still the icons only task manager as the Unity API works on a \
per-application/per-launcher basis rather than per-window, however, where it makes \
sense the functonality is also offered to the traditional task manager. A downside of \
this is that (in the current implementation, anyways) in case you do not group your \
tasks and you have multiple windows of the same task open, it will show the same \
information on all entries.
For progress it gradually fills up the background of the task (VDG: I need a better \
graphic here, I currently abuse the "hover" tasks SVG, also I find the "+2" badge \
less than optimal) in all cases. The badges are only shown if 1) the cell is large \
enough 2) the label is *not* shown (ie. icon tasks manager or the regular one in very \
narrow). The badge is just a circle in theme highlight color, not an actual SVG - it \
cuts off part of the icon to provide more contrast. The "urgent" state just \
highlights the entry as if it requested attention, there's no wiggle animation and \
this is nowhere integrated with the window manager, also I did not find an \
application that used this, so I have no idea how it's used by them.
A video with the regular task manager on the top, a regular task manager and an icon \
tasks applet in the left panel and an icon tasks applet on the desktop (the latter is \
broken in the video, fixed in this patch) can be found here [4].
(There's still a ton of qDebug in the code at this point)
[1] https://wiki.ubuntu.com/Unity/LauncherAPI
[2] http://kde-apps.org/content/show.php/Icon+Tasks?content=144808
[3] http://bazaar.launchpad.net/~unity-team/unity8/trunk/files/head:/plugins/Unity/Launcher/
[4] https://www.youtube.com/watch?v=aBPGjlP6Wd8
Diffs (updated)
-----
applets/taskmanager/CMakeLists.txt 3c94cb7
applets/taskmanager/package/contents/config/main.xml f89fd51
applets/taskmanager/package/contents/ui/ConfigGeneral.qml bd97506
applets/taskmanager/package/contents/ui/Task.qml 2a88886
applets/taskmanager/package/contents/ui/TaskBadgeOverlay.qml PRE-CREATION
applets/taskmanager/package/contents/ui/TaskProgressOverlay.qml PRE-CREATION
applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.h PRE-CREATION
applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.cpp PRE-CREATION
applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.h PRE-CREATION
applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.cpp PRE-CREATION
applets/taskmanager/plugin/taskmanagerplugin.cpp 1be1fed
Diff: https://git.reviewboard.kde.org/r/126621/diff/
Testing
-------
I started a download in Chromium and got progress as well as a badge indicating how \
many download jobs were in progress. Thunderbird also registered a launcher on DBus \
but it never actually set a badge although I had unread mail. I copied a folder in \
Dolphin and got proper progress; also I played around with JobViewServer on DBus and \
it behaved correctly, even in case there were multiple progresses in one application.
File Attachments
----------------
Unity Launchers
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/03/c4c552a5-0449-47b1-a7f2-35ac20886cd0__unitylauncherstuff.png
Two unrelated windows
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/03/3817426d-42e7-4aca-9478-2774f6908589__twounrelatedlaunchers.png
Dolphin copying a file (notification applet next to it for demo)
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/04/4f200297-2c88-45fd-a986-8285203b6950__jobviewprogress.png
Config option
https://git.reviewboard.kde.org/media/uploaded/files/2016/01/04/24ab5241-1a10-45fe-8ede-923c678644a7__unitylauncherconfig.jpg
Thanks,
Kai Uwe Broulik
--===============1577370705329133613==
MIME-Version: 1.0
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 8bit
<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/126621/">https://git.reviewboard.kde.org/r/126621/</a>
</td>
</tr>
</table>
<br />
<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, KDE Usability, Craig Drummond, Eike Hein, and Lukáš \
Tinkl.</div> <div>By Kai Uwe Broulik.</div>
<p style="color: grey;"><i>Updated Jan. 7, 2016, 9:21 nachm.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">* Handle incoming 64 bit count and discard it - it's utter rubbish \
for this to be 64 bit
* Indicate job count as badge for application jobs
* always re-populate launcher info when URL changes rather than just in init, in case \
delegate is re-used</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="https://bugs.kde.org/show_bug.cgi?id=343632">343632</a>
</div>
<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 adds support for the Unity Launcher API [1] with \
which applications can show a progress indicator, a number badge as well as demand \
the user's attention. It also shows application progress (such as copying a file) \
with the respective application.</p> <p style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: inherit;">This functionality has \
been present in the Icon Tasks applet [2] in Plasma 4 times. libunity must be present \
for most applications to actually use this.</p> <p style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: inherit;">I did not want to \
pollute libtaskmanager with this, so I made it a plugin in the Task Manager applet, \
which should also be fairly trivial to update to the new Launcher API that Unity 8 is \
going to use [3], or if we come up with our own solution at some point, maybe.</p> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">The primary focus is still the icons only task manager as the Unity API \
works on a per-application/per-launcher basis rather than per-window, however, where \
it makes sense the functonality is also offered to the traditional task manager. A \
downside of this is that (in the current implementation, anyways) in case you do not \
group your tasks and you have multiple windows of the same task open, it will show \
the same information on all entries.</p> <p style="padding: 0;text-rendering: \
inherit;margin: 0;line-height: inherit;white-space: inherit;">For progress it \
gradually fills up the background of the task (VDG: I need a better graphic here, I \
currently abuse the "hover" tasks SVG, also I find the "+2" badge less than optimal) \
in all cases. The badges are only shown if 1) the cell is large enough 2) the label \
is <em style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: normal;">not</em> shown (ie. icon tasks manager or the regular \
one in very narrow). The badge is just a circle in theme highlight color, not an \
actual SVG - it cuts off part of the icon to provide more contrast. The "urgent" \
state just highlights the entry as if it requested attention, there's no wiggle \
animation and this is nowhere integrated with the window manager, also I did not find \
an application that used this, so I have no idea how it's used by them.</p> <p \
style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: \
inherit;">A video with the regular task manager on the top, a regular task manager \
and an icon tasks applet in the left panel and an icon tasks applet on the desktop \
(the latter is broken in the video, fixed in this patch) can be found here [4].</p> \
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">(There's still a ton of qDebug in the code at this \
point)</p> <p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">[1] https://wiki.ubuntu.com/Unity/LauncherAPI [2] \
http://kde-apps.org/content/show.php/Icon+Tasks?content=144808 [3] \
http://bazaar.launchpad.net/~unity-team/unity8/trunk/files/head:/plugins/Unity/Launcher/
[4] https://www.youtube.com/watch?v=aBPGjlP6Wd8</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: \
inherit;white-space: inherit;">I started a download in Chromium and got progress as \
well as a badge indicating how many download jobs were in progress. Thunderbird also \
registered a launcher on DBus but it never actually set a badge although I had unread \
mail. I copied a folder in Dolphin and got proper progress; also I played around with \
JobViewServer on DBus and it behaved correctly, even in case there were multiple \
progresses in one application.</p></pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> \
(updated)</h1> <ul style="margin-left: 3em; padding-left: 0;">
<li>applets/taskmanager/CMakeLists.txt <span style="color: \
grey">(3c94cb7)</span></li>
<li>applets/taskmanager/package/contents/config/main.xml <span style="color: \
grey">(f89fd51)</span></li>
<li>applets/taskmanager/package/contents/ui/ConfigGeneral.qml <span style="color: \
grey">(bd97506)</span></li>
<li>applets/taskmanager/package/contents/ui/Task.qml <span style="color: \
grey">(2a88886)</span></li>
<li>applets/taskmanager/package/contents/ui/TaskBadgeOverlay.qml <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>applets/taskmanager/package/contents/ui/TaskProgressOverlay.qml <span \
style="color: grey">(PRE-CREATION)</span></li>
<li>applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.h <span \
style="color: grey">(PRE-CREATION)</span></li>
<li>applets/taskmanager/plugin/smartlaunchers/smartlauncherbackend.cpp <span \
style="color: grey">(PRE-CREATION)</span></li>
<li>applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.h <span \
style="color: grey">(PRE-CREATION)</span></li>
<li>applets/taskmanager/plugin/smartlaunchers/smartlauncheritem.cpp <span \
style="color: grey">(PRE-CREATION)</span></li>
<li>applets/taskmanager/plugin/taskmanagerplugin.cpp <span style="color: \
grey">(1be1fed)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/126621/diff/" style="margin-left: \
3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments \
</h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2016/01/03/c4c552a5-0449-47b1-a7f2-35ac20886cd0__unitylauncherstuff.png">Unity \
Launchers</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2016/01/03/3817426d-42e7-4aca-9478-2774f6908589__twounrelatedlaunchers.png">Two \
unrelated windows</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2016/01/04/4f200297-2c88-45fd-a986-8285203b6950__jobviewprogress.png">Dolphin \
copying a file (notification applet next to it for demo)</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2016/01/04/24ab5241-1a10-45fe-8ede-923c678644a7__unitylauncherconfig.jpg">Config \
option</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>
--===============1577370705329133613==--
[Attachment #3 (text/plain)]
_______________________________________________
kde-usability mailing list
kde-usability@kde.org
https://mail.kde.org/mailman/listinfo/kde-usability
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic