> On 2010-08-23 09:00:48, Marco Martin wrote: > > 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? > > Markus Slopianka wrote: > If this patch works with the other one that implements launcher support, > a Mac OS X-like Dock (AFAIK it's similar in Win7) can be implemented without > the need to get 3rd party widgets. > With a Dock-like setup I wouldn't want text other than tooltips. > > Beat Wolf wrote: > i would actually agree on adding it from the feedback i get when i show > kde to people used to windows. it's one of the first things they ask for. > > Aaron Seigo wrote: > the number of such features that have appeared over the years is immense, > and always people ask for those features ... as long as they are new in > Windows. there is no point in chasing taillights just to chase taillights. if > the idea is a good one, let's do it; if it isn't, let's not.
sure. i was just mentioning it because i for myself think its a good idea and other people want it, so from my point of view if the patch has a good quality, and it's actually using a feature that is there anyway, why not have it. But i see the other point of views too and all have their merit i think. - Beat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5078/#review7162 ----------------------------------------------------------- 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.ui > 1166313 > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/tasks.cpp 1166313 > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskitemlayout.h > 1166313 > > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskitemlayout.cpp > 1166313 > > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/abstracttaskitem.h > 1166313 > > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/abstracttaskitem.cpp > 1166313 > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskgroupitem.h > 1166313 > /trunk/KDE/kdebase/workspace/plasma/desktop/applets/tasks/taskgroupitem.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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
