<div>drosca added inline comments.</div><br /><div><strong>INLINE COMMENTS</strong><div><div>kcms/desktoptheme-qml/kcm.h:41 It is actually needed to cast to int in model::roleNames()</div><div>kcms/desktoptheme-qml/package/contents/ui/ThemePreview.qml:31 Because air's background svg has huge margins.<br />
This special case could be removed when we can get transparent margins size from FrameSvg.</div><div>kcms/desktoptheme-qml/package/contents/ui/main.qml:192 Good idea, I think it would make sense to have it in kdeclarative.</div><div>kcms/desktoptheme-qml/package/metadata.desktop:7 Actually, this is metadata for kpackage and not used from KCM. So I think this line can just be removed.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMADESKTOP Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1113" rel="noreferrer">https://phabricator.kde.org/D1113</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>drosca, sebas, Plasma<br /><strong>Cc: </strong>sebas, plasma-devel<br /></div>