Review Request 118928: Add support for showing window menu in NETRootInfo
Martin Gräßlin
mgraesslin at kde.org
Wed Jun 25 06:18:24 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118928/#review60927
-----------------------------------------------------------
see also for KWin side https://git.reviewboard.kde.org/r/118929/
- Martin Gräßlin
On June 25, 2014, 8:17 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 25, 2014, 8:17 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140625/cc101253/attachment.html>
More information about the Kde-frameworks-devel
mailing list