[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    Re: Spacing in our apps
From:       Ingo =?ISO-8859-1?Q?Kl=F6cker?= <kloecker () kde ! org>
Date:       2023-12-17 14:32:35
Message-ID: 6002977.lOV4Wx5bFT () daneel
[Download RAW message or body]


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" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]