Bug 419591

Summary: "Configure" button is bigger than Weather Report widget added to desktop
Product: [Unmaintained] kdeplasma-addons Reporter: Patrick Silva <bugseforuns>
Component: WeatherAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: nate
Priority: NOR    
Version First Reported In: 5.18.4   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 5.24.7
Sentry Crash Report:
Attachments: screen recording

Description Patrick Silva 2020-04-03 14:38:57 UTC
Created attachment 127238 [details]
screen recording

SUMMARY
Watch the attached screen recording please.

Operating System: Arch Linux 
KDE Plasma Version: 5.18.4
KDE Frameworks Version: 5.68.0
Qt Version: 5.14.2
Comment 1 Patrick Silva 2022-08-11 18:39:44 UTC
Can reproduce.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.25.80
KDE Frameworks Version: 5.97.0
Qt Version: 5.15.5
Graphics Platform: Wayland
Comment 2 Bug Janitor Service 2022-08-11 21:38:30 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/216
Comment 3 Nate Graham 2022-08-12 17:33:42 UTC
Git commit bd848ce47c982dc351e3572dce9e5ae108a6283f by Nate Graham.
Committed on 11/08/2022 at 21:37.
Pushed by ngraham into branch 'master'.

applets/weather: Make desktop widget big enough to fit Configure button

If we don't set Layout.minimum{width,height} properties on the
FullRepresentation, it will default to something tiny and the
Configure… button shown on first launch will overflow.

Set it to a sane size, and also take into account the final layout once
content later gets added so it isn't too small once configured.
FIXED-IN: 5.24.7

M  +5    -0    applets/weather/package/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/kdeplasma-addons/commit/bd848ce47c982dc351e3572dce9e5ae108a6283f
Comment 4 Nate Graham 2022-08-12 17:34:32 UTC
Git commit fdcb4f8982d2084a42a7ace8058d5812c211b982 by Nate Graham.
Committed on 12/08/2022 at 17:34.
Pushed by ngraham into branch 'Plasma/5.25'.

applets/weather: Make desktop widget big enough to fit Configure button

If we don't set Layout.minimum{width,height} properties on the
FullRepresentation, it will default to something tiny and the
Configure… button shown on first launch will overflow.

Set it to a sane size, and also take into account the final layout once
content later gets added so it isn't too small once configured.
FIXED-IN: 5.24.7


(cherry picked from commit bd848ce47c982dc351e3572dce9e5ae108a6283f)

M  +5    -0    applets/weather/package/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/kdeplasma-addons/commit/fdcb4f8982d2084a42a7ace8058d5812c211b982
Comment 5 Nate Graham 2022-08-12 17:34:52 UTC
Git commit d2b68d7671f66a398606f2cc873da732a05560ca by Nate Graham.
Committed on 12/08/2022 at 17:34.
Pushed by ngraham into branch 'Plasma/5.24'.

applets/weather: Make desktop widget big enough to fit Configure button

If we don't set Layout.minimum{width,height} properties on the
FullRepresentation, it will default to something tiny and the
Configure… button shown on first launch will overflow.

Set it to a sane size, and also take into account the final layout once
content later gets added so it isn't too small once configured.
FIXED-IN: 5.24.7


(cherry picked from commit bd848ce47c982dc351e3572dce9e5ae108a6283f)

M  +5    -0    applets/weather/package/contents/ui/FullRepresentation.qml

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