https://bugs.kde.org/show_bug.cgi?id=419391
Nicolas Fella <nicolas.fe...@gmx.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|REPORTED |RESOLVED
Latest Commit| |https://invent.kde.org/fram
| |eworks/kirigami/commit/cd9b
| |ef528dc1b8614ac9db70455fafb
| |156ac7fe7
--- Comment #1 from Nicolas Fella <nicolas.fe...@gmx.de> ---
Git commit cd9bef528dc1b8614ac9db70455fafb156ac7fe7 by Nicolas Fella.
Committed on 08/08/2020 at 14:04.
Pushed by nicolasfella into branch 'master'.
[passivenotification] Set explicit padding
It currently relies on Control having an implicit padding in qqc2-desktop-style
(https://invent.kde.org/frameworks/qqc2-desktop-style/-/blob/master/org.kde.desktop/Control.qml#L23).
PC3 Control does not have this implicit padding (and I'm not sure what the
implications of adding it would be). This results in the notification not
having proper padding when using the Plasma QQC2 style.
The resulting padding is 6 on my machine, which is neither smallSpacing (4) nor
largeSpacing (8). I think it's better to use the existing units, but it results
in a visual change. Therefore CC @teams/vdg
M +5 -0 src/controls/templates/private/PassiveNotification.qml
https://invent.kde.org/frameworks/kirigami/commit/cd9bef528dc1b8614ac9db70455fafb156ac7fe7
--
You are receiving this mail because:
You are watching all bug changes.
