Review Request: Add "Show Icon only" option to the tasks applet

[Björn Ruberg]

An Montag, 23. August 2010 11:00:46 schrieben Sie:
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5078/#review7162
> -----------------------------------------------------------
> 
> 
> this very patch appeared here for several times already.
> and as usual, the question is: what real value gives over auto hiding the
> text when there is not enough space?

It makes grouping - what increases the amount of clicks you need to get to 
your application by one - unnecessary. You can usually see what applications 
are running because you have to look at some icons only instead of having to 
look at the whole panel width. The later often needs eye movements (depends on 
your screen). It's much more appealing to have just an icon instead of a task-
item with a much shortened window title in it.

It it is on place six of the most wanted plasma features. 

> On 2010-08-22 13:52:33, Björn Ruberg wrote:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://reviewboard.kde.org/r/5078/
> > -----------------------------------------------------------
> > 
> > (Updated 2010-08-22 13:52:33)
> > 
> > 
> > Review request for Plasma.
> > 
> > 
> > Summary
> > -------
> > 
> > This patch adds the option to put the taskbar in an icon-only mode -
> > similar as in Windows 7 . This is an much requested feature in bugzilla.
> > It is fairly simple and just using features already existing in the
> > code, adding an m_showIconOnly member to the layout and the abstractitem
> > plus the adaption of the config ui.
> > 
> > 
> > This addresses bug 159480.
> > 
> >     https://bugs.kde.org/show_bug.cgi?id=159480
> > 
> > Diffs
> > -----
> > 
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/tasksConfig.u
> >   i 1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/tasks.cpp
> >   1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskitemlayo
> >   ut.h 1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskitemlayo
> >   ut.cpp 1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/abstracttask
> >   item.h 1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/abstracttask
> >   item.cpp 1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskgroupite
> >   m.h 1166313
> >   /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskgroupite
> >   m.cpp 1166313
> > 
> > Diff: http://reviewboard.kde.org/r/5078/diff
> > 
> > 
> > Testing
> > -------
> > 
> > Moved panel around and made sure it works. Looks actually pretty good
> > this icon-only mode!
> > 
> > 
> > Thanks,
> > 
> > Björn