ervin added inline comments.

INLINE COMMENTS

> broulik wrote in fonts.cpp:479
> Kinda defies the purpose of using kconfigxt if we end up hardcoding the state 
> in code again?

That part was never transitioned to KConfigXT though since it doesn't have a 
KConfig backend.

Clearly inheriting by hand from KCoreConfigSkeleton should be next on the 
options to be evaluated (not in that patch though).

REPOSITORY
  R119 Plasma Desktop

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

To: bport, #plasma, ervin, crossi, meven
Cc: broulik, ngraham, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, 
jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, 
lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to