Bug 425305

Summary: mis-positioned weather icon in system tray
Product: [Plasma] kdeplasma-addons Reporter: Patrick Silva <bugseforuns>
Component: WeatherAssignee: Plasma Bugs List <plasma-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: biltamirzo, cappelikan, IevgenSobko, jeff, jf.mundox, kafoha1888, lemmyg, nate, nortexoid, sonichedgehog_hyperblast00, twinshadows404, vbsampath
Priority: HI    
Version: 5.25.90   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: Version Fixed In: 5.27
Sentry Crash Report:
Attachments: screenshot
screenshot
video recorded on neon unstable

Description Patrick Silva 2020-08-13 14:23:15 UTC
Created attachment 130851 [details]
screenshot

SUMMARY
Following Nate's request
https://bugs.kde.org/show_bug.cgi?id=424711#c10

STEPS TO REPRODUCE
1. set panel height to 46px
2. open system tray settings
3. set "Panel icon size" to "Small and enable Weather Report applet in "Entries" section

OBSERVED RESULT
mis-positioned weather icon, see the attached screenshot please

EXPECTED RESULT
correctly positioned weather icon

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.74.0
Qt Version: 5.14.2
Comment 1 Patrick Silva 2020-10-14 20:34:40 UTC
*** Bug 427679 has been marked as a duplicate of this bug. ***
Comment 2 Patrick Silva 2020-10-19 20:18:45 UTC
*** Bug 427991 has been marked as a duplicate of this bug. ***
Comment 3 twinshadows404 2020-11-09 19:49:37 UTC
*** Bug 428411 has been marked as a duplicate of this bug. ***
Comment 4 twinshadows404 2020-11-09 19:51:37 UTC
Created attachment 133182 [details]
screenshot

I managed to "fix" it by increasing the font size. Umm.
Comment 5 Nate Graham 2021-03-31 01:56:33 UTC
*** Bug 434951 has been marked as a duplicate of this bug. ***
Comment 6 Nate Graham 2021-06-03 12:41:17 UTC
*** Bug 433556 has been marked as a duplicate of this bug. ***
Comment 7 Patrick Silva 2021-10-24 12:03:41 UTC
*** Bug 444304 has been marked as a duplicate of this bug. ***
Comment 8 galder 2022-02-12 13:40:00 UTC
this is still reproducible in 5.24.0
Comment 9 Jeff Cuevas-Koch 2022-09-16 18:45:47 UTC
I am a software developer and would like to contribute to KDE.  This bug seems like a small, easy place to get started -- but I'm not sure where to go from here! Is there a "Contributing for Noobs" manual somewhere that I haven't been able to find?
Comment 10 Nate Graham 2022-09-19 18:54:58 UTC
That would be lovely! There is https://community.kde.org/Get_Involved/development.

I'd be happy to help you get started.
Comment 11 Paul Worrall 2022-11-08 11:58:58 UTC
*** Bug 461576 has been marked as a duplicate of this bug. ***
Comment 12 Nate Graham 2022-11-26 05:53:24 UTC
Patrick, can you still reproduce this with current git master on Neon Unstable?
Comment 13 Patrick Silva 2022-11-26 10:35:02 UTC
Yes, I can.
Comment 14 Nate Graham 2022-11-29 18:27:56 UTC
I'm still unable to reproduce with the instructions provided in the original description. :/

Any chance you could record yourself reproducing the bug?
Comment 15 Patrick Silva 2022-11-30 11:13:55 UTC
Created attachment 154165 [details]
video recorded on neon unstable
Comment 16 Nate Graham 2022-11-30 17:41:53 UTC
Thanks. It looks like the layout isn't respecting the margins of the container it lives in.
Comment 17 Bug Janitor Service 2022-12-02 17:53:24 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/287
Comment 18 Ismael Asensio 2022-12-02 21:39:52 UTC
Git commit e7df214403f6a019b9b29d239426a465eb964e24 by Ismael Asensio.
Committed on 02/12/2022 at 18:00.
Pushed by iasensio into branch 'master'.

applets/weather: Fix icon position in system tray

Remove the external ColumnLayout and instantiate the Loader
directly. This fixes the wrong positioning of the icon when
in the system tray (either directly or in the pop-up)
FIXED-IN: 5.27

M  +12   -16   applets/weather/package/contents/ui/CompactRepresentation.qml

https://invent.kde.org/plasma/kdeplasma-addons/commit/e7df214403f6a019b9b29d239426a465eb964e24