Am Samstag 27 Juni 2009 16:44:11 schrieb Marco Martin: > On 6/27/09, Aaron Seigo <ase...@kde.org> wrote: > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > http://reviewboard.kde.org/r/897/#review1400 > > ----------------------------------------------------------- > > > > > > i'm with Ivan on this one, i like the idea ... but i'm not comfortable > > with how it's being exposed to the user. there should probably be a way > > to select which effect is associated with the taskbar "show me these > > windows" effect. when this effect is enabled, it should also probably > > override the taskbar tooltips? finally, the context menu is pretty long > > already and it's a very slow way to access something that is really a > > shortcut. i do like the idea of a tooltip-replacing hover effect with one > > xatom to set with kwin deciding which effect gets employed. then plasma > > only needs to decide whether it sets that atom or shows the thumbnail > > tooltip. > > what about a button (label less?) on the tooltip since now can be clicked? > or hmm, a way to replace the left click menu of the task group? I like the idea of replacing the menu of the task group. It's kind of consistent: clicking on a task item will raise the window, clicking on a group item will raise all windows. Sounds like the right thing to do :-)
Button on the tooltip is kind of useless. Why clicking on a button when you are able to select the window directly? > > > trunk/KDE/kdebase/workspace/libs/taskmanager/taskactions.cpp > > <http://reviewboard.kde.org/r/897/#comment876> > > > > probably too much "geek speak": "what's 'present windows'"? :) > > > > perhaps just "present windows" or "show overview of windows" ... hm.. > > this will require some user testing i think. > > > > > > - Aaron > > > > On 2009-06-27 04:30:57, Martin Gräßlin wrote: > >> ----------------------------------------------------------- > >> This is an automatically generated e-mail. To reply, visit: > >> http://reviewboard.kde.org/r/897/ > >> ----------------------------------------------------------- > >> > >> (Updated 2009-06-27 04:30:57) > >> > >> > >> Review request for Plasma. > >> > >> > >> Summary > >> ------- > >> > >> Patch adds a new menu entry to task group context menu to show the group > >> of windows in kwin's present windows effect. When this entry is selected > >> the window ids of the tasks listed in the group are passed to an XAtom > >> which activates present windows effect for the passed windows (other > >> windows will be hidden). The atom will be removed by present windows > >> effect when it get's disabled. > >> > >> This patch requires current kwin from trunk. What's missing is the > >> detection if present windows effect is available or not (e.g. > >> compositing not active, Compiz used instead of KWin). KWin announces > >> support via setting atom on root window. > >> > >> > >> Diffs > >> ----- > >> > >> trunk/KDE/kdebase/workspace/libs/taskmanager/taskactions.cpp 972097 > >> trunk/KDE/kdebase/workspace/libs/taskmanager/taskactions_p.h 972097 > >> > >> Diff: http://reviewboard.kde.org/r/897/diff > >> > >> > >> Testing > >> ------- > >> > >> Tested with both manual and automatic grouping. > >> > >> > >> Thanks, > >> > >> Martin > > > > _______________________________________________ > > Plasma-devel mailing list > > Plasma-devel@kde.org > > https://mail.kde.org/mailman/listinfo/plasma-devel > > _______________________________________________ > Plasma-devel mailing list > Plasma-devel@kde.org > https://mail.kde.org/mailman/listinfo/plasma-devel
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
