Bug 421038

Summary: Vertically misaligned battery icon in panel
Product: [Plasma] plasmashell Reporter: Leon De Andrade <leondeandrade>
Component: Battery MonitorAssignee: Kai Uwe Broulik <kde>
Status: RESOLVED FIXED    
Severity: normal CC: mvourlakos, nate, plasma-bugs
Priority: NOR Keywords: regression
Version: 5.19.0   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 5.19.1
Sentry Crash Report:
Attachments: Shows a top panel with the misaligned battery icon
Perfect with plasma panel

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.