Review Request: TaskManager: TaskAction changes from IconTasks

Tue Nov 1 17:02:59 UTC 2011

> On Nov. 1, 2011, 2:05 p.m., Aaron J. Seigo wrote:
> > libs/taskmanager/taskactions.cpp, line 384
> > <http://git.reviewboard.kde.org/r/103004/diff/1/?file=39919#file39919line384>
> >
> >     we have a name, it should be used. it hardly matters what the eventual Launcher is called. shortening it with the removal of "It Is" is probably alright ("computer application english" :)
> 
> Craig Drummond wrote:
>     "Show A Launcher for Systemsettings When It is Not Running" is a *very* long menu item. What advantage does adding the name here give? The user already right-clicked on the taskbar entry - so they know what the item is. Placing its name here just adds extra noise. In fact I'd actually prefer just "Pin To Taskbar" and "Unpin From Taskbar"

a) it tells the user what is (likely) to happen with the launcher; as it doesn't actually relate directly to the window itself, this is useful information .. particularly as many times you can't even see the name of the application in the task widget button, so have nothing to really reference visually.

b) "pin" has got to be one of the worst bits of new jargon that has come with docks. in my testing with run-of-the-mill users not used to docks they tend to fail with complete reliability at explaining what it will do

it could be shortened to just "Show a launcher for <name>" though? 

> On Nov. 1, 2011, 2:05 p.m., Aaron J. Seigo wrote:
> > libs/taskmanager/taskactions.cpp, line 415
> > <http://git.reviewboard.kde.org/r/103004/diff/1/?file=39919#file39919line415>
> >
> >     how well does this work with KUniqueApplications?
> 
> Craig Drummond wrote:
>     The only issue I've noticed is when Amarok is minimized to the system tray, and you click on its launcher. Then the spinner is re-shown.

yet another reason we need to be able to identify such apps in some way ... we have this same issue in another UI element in Plasma Active (which is why i asked the question :) and we determined we were going to need something eventually to identify, at runtime, unique apps. this just confirms it for me. (though it has no bearing on this actual patch :)

> On Nov. 1, 2011, 2:05 p.m., Aaron J. Seigo wrote:
> > libs/taskmanager/taskactions.cpp, lines 508-510
> > <http://git.reviewboard.kde.org/r/103004/diff/1/?file=39919#file39919line508>
> >
> >     there's a line between "simplifiying" and "making it useless in too many cases" that this crosses imho. this would have me reaching for the window titlebar in too many cases.
> >     
> >     i would be in favour of renaming "Advanced" (a word we are trying to get away from) with something like "More actions"
> 
> Craig Drummond wrote:
>     hehe! knew this would get some reactions. Its only an option anyway. Perhaps it would be better if the user could supply a mask of items they did *not* want in the basic menu? Then it's up to the taskbar applet to decide what to show?

sounds like a great recipe for inconsistency, poor choices and the creation of new user-facing options. i'd really like to avoid it. coming to a good compromise situation that works well enough for everyone should be possible.

> On Nov. 1, 2011, 2:05 p.m., Aaron J. Seigo wrote:
> > libs/taskmanager/taskactions.cpp, lines 550-552
> > <http://git.reviewboard.kde.org/r/103004/diff/1/?file=39919#file39919line550>
> >
> >     could be moved into the "Advanced"(/"More Actions") menu?
> 
> Craig Drummond wrote:
>     But then it is different to the kwin menu, even in the non-basic mode.

i know; what we need to do at some point is move kwin to using this same implementation. i have plans for that, but i'll do so after your patches are all merged in successfully so there aren't too many things happening at once.

and i really do think "shade" is something that is no longer "common place" to do .. it isn't 1996 anymore ;)

> On Nov. 1, 2011, 2:05 p.m., Aaron J. Seigo wrote:
> > libs/taskmanager/taskactions.cpp, line 600
> > <http://git.reviewboard.kde.org/r/103004/diff/1/?file=39919#file39919line600>
> >
> >     this is a misuse of QAction. you can add QActions to a QAction and that parent QAction becomes the submenu. no need for such custom property setting/reading and menu creation.
> 
> Craig Drummond wrote:
>     This was for dock-manager plugin support - just to make things easier on my side - then there's a 1:1 mapping between the dockmanager item and the QAction. No biggie, I can always remove this.

hm... can't you add all the dockmanager items to a common QAction? should be no harder than calling setProperty on them? or are these properties already set by the dockmanager API? (in which case i'd suggest the dock manager API could use improving :)

- Aaron J.

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

On Oct. 31, 2011, 8:40 p.m., Craig Drummond wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103004/
> -----------------------------------------------------------
> 
> (Updated Oct. 31, 2011, 8:40 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Description
> -------
> 
> 1. Add NewInstance action to launch a new instance of an application.
> 2. Move toggle launcher action out of advanced menu.
> 3. Don't show application name in 'Show A Launcher' action, as might not know the real name at this time (e.g. no desktop file read).
> 4. Add application actions, to be shown at the top of the right-click menu. IconTasks uses this to show recent docuents, dock manager menu items, and unity items.
> 5. A ToolTipMenu class is created, so that tooltips may be give for menu items. This is so that IconTasks can display the full path of a documents in the recent documents menu.
> 6. Add option to have only basic window controls in menu.
> 
> 
> Diffs
> -----
> 
>   libs/taskmanager/taskactions.h 2b5a641 
>   libs/taskmanager/taskactions.cpp 0e6ba8e 
>   libs/taskmanager/taskactions_p.h 913966c 
> 
> Diff: http://git.reviewboard.kde.org/r/103004/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Craig Drummond
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20111101/ef5e7af6/attachment-0001.html>