Spacing in our apps

[Ingo Klöcker]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-devel/attachments/20231217/b0666b65/attachment.sig>