On Sonntag, 17. Dezember 2023 13:21:32 CET Carl Schwan wrote: > a) Remove smallSpacing and largeSpacing from Kirigami, and rename > mediumSpacing to just spacing. This unified spacing value would be defined > in qqc2-desktop-style to use whatever value is defined in the current > QStyle. > > a bis) Instead of creating only a generic "spacing" property, we create a > "Kirigami.Units.margins" or "Kirigami.Units.paddings" property to use for > paddings of QtQuick Controls and mapped to the Layout*Margin pixel metrics > and a "Kirigami.Units.spacing" property mapped to the Layout*Spacing pixel > metrics. For Breeze and Oxygen, both value would map to 6 pixels anyway, > but it might make it easier to switch to other values in the future as well > as make the usage of Units value more explit.
+1 for getting rid of choices because it makes my life easier if I don't have to think about which value is the correct one in some situation. I don't have an opinion on a bis), but I guess it makes sense to mirror the QWidget pixel metrics in QML. Regards, Ingo
signature.asc
Description: This is a digitally signed message part.