meven added inline comments.

INLINE COMMENTS

> gtkconfig.cpp:215
> +    if (currentGtk2Theme.isEmpty()) {
> +        setGtk2Theme(QStringLiteral("Breeze"));
> +    }

What if users have a different default / theme set on first launch ?
I would suggest to read the current theme instead of hardcoding Breeze.

REPOSITORY
  R99 KDE Gtk Configuration Tool

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

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

Reply via email to