Bug 347591

Summary:	Icons are way too small when in vertical panel
Product:	[Plasma] plasmashell	Reporter:	davidebasilio <marsicanbear>
Component:	Icons-only Task Manager	Assignee:	Eike Hein <hein>
Status:	RESOLVED FIXED
Severity:	normal	CC:	andrew, mklapetek, notmart, plasma-bugs
Priority:	NOR
Version:	5.2.2
Target Milestone:	1.0
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Example of the issue

Description davidebasilio 2015-05-12 09:59:01 UTC

Task icons are too small when used in a vertical panel (will attach screenshot).

Reproducible: Always

Steps to Reproduce:
1. Put icons-only task manager in vertical panel
2.
3.

Actual Results:  
Icons are minuscule.

Expected Results:  
Icons are of the right size.

Comment 1 davidebasilio 2015-05-12 09:59:36 UTC

Created attachment 92554 [details]
Example of the issue

Comment 2 Eike Hein 2015-05-13 20:51:17 UTC

Git commit 861ef07eb12a285b3a89e4bb8583ed6def0cb47f by Eike Hein.
Committed on 13/05/2015 at 20:48.
Pushed by hein into branch 'master'.

Fix vertical scaling having gotten out of sync with horizontal scaling.

This needs Marco's pending patch to fix the panel view's margin scaling
for vertical margins to do much. Marco, please CCBUG 347591 when you
push it.
CCMAIL:notmart@gmail.com

Unfortunately even with both patches bug 347591 isn't fixed; for as yet
unknown reasons the margins of the frame element are a third bigger on
the vertical than the horizontal, even though it's not showing as such
in the SVG.

M  +15   -7    applets/taskmanager/package/contents/ui/Task.qml

http://commits.kde.org/plasma-desktop/861ef07eb12a285b3a89e4bb8583ed6def0cb47f

Comment 3 Eike Hein 2015-05-15 01:16:14 UTC

Marco unfortunately forgot to CCBUG: http://commits.kde.org/plasma-desktop/262ecde12d4881f005c9be8fc08aae4a49ba8c71

Comment 4 Andrew Chen 2015-11-30 02:13:12 UTC

This seems to be fixed in 5.4.3, as I could not reproduce the issue.

Comment 5 Martin Klapetek 2015-12-04 20:22:28 UTC

Following the last comment, closing as fixed.

If you can reproduce with Plasma 5.5, please reopen this report.

Thanks.