Summary: | System Tray widget has no padding between icons at 24px panel thickness | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Keith <keithhacks-kdebug> |
Component: | System Tray | Assignee: | Nate Graham <nate> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | leondeandrade, nate |
Priority: | VHI | Keywords: | regression |
Version: | master | ||
Target Milestone: | 1.0 | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/commit/8e3de778c666adf3b9f6b94ed5ddc7369ebad32c | Version Fixed In: | 5.20.1 |
Attachments: |
Normal Looking 22px
No spacing 24px |
Description
Keith
2020-10-14 12:47:19 UTC
Can you attach a screenshot?
> The widget's appearance should not induce projectile vomiting.
Please be polite in bug reports.
Created attachment 132425 [details]
Normal Looking 22px
Created attachment 132426 [details]
No spacing 24px
Can reproduce, the spacing is odd with 24px but fine with 22px and 26px Can confirm. Will fix. I bet this is a case of > needing to be >= or something... (In reply to Nate Graham from comment #1) > > The widget's appearance should not induce projectile vomiting. > Please be polite in bug reports. I apologize, that was not intended to come off as rude. Thanks, no hard feelings. :) Git commit afd1b432c2e6ca7526e36080502237486725c728 by Nate Graham. Committed on 16/10/2020 at 18:41. Pushed by ngraham into branch 'master'. [applets/systemtray] Fix icon size for 24px panels We were using a > when we should have been using a >=, so there was one panel thickness that would make icons be too large for their panel and get squeezed together uncomfortably. FIXED-IN: 5.20.1 M +1 -1 applets/systemtray/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/afd1b432c2e6ca7526e36080502237486725c728 Git commit 8e3de778c666adf3b9f6b94ed5ddc7369ebad32c by Nate Graham. Committed on 16/10/2020 at 18:46. Pushed by ngraham into branch 'Plasma/5.20'. [applets/systemtray] Fix icon size for 24px panels We were using a > when we should have been using a >=, so there was one panel thickness that would make icons be too large for their panel and get squeezed together uncomfortably. FIXED-IN: 5.20.1 (cherry picked from commit afd1b432c2e6ca7526e36080502237486725c728) M +1 -1 applets/systemtray/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/8e3de778c666adf3b9f6b94ed5ddc7369ebad32c |