ervin requested changes to this revision. ervin added inline comments. This revision now requires changes to proceed.
INLINE COMMENTS > kcmstyle.cpp:317 > + > + m_model->setSelectedStyle(m_settings->widgetStyle()); > That line and the metaenum wrangling could likely be factored out somehow and shared with load(). Especially the enum part is uncommon enough that we probably want to hide it behind a nicer name (also would help a bit having a more consistent level of abstraction in the code of those functions). > kcmstyle.h:48 > Q_PROPERTY(StylesModel *model READ model CONSTANT) > - > - Q_PROPERTY(bool iconsOnButtons READ iconsOnButtons WRITE > setIconsOnButtons NOTIFY iconsOnButtonsChanged) > - Q_PROPERTY(bool iconsInMenus READ iconsInMenus WRITE setIconsInMenus > NOTIFY iconsInMenusChanged) > + Q_PROPERTY(StyleSettings* styleSettings READ styleSettings CONSTANT) > Q_PROPERTY(ToolBarStyle mainToolBarStyle READ mainToolBarStyle WRITE > setMainToolBarStyle NOTIFY mainToolBarStyleChanged) Please have the space before the * and not after to follow the style of that file (see the Q_PROPERTY just above). > kcmstyle.h:66 > > - bool iconsOnButtons() const; > - void setIconsOnButtons(bool enable); > - Q_SIGNAL void iconsOnButtonsChanged(); > - > - bool iconsInMenus() const; > - void setIconsInMenus(bool enable); > - Q_SIGNAL void iconsInMenusChanged(); > + StyleSettings* styleSettings() const; > ditto REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D24917 To: crossi, ervin, mart, bport, #plasma Cc: 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
