'Re: Review Request: Completed jobs in systray show up as seperate'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Review Request: Completed jobs in systray show up as seperate
From:       "Rob Scheepmaker" <r.scheepmaker () student ! utwente ! nl>
Date:       2009-04-27 14:37:55
Message-ID: 20090427143755.20614.16179 () localhost
[Download RAW message or body]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/635/
-----------------------------------------------------------

(Updated 2009-04-27 07:37:55.617127)

Review request for Plasma.

Changes
-------

Added the screenshot

Summary
-------

To further improve the systray job usability, I've partly implemented something discussed on \
plasma-devel. This patch, besides some small other tweaks, makes the job widget disappear \
immediately when finished, and spawns a new widget without a progress bar and what not. These \
completed jobs widgets are put in a nice completed jobs extendergroup, as shown in the attached \
screenshot. We also discussed a 'completed action' that should be added to kjob and the JobView \
dbus interface. This is not yet done in this patch: at the moment this patch does kind of what \
the kwidgetjobtracker (those oldschool copy dialogs) also did: check if there's a second label \
set, see if that is the name of a file/directory, and show a way to open that file if it is \
(second label usually is a destination file, if there is one). Though the proper way to do \
things is to add the ability to set a completed action to kjob/jobview and use this in kio to \
set appropriate actions (when copying multiple file, opening the destination folder is a more \
sensible action then opening the file that was last copied for example). There are a couple of \
                other problems too:
* the infomessage should be changed by kio to "Finished copying" or something like that when \
it's finished. It's a bit weird to have finished items with the title "Copying" imo. Maybe even \
                something a bit more descriptive: "Dolphin finished copying 4 files".
* there are currently some giant margins around the finished item's text. Not sure where that \
                comes from yet.
* the label of the completed jobs group widgets starts out visible even though it's parent is \
not. This doesn't happen with the progress bar of the jobs in progress group, so I suspect some \
                QGraphicsProxyWidget weirdness, but I have to further investigate
* after using this some time, it started to irritate me that I'd always have to interact with \
the systray to clean up old completed jobs (clear all). I liked having a list of stuff that \
recently completed, but that list can, without interaction, grow a bit large. Maybe a maximum \
(configurable) age of items might make sense? So for example they get automatically destroyed \
                after an hour. Still not a perfect solution though. Anybody got any ideas on \
                this?
* I now made the popup autohide always after 6 seconds, even when for example a persistent \
notification is shown. Personally I find that quite convenient. Even when I go away, the \
spinner/number indicates clearly if there's still stuf waiting for me to look at, and while \
working I can freely ignore the jobs and notifications and be happy they always disappear after \
6 seconds. But I also discovered the autohiding behavior confuse some users. Other people might \
think 6 seconds is a bit too long. And then there's the very valid use case of having a \
presentation or what not, and not wanting to automatically show anything. I think allowing \
people to configure auto show/hide policies might be useful. So: "never show" "show x seconds" \
(plus slider/spinbox for x) "show as long as there's stuff to show"

Since the freeze is next week, I'd also like to announce that I could use some help to make \
high awesomeness jobs in the systray a reality for 4.3. I'd like to focus more on GSoC and \
there are also a few extender related bugs that could use my attention. So if somebody wants to \
help with the stuff that needs to be done before the freeze, I'd greatly appreciate it. You can \
always ping me and I'll give you some pointers if you need them.

Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/core/manager.h 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/core/manager.cpp 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/applet.h 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/applet.cpp 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/extendertask.h 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/extendertask.cpp 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/jobwidget.h 958972 
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/jobwidget.cpp 958972 

Diff: http://reviewboard.kde.org/r/635/diff

Testing
-------

Screenshots
-----------

  http://reviewboard.kde.org/r/635/s/108/

Thanks,

Rob

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[prev in list] [next in list] [prev in thread] [next in thread] 