bruns added inline comments. INLINE COMMENTS
> ngraham wrote in main.qml:59 > Instead adding these new layouts, you could just set the following on each > checkbox that needs to be indented: > > Layout.leftMargin: !LayoutMirroring.enabled ? Kirigami.Units.largeSpacing : > 0 > Layout.rightMargin: LayoutMirroring.enabled ? Kirigami.Units.largeSpacing : > 0 > > IMO that's a bit cleaner. The effect would be different, as currently there already is some margin. Also, this has to be repeated for each item, and I plan to add (at least) one more option. REPOSITORY R119 Plasma Desktop REVISION DETAIL https://phabricator.kde.org/D28024 To: bruns, #baloo, #vdg, ngraham Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart