----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/3800/#review5365 -----------------------------------------------------------
Ship it! Looks ok to me. - Lubos On 2010-05-02 14:13:46, Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/3800/ > ----------------------------------------------------------- > > (Updated 2010-05-02 14:13:46) > > > Review request for Plasma, Aaron Seigo, Marco Martin, and Lubos Lunak. > > > Summary > ------- > > This patch adds _KDE_STATE_SKIP_SWITCHER to _NET_WM_STATE. Alt+Tab will > exclude windows with this state set (this automatically includes the alt+tab > replacement effects). I will extend Present Windows and Desktop Grid to > honour this flag, too. > > As soon as the patch is committed I will bring this to the NETWM mailinglist > to standardise the flag, so that it becomes _NET_WM_STATE_SKIP_SWITCHER. > > This can be used by plasma to exclude windows from alt+tab. Therefore > KWindowSystem::setState( winId, Net::SkipSwitcher ) can be used. > > > This addresses bugs 171192 and 179723. > https://bugs.kde.org/show_bug.cgi?id=171192 > https://bugs.kde.org/show_bug.cgi?id=179723 > > > Diffs > ----- > > trunk/KDE/kdebase/workspace/kwin/client.h 1121836 > trunk/KDE/kdebase/workspace/kwin/client.cpp 1121836 > trunk/KDE/kdebase/workspace/kwin/effects.h 1121836 > trunk/KDE/kdebase/workspace/kwin/effects.cpp 1121836 > trunk/KDE/kdebase/workspace/kwin/effects/desktopgrid/desktopgrid.cpp > 1121836 > trunk/KDE/kdebase/workspace/kwin/effects/presentwindows/presentwindows.cpp > 1121836 > trunk/KDE/kdebase/workspace/kwin/kcmkwin/kwinrules/ruleswidget.h 1121836 > trunk/KDE/kdebase/workspace/kwin/kcmkwin/kwinrules/ruleswidget.cpp 1121836 > trunk/KDE/kdebase/workspace/kwin/kcmkwin/kwinrules/ruleswidgetbase.ui > 1121836 > trunk/KDE/kdebase/workspace/kwin/lib/kwineffects.h 1121836 > trunk/KDE/kdebase/workspace/kwin/manage.cpp 1121836 > trunk/KDE/kdebase/workspace/kwin/rules.h 1121836 > trunk/KDE/kdebase/workspace/kwin/rules.cpp 1121836 > trunk/KDE/kdebase/workspace/kwin/sm.h 1121836 > trunk/KDE/kdebase/workspace/kwin/sm.cpp 1121836 > trunk/KDE/kdebase/workspace/kwin/tabbox.cpp 1121836 > > Diff: http://reviewboard.kde.org/r/3800/diff > > > Testing > ------- > > Adding a window specific rule updates the state and the window is not shown > in tabbox. > > > Thanks, > > Martin > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel