D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Nathaniel Graham
ngraham added a comment. In D23350#516835 , @anthonyfieroni wrote: > In D23350#516765 , @ngraham wrote: > > > So this is a complaint that comes up a lot. I do see what you mean, and in my opinion,

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Anthony Fieroni
anthonyfieroni added a comment. In D23350#516765 , @ngraham wrote: > So this is a complaint that comes up a lot. I do see what you mean, and in my opinion, what we need is a dual-mode size control UI that lets you switch between "scale icons to

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Alexey Min
alexeymin added a comment. I can confirm the issue that tray icons are to small on 4k display REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D23350 To: xshram, ngraham Cc: alexeymin, ngraham, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh,

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Nathaniel Graham
ngraham added a comment. So this is a complaint that comes up a lot. I do see what you mean, and in my opinion, what we need is a dual-mode size control UI that lets you switch between "scale icons to fit in panel", "static icon size", and "n columns". However we don't have a UI that

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Vladislav Vekyu
xshram added a comment. In D23350#516753 , @ngraham wrote: > Looks to me that scaling is working just fine for system tray items in your screenshot. It's deliberate that they have that much padding on the top and bottom (at least with the

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Nathaniel Graham
ngraham added a comment. Looks to me that scaling is working just fine for system tray items in your screenshot. It's deliberate that they have that much padding on the top and bottom (at least with the default size settings). REPOSITORY R120 Plasma Workspace REVISION DETAIL

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Vladislav Vekyu
xshram added a comment. In D23350#516738 , @ngraham wrote: > The label says: > > > On high DPI systems those values would be scaled up, depending on the DPI > > So I gather that's not working? If not, we should fix that, not work around

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Nathaniel Graham
ngraham requested changes to this revision. ngraham added a comment. This revision now requires changes to proceed. The label says: > On high DPI systems those values would be scaled up, depending on the DPI So I gather that's not working? If not, we should fix that, not work around

D23350: Chaning default iconSize in systemtray

2019-08-22 Thread Vladislav Vekyu
xshram created this revision. xshram created this object with edit policy "Administrators". Herald added a project: Plasma. Herald added a subscriber: plasma-devel. xshram requested review of this revision. REVISION SUMMARY A patch for small icons on 4k displays and it doesn't break icons on