D25223: Avoid side effects during menu initialization

Nicolas Fella noreply at phabricator.kde.org
Tue Nov 12 14:24:13 GMT 2019


nicolasfella added subscribers: davidedmundson, nicolasfella.
nicolasfella added a comment.


  In D25223#561561 <https://phabricator.kde.org/D25223#561561>, @kmaterka wrote:
  
  > Off-topic idea: This QPA integration uses KStatusNotifierItem, which then translates it to DBus. Wouldn't it be better to talk to DBus directly? From the other side, this may be a duplication of work... Was the idea discussed?
  
  
  We did discuss it.
  
  IMHO the best solution would be to reuse the DBus tray implementation that is part of Qt (https://code.woboq.org/qt5/qtbase/src/platformsupport/themes/genericunix/dbustray/qdbustrayicon_p.h.html). However, it is part of the genericunixtheme and thus not "public". @davidedmundson had an idea on how we can "inherit" from the gericunix theme and reuse the implementation

REPOSITORY
  R135 Integration for Qt applications in Plasma

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

To: kmaterka, #plasma, #frameworks, broulik
Cc: nicolasfella, davidedmundson, cgiboudeaux, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20191112/558ca3e6/attachment-0001.html>


More information about the Plasma-devel mailing list