leinir updated this revision to Diff 68958. leinir added a comment.
Together with the change in D25041 <https://phabricator.kde.org/D25041> this makes the model reloading work (without that patch, it simply fails quietly as the signal forwarding fails). If wanted, we could also use the count property in the same diff to reintroduce the previous logic (which of course would then require that patch having been merged first) - Simplify the model reload logic a touch REPOSITORY R119 Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24942?vs=68742&id=68958 BRANCH use-newstuffquick-in-lookandfeel-kcm (branched from master) REVISION DETAIL https://phabricator.kde.org/D24942 AFFECTED FILES CMakeLists.txt kcms/lookandfeel/CMakeLists.txt kcms/lookandfeel/kcm.cpp kcms/lookandfeel/kcm.h kcms/lookandfeel/package/contents/ui/main.qml To: leinir, #plasma, #knewstuff, #vdg, broulik Cc: ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart