D27464: Add application menu dbus paths to org_kde_plasma_window interface

[David Edmundson]

davidedmundson added a comment.


  Good start. The direction makes sense.
  
  display.cpp and registry.cpp need updating to the new version number. Otherwise we won't be able to use this new method.
  
  It'd be great if you could update the unit test too.

INLINE COMMENTS

> plasmawindowmanagement_interface.cpp:599
> +        }
> +        org_kde_plasma_window_send_application_menu_changed(resource, qUtf8Printable(service), qUtf8Printable(object));
> +    }

We're mixing up our local 8bit and utf8 on different sides of this.

REPOSITORY
  R127 KWayland

REVISION DETAIL
  https://phabricator.kde.org/D27464

To: cblack, #kwin, zzag
Cc: davidedmundson, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200220/1665579e/attachment.html>