Created attachment 149440 [details] Examples with 26, 27 and 28 on panel height. SUMMARY Setting the panel height to an odd value makes the notification icons too small. More than if it's set the height to the value before that number. STEPS TO REPRODUCE 1. Set panel height to an odd value (27 for example). OBSERVED RESULT Notification icons are noticeably reduced in size, more so than setting the panel height to the before value (26 for example) EXPECTED RESULT Notification icons are not noticeably reduced in size by setting odd values. SOFTWARE/OS VERSIONS Operating System: KDE neon 5.24 KDE Plasma Version: 5.24.5 KDE Frameworks Version: 5.94.0 Qt Version: 5.15.4
Can reproduce
Me too. Maybe we're missing a Math.Round() somewhere.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/981
Git commit bf45a1d59fe7bec994e9bf9be61f724cbdfc267b by Nate Graham, on behalf of Anthony Hung. Committed on 13/06/2022 at 14:24. Pushed by ngraham into branch 'master'. containments/panel: `spacingAtMinSize` use Math.floor The `spacingAtMinSize` will be used to calculate the container margins. `Math.floor` is used to get a more consistent scaling when the panel height is changed M +1 -1 containments/panel/contents/ui/main.qml https://invent.kde.org/plasma/plasma-desktop/commit/bf45a1d59fe7bec994e9bf9be61f724cbdfc267b
Git commit ce9515f442ffd0787bc7d4559e43a59ec31b0087 by Nate Graham, on behalf of Anthony Hung. Committed on 13/06/2022 at 14:32. Pushed by ngraham into branch 'Plasma/5.25'. containments/panel: `spacingAtMinSize` use Math.floor The `spacingAtMinSize` will be used to calculate the container margins. `Math.floor` is used to get a more consistent scaling when the panel height is changed (cherry picked from commit bf45a1d59fe7bec994e9bf9be61f724cbdfc267b) M +1 -1 containments/panel/contents/ui/main.qml https://invent.kde.org/plasma/plasma-desktop/commit/ce9515f442ffd0787bc7d4559e43a59ec31b0087