| Summary: | Panel height with an odd value makes the System Tray icons too small and blurry | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | ]\/[ ]\/[ ]_ 1357 <mmarreroleyva> |
| Component: | System Tray | Assignee: | Plasma Bugs List <plasma-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | materka, nate, niccolo.venerandi, qydwhotmail |
| Priority: | NOR | Keywords: | junior-jobs |
| Version: | 5.24.5 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Neon | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/commit/ce9515f442ffd0787bc7d4559e43a59ec31b0087 | Version Fixed In: | 5.25.1 |
| Sentry Crash Report: | |||
| Attachments: | Examples with 26, 27 and 28 on panel height. | ||
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 |
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