Otherwise it gets too relevant and takes some of the actual task prominence
Details
Diff Detail
- Repository
- R119 Plasma Desktop
- Branch
- arcpatch-D4090
- Lint
No Linters Available - Unit
No Unit Test Coverage
Can you make a screenshot? It's intentional, at least for regular task manager to be the same size. Icon tasks situation isn't ideal, though.
Should I try a different multiplier? 0.8 instead of 0.7...
I fear it would collide a bit with the color schemes... Besides we're already gray on gray.
I'm against reducing opacity, using a fade-out for something that's "on" gets us into design trouble when we want to use fade out to say "off". I'm OK with reducing size a bit, but I also don't have a problem with the version on master personally. If we do resize size, I think we should try to make it the same size as our new tiny systray icons for a semblance of cohesion on the panel instead of reducing another new size class.
I prefer reducing the size, and the screenshots here look ok to me (even those brousgt as an example of "barely visible")
if the icon never goes less than iconSizes.small, then semantically the size of the icon is always semantically correct
(if iconsizes.small is non visible on a particular device, then it's a problem of units scaling on dpi, not a taskbar problem)
applets/taskmanager/package/contents/ui/Task.qml | ||
---|---|---|
439 ↗ | (On Diff #10068) | to be onthe safe side: |
if iconsizes.small is non visible on a particular device, then it's a problem of units scaling on dpi, not a taskbar problem
It's also super tiny on my non high dpi desktop
After living it for a while I now agree that the indicators are too large, so I'd prefer this to go into 5.9.
I'm worried about the 0.7 magic number though. What does the systray do? The tray icons are much smaller than the audio indicators on my panel, and feel like they would be the right size. and be nice for consistency as mentioned.
I've asked twice to look into using the tray's sizing logic; since it was being ignored I've just done it:
https://commits.kde.org/plasma-desktop/cd361efff2fb4eaf456a9f1186d795a869f45d51
Anchoring to parent.verticalCenter had already been done seperately by Kai.