----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/1114/#review1792 -----------------------------------------------------------
my only comment is that it would be nice if one could type "close kop" as "kop close" seems backwards (though i do recognize that that is a language issue, where in English we do verb->noun :) not using libtaskmanager in this case is just fine; it's not a lot of code, the code is clear and libtaskmanager does a ton of extra stuff that just isn't needed here. - Aaron On 2009-07-26 12:34:22, Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/1114/ > ----------------------------------------------------------- > > (Updated 2009-07-26 12:34:22) > > > Review request for Plasma. > > > Summary > ------- > > This runner lists the windows and virtual desktops. It allows to activate a > selected window or switch to a selected desktop. The runner works in the > following way: > * input is matched on window name, class or role; matching windows are listed > * input is matched on desktop name. Matching desktops are shown for > switching to, all windows on matching desktops are listed. > * keyword "desktop": lists all desktops (except current) if additional > number is inserted the list is reduced to that desktop > * keyword "window": lists all windows. Additional text will be used to > restrict like in first case. Following sub queries to restrict search are > possible: > * name=: restrict on name > * class=: restrict on window class > * role=: restrict on window role > * desktop=: restrict on desktop > those subqueries can be combined in any order. Each input not containing a > '=' will be interpreted as a name restriction if there is no explicit name > restriction. In that case it's ignored. Example query: "window desktop=2 > class=kmail role=composer" will list all open KMail composer windows on > desktop 2. > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/plasma/runners/CMakeLists.txt 1000707 > trunk/KDE/kdebase/workspace/plasma/runners/windows/CMakeLists.txt > PRE-CREATION > > trunk/KDE/kdebase/workspace/plasma/runners/windows/plasma-runner-windows.desktop > PRE-CREATION > trunk/KDE/kdebase/workspace/plasma/runners/windows/windowsrunner.h > PRE-CREATION > trunk/KDE/kdebase/workspace/plasma/runners/windows/windowsrunner.cpp > PRE-CREATION > > Diff: http://reviewboard.kde.org/r/1114/diff > > > Testing > ------- > > > Thanks, > > Martin > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
