Re: System tray icon sizes

2014-09-22 Thread Marco Martin
On Sunday 21 September 2014, Andrew Lake wrote: On Sun, Sep 21, 2014 at 2:06 AM, Marco Martin wrote: Btw, since we are refining systray icons, do they work also on a size of 22x22(if not I'll fix the code)? I would love to get rid of the old hack of 24x24, that is just a remnant of the

Re: System tray icon sizes

2014-09-22 Thread Marco Martin
On Sunday 21 September 2014, Kai Uwe Broulik wrote: Btw, since we are refining systray icons, do they work also on a size of 22x22(if not I'll fix the code)? I would love to get rid of the old hack of 24x24, that is just a remnant of the xembed legacy icons Is that the reason why even if

Re: System tray icon sizes

2014-09-21 Thread Andrew Lake
I'm running into a little bit of the inconsistency in systray icon sizes as we're working to try to improve the systray icons. http://wstaw.org/m/2014/09/21/5.1_systray_scaling_bug.png The icons are still a WIP so please ignore that for the moment. All the icons are 16x16 sitting on a padding

Re: System tray icon sizes

2014-09-21 Thread Andrew Lake
Correction: it's a 30x30 padding rect that gets the icon sizes the same, not a 32x32 padding rect. Hope this helps, Andrew On Sun, Sep 21, 2014 at 1:17 AM, Andrew Lake wrote: I'm running into a little bit of the inconsistency in systray icon sizes as we're working to try to improve the

Re: System tray icon sizes

2014-09-21 Thread Marco Martin
On Sep 21, 2014 10:39 AM, Andrew Lake jamboar...@gmail.com wrote: Correction: it's a 30x30 padding rect that gets the icon sizes the same, not a 32x32 padding rect. The battery (and possibly metworkmanager) were known for being a bit too big. I have now added api to make it possible to compute

Re: System tray icon sizes

2014-09-21 Thread Eike Hein
Tangentially related: The icons aren't horizontally centered in a vertical panel. Cheers, Eike ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: System tray icon sizes

2014-09-21 Thread Kai Uwe Broulik
Btw, since we are refining systray icons, do they work also on a size of 22x22(if not I'll fix the code)? I would love to get rid of the old hack of 24x24, that is just a remnant of the xembed legacy icons Is that the reason why even if I use units.roundToIconSize on the batterymonitor the

Re: System tray icon sizes

2014-09-21 Thread Andrew Lake
On Sun, Sep 21, 2014 at 2:06 AM, Marco Martin wrote: Btw, since we are refining systray icons, do they work also on a size of 22x22(if not I'll fix the code)? I would love to get rid of the old hack of 24x24, that is just a remnant of the xembed legacy icons I changed the padding rect on

Re: System tray icon sizes

2014-09-21 Thread David Edmundson
​Padding inside the SVG? ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: System tray icon sizes

2014-09-21 Thread Andrew Lake
On Sep 21, 2014 3:47 PM, David Edmundson wrote: ​Padding inside the SVG? Yes ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: System tray icon sizes

2014-09-21 Thread David Edmundson
​Padding inside the SVG? Yes I don't think this is the right long-term approach. We end up with problems like in the original post where it's difficult to make another item line up with an icon. IMHO padding should be applied at a QML level, not inside the component. It makes the behaviour a

Re: System tray icon sizes

2014-09-21 Thread Andrew Lake
On Sun, Sep 21, 2014 at 5:39 PM, David Edmundson wrote: ​Padding inside the SVG? Yes I don't think this is the right long-term approach. We end up with problems like in the original post where it's difficult to make another item line up with an icon. IMHO padding should be applied at a

Re: System tray icon sizes

2014-09-15 Thread Marco Martin
On Friday 12 September 2014, Kai Uwe Broulik wrote: Hi, One of the problems is that most of the tray icons use a Plasma IconItem which follows the icon sizes (16, 22, 32, 48, ...) whereas a custom item does not. This is also the reason why battery and network usually appear in a

System tray icon sizes

2014-09-12 Thread Martin Yrjölä
Hi! I'm developing the keyboard layout switcher plasmoid but can't match the other system tray icon sizes. The plasmoid is a rounded rectangle with text inside. If I use anchors.fill: parent I get something like the picture in [1]. If i use a multiplier of 0.7 I get it to line up nicely in bigger

Re: System tray icon sizes

2014-09-12 Thread Kai Uwe Broulik
this magic. Imho that needs fixing in systray. Cheers, Kai Uwe Am 12.09.2014 14:08 schrieb =?ISO-8859-1?Q?Martin_Yrj=F6l=E4?= martin.yrj...@gmail.com: Hi! I'm developing the keyboard layout switcher plasmoid but can't match the other system tray icon sizes. The plasmoid is a rounded rectangle

Re: System tray icon sizes

2014-09-12 Thread Martin Yrjölä
On Fri, Sep 12, 2014 at 3:23 PM, Kai Uwe Broulik k...@privat.broulik.de wrote: Hi, One of the problems is that most of the tray icons use a Plasma IconItem which follows the icon sizes (16, 22, 32, 48, ...) whereas a custom item does not. This is also the reason why battery and network