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

Reply via email to