broulik added inline comments.

INLINE COMMENTS

> gikari wrote in fonts.cpp:607
> @broulik I do not understand. `KDEDModule` (which is a parent of `GtkConfig`) 
> is `QObject`, not a `QGuiApplication`. Maybe I am missing something? You want 
> me to create a connection with that signal within a module, so I don't have 
> to ask from KCM to change a font via DBus?

kded (the application you're running in) is a `QApplication`, so from the 
constructor of your kdedmodule you could do

  #include <QApplication>
  connect(qApp, &QGuiApplication::fontChanged, this, [this](this QFont &font) {
      // now check if you have the new fonts
  });

I can't guarantee this will work (accessing `font` from there will actually 
deadlock :D but I am curious whether that signal is emitted properly, so you 
dont have to listen to or ask the font kcm to tell you.

REPOSITORY
  R119 Plasma Desktop

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

To: gikari, #plasma, apol, davidedmundson
Cc: davidedmundson, nicolasfella, broulik, GB_2, abetts, ngraham, plasma-devel, 
LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, ragreen, Pitel, ZrenBot, 
himcesjf, lesliezhai, ali-mohamed, jensreuterberg, sebas, apol, mart

Reply via email to