> On June 25, 2014, 9:37 a.m., Thomas Lübking wrote: > > Do we really need this in NETRootInfo? > > > > - it's (yet) a nonstandard item which went through no review, so the final > > protocol (if any ever) may differ from our API > > - (from what i read from eg. xfce and awesom, lxde switches to Qt and > > whatnot): it will likely remain a gtk+gnome only thing anyway > > - gtk "is holding it wrongly" (ie. CSD should seek for CSD support > > announcement from the WM) > > - apparently the WM does not announce support for this (and that's probably > > neither intended, since it would become equal to announcing support for CSD > > ...) > > > > -> we could mangle support for some gtk proprietary properties (shadow > > padding...) and this message into kwin (with a grumpy comment >-), and > > record every "bug" report reg. gtk+ CSD support as "whishlist" for kwin as > > well > > > > I do not want KDE users to suffer from this ... situation, but i do really > > not see a reason to dump this (and esp. not in its current state) into > > public NETRootInfo API. > > > > Gtk+ CSD has the shape of a client side hack and imo it therefore should > > also be treated as one. > > Sorry. > > Martin Gräßlin wrote: > on Friday is our last chance to change the ABI, that was kind of my > thought why to bring it in - last chance without having to introduce a > NETRootInfo2. I'll wait whether something happens concerning NETWM > specification (I asked for that) and for explanation what the device id is (I > also asked for that). If nothing happens I'll discard it and implement > support just in KWin. > > Martin Gräßlin wrote: > discarding this review - it's not yet in a state that we can provide ABI > guarantee for it. I'll do it in KWin directly.
I forgot we still need the NET::WM2ShowWindowMenu otherwise we cannot announce support in KWin: https://git.reviewboard.kde.org/r/118946/ - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118928/#review60929 ----------------------------------------------------------- On June 26, 2014, 9:02 a.m., Martin Gräßlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/118928/ > ----------------------------------------------------------- > > (Updated June 26, 2014, 9:02 a.m.) > > > Review request for KDE Frameworks, kwin and David Edmundson. > > > Repository: kwindowsystem > > > Description > ------- > > Add support for showing window menu in NETRootInfo > > This implements the non-standard _GTK_SHOW_WINDOW_MENU which will be > used by GTK's client-side-decoration (CSD) to show the window menu. I'm > neither happy with implementing a GTK specific hint nor am I happy > with adding any support for the completely broken GTK CSD [1] in our > source base. Outside GNOME Shell GTK should not use their CSD or at > least check whether the window manager supports CSD. Unfortunately > GTK is not doing that and leaves the windows in a rather broken state > on our side. For example the shadows are part of the windows breaking > any window snapping, moving and window activation. By implementing this > extension to show the window menu we can make the life for our users > a little bit better. It allows to show our KWin user actions menu > instead a broken GTK menu [2]. Especially it allows our users to add > sane window decorations back to the window. > > Adding support for a GTK specific hint is a one time exception. We > need to add it now otherwise we cannot add the required virtual method > to NETRootInfo. I do hope that GTK tries to bring this to NETWM before > we release 5.0. Even after 5.0 I'm going to replace it by the NETWM > hint as soon as it gets standardized and thus breaking support for the > GTK hint. > > [1] https://bugzilla.gnome.org/show_bug.cgi?id=729721 > [2] https://bugzilla.gnome.org/show_bug.cgi?id=729788 > > > Diffs > ----- > > autotests/netrootinfotestwm.cpp f8c28be51e5a5b19d436c54eede0e8659a65c84e > src/netwm.h 2d812a755565a09b2850958a014060a864a652a0 > src/netwm.cpp 1daad1e5fc87fa85da6348a059d0ae0acec26eaf > src/netwm_def.h 0edadc085e08531ec81bcde5651e8475e8573091 > > Diff: https://git.reviewboard.kde.org/r/118928/diff/ > > > Testing > ------- > > > Thanks, > > Martin Gräßlin > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel