broulik added inline comments.
INLINE COMMENTS
> kwindowinfo.h:579
> + *
> + * Requires NET::WMPid passed as properties parameter to the constructor.
> + *
Copy paste error, requires `NET::WM2ApplicationMenuServiceName`
> kwindowinfo.h:588
> + *
> + * Requires NET::WMPid
zzag accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R278 KWindowSystem
BRANCH
appmenu-x11 (branched from master)
REVISION DETAIL
https://phabricator.kde.org/D28112
To: cblack, #plasma, broulik, zzag, #kwin
Cc: zzag, kde-frameworks-devel, LeGast00n,
This revision was automatically updated to reflect the committed changes.
Closed by commit R278:93814b513d45: Expose application menu via KWindowInfo
(authored by cblack).
REPOSITORY
R278 KWindowSystem
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28112?vs=77921=77922
REVISION
cblack edited the summary of this revision.
REPOSITORY
R278 KWindowSystem
REVISION DETAIL
https://phabricator.kde.org/D28112
To: cblack, #plasma, broulik, zzag, #kwin
Cc: zzag, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham,
bruns
cblack retitled this revision from "[WIP] Expose application menu via
KWindowInfo" to "Expose application menu via KWindowInfo".
REPOSITORY
R278 KWindowSystem
REVISION DETAIL
https://phabricator.kde.org/D28112
To: cblack, #plasma, broulik, zzag, #kwin
Cc: zzag, kde-frameworks-devel,
cblack marked 2 inline comments as done.
REPOSITORY
R278 KWindowSystem
REVISION DETAIL
https://phabricator.kde.org/D28112
To: cblack, #plasma, broulik, zzag, #kwin
Cc: zzag, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham,
bruns
