pgkos (Piotr Kosinski)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Sep 16 2017, 11:33 AM (400 w, 5 d)
Availability
Available

Recent Activity

Feb 15 2018

pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Things look bad when a bitmap icon appears in the tray... (e.g. Remmina) It seems roundToIconSize is necessary for bitmap icons.

Feb 15 2018, 2:10 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

With this new version the tray icons scale exactly like the application icons on the left (the "jumps" between icon sizes happen at exactly the same time).

Feb 15 2018, 1:42 PM · Plasma
pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Cancel out the enlargement of AbstractItem by the tasksRow's marginHints.

Feb 15 2018, 1:24 PM · Plasma

Feb 14 2018

pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

It seems the problem is not with SVG icons, but with the systemtray code. If I make the systemtray's Flow's marginHints zero, the padding is preserved correctly! No magic constant is needed anymore:

//Do spacing with margins, to correctly compute the number of lines
property QtObject marginHints: QtObject {
    property int left: Math.round(0)
    property int top: Math.round(0)
    property int right: Math.round(0)
    property int bottom: Math.round(0)
}
Feb 14 2018, 6:29 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

There is something wrong and inconstent about SVG icon scaling. For example, if I put this line in systemtray ui main QML code instead:

Feb 14 2018, 5:45 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

I have returned to the previous version, as the version which used units.smallSpacing looked awful.

Feb 14 2018, 9:28 AM · Plasma
pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Fix the previous commit

Feb 14 2018, 9:25 AM · Plasma
pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Return to previous version, which was better.

Feb 14 2018, 9:11 AM · Plasma

Feb 13 2018

pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Use units.smallSpacing for padding instead of magic numbers.

Feb 13 2018, 10:25 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

I have tried and tested this diff without any padding, and trust me, it looks very bad - I would rather have this patch not accepted - than not to add any padding.

Feb 13 2018, 9:59 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

I could argue that the numbers come from Material Design guidelines - it recommends between 10% to 20% of padding around icons.

Feb 13 2018, 9:37 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

So maybe you tell me where this comes from? https://cgit.kde.org/plasma-workspace.git/commit/applets/digital-clock/package/contents/ui/DigitalClock.qml?id=90ea27f49c84f0dffbbf79b29eaa14c57f31a24d

Feb 13 2018, 9:37 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

I have tested it with the panel in both horizontal and vertical orientations, and 0.85 looks optimal for me.

Feb 13 2018, 9:15 PM · Plasma
pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Squashed the commits

Feb 13 2018, 9:08 PM · Plasma
pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Make the padding of all the panel icons the same

Feb 13 2018, 9:01 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

This patch also enables the user to configure the tray icon's size limit using System Settings -> Icons -> Advanced -> Panel

Feb 13 2018, 7:43 PM · Plasma
pgkos added a comment to D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

The 0.8 is used to create a small padding, so the icons never touch the panel's borders.

Feb 13 2018, 7:41 PM · Plasma
pgkos updated the summary of D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).
Feb 13 2018, 7:38 PM · Plasma
pgkos updated the diff for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).

Fix tray icon scaling

Feb 13 2018, 7:37 PM · Plasma
pgkos updated the test plan for D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).
Feb 13 2018, 7:26 PM · Plasma
pgkos requested review of D10487: Fix tray icon size scaling when changing the panel size (fix bug 360333).
Feb 13 2018, 6:26 PM · Plasma

Feb 10 2018

pgkos added a comment to D7849: Fix the tray icon scaling on HiDPI screens.

In my opinion, roundToIconSize should operate on scaled units - it is used multiple times in a few plasmoids - all calls from them to roundToIconSize assume it will operate on scaled units.

Feb 10 2018, 4:34 PM · Frameworks, Plasma
pgkos updated the diff for D7849: Fix the tray icon scaling on HiDPI screens.

This is a simpler implementation - the diff changes roundToIconSize so it uses scaled units.

Feb 10 2018, 4:25 PM · Frameworks, Plasma

Sep 17 2017

pgkos added a comment to D7849: Fix the tray icon scaling on HiDPI screens.

@anthonyfieroni my first version of the diff changed only the tray icon QML file, but I think that it is better to fix it in the plasma framework, as that function is used in multiple other places and there is the same problem with wrongly sized icons on hidpi screens.

Sep 17 2017, 3:19 PM · Frameworks, Plasma
pgkos added a comment to D7849: Fix the tray icon scaling on HiDPI screens.

@broulik consider this case:

Sep 17 2017, 3:04 PM · Frameworks, Plasma
pgkos updated the diff for D7849: Fix the tray icon scaling on HiDPI screens.

Fixed the previous diff.

Sep 17 2017, 2:32 PM · Frameworks, Plasma

Sep 16 2017

pgkos updated the diff for D7849: Fix the tray icon scaling on HiDPI screens.

Edited the function Units::roundToIconSize so it both accepts and outputs a dpi-scaled pixel value.

Sep 16 2017, 5:31 PM · Frameworks, Plasma
pgkos added a reviewer for D7849: Fix the tray icon scaling on HiDPI screens: Plasma.
Sep 16 2017, 2:58 PM · Frameworks, Plasma
pgkos created D7849: Fix the tray icon scaling on HiDPI screens.
Sep 16 2017, 2:43 PM · Frameworks, Plasma