Bug 421038 - Vertically misaligned battery icon in panel
Summary: Vertically misaligned battery icon in panel
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Battery Monitor (show other bugs)
Version: 5.19.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: 1.0
Assignee: Kai Uwe Broulik
URL:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2020-05-05 06:43 UTC by Leon De Andrade
Modified: 2020-06-18 07:59 UTC (History)
3 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/plasma-workspace/commit/0b92a51ee70cc159a1f6ba04905999e0117738b2
Version Fixed In: 5.19.1

Attachments
Shows a top panel with the misaligned battery icon (7.78 KB, image/png)
2020-05-05 06:43 UTC, Leon De Andrade 		Details
Perfect with plasma panel (8.44 KB, image/png)
2020-06-03 07:06 UTC, Leon De Andrade 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Leon De Andrade 2020-05-05 06:43:42 UTC 
Created attachment 128159 [details]
Shows a top panel with the misaligned battery icon

SUMMARY

The battery icon is not aligned properly.


STEPS TO REPRODUCE
1. Add a panel (top / bottom)
2. Add the battery widget
3. Possibly add more widgets (like netword, audio) to see the difference

OBSERVED RESULT

The battery icon is vertically misaligned (it's sticking to the top, no matter if you have a bottom or top panel).


EXPECTED RESULT

The battery icon should be aligned in the center.
Comment 1 Michail Vourlakos 2020-06-02 22:01:12 UTC 
Don't you have the same issue with Plasma panels?
Comment 2 Leon De Andrade 2020-06-03 07:05:06 UTC 
No I don't.
Comment 3 Leon De Andrade 2020-06-03 07:06:12 UTC 
Created attachment 129017 [details]
Perfect with plasma panel
Comment 4 Michail Vourlakos 2020-06-12 19:12:03 UTC 
the icon in compact representation needs to be centered aligned. More precise at file: contents/ui/CompactRepresentation.qml

the Row element at line 39, needs a:
anchors.centerIn: parent
Comment 5 Michail Vourlakos 2020-06-14 14:00:23 UTC 
Git commit 0b92a51ee70cc159a1f6ba04905999e0117738b2 by Michail Vourlakos.
Committed on 13/06/2020 at 15:51.
Pushed by mvourlakos into branch 'Plasma/5.19'.

center align battery icon in compact representation

--Battery monitor in compact representation must align
its battery icon/s in center. That was first noticed
by a user in some systray cases.

M  +1    -0    applets/batterymonitor/package/contents/ui/CompactRepresentation.qml

https://invent.kde.org/plasma/plasma-workspace/commit/0b92a51ee70cc159a1f6ba04905999e0117738b2
Comment 6 Leon De Andrade 2020-06-18 07:59:39 UTC 
Can confirm, it's fixed.