Bug 440843

Summary: Popup sometimes cuts off contents wider than itself
Product: [Unmaintained] kdeplasma-addons Reporter: Timothy B <yule2000>
Component: WeatherAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: bharadwaj.raju777, nate
Priority: NOR    
Version First Reported In: 5.22.4   
Target Milestone: ---   
Platform: Manjaro   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Screenshot of Weather Report popout with full 7-day forecast
Weather Report popout with long station/city name

Description Timothy B 2021-08-10 17:53:26 UTC 
Created attachment 140640 [details]
Screenshot of Weather Report popout with full 7-day forecast

SUMMARY
The Weather Report plasmoid popup doesn't adapt its width when the full seven-day forecast is shown, causing its contents to be cut off on the righthand side. 

STEPS TO REPRODUCE
1. Add the Weather Report plasmoid to a dock/panel or enable it in the system tray
2. Configure the plasmoid as usual
3. Click on the plasmoid. If there are only six icons in the 7 Days tab, wait patiently until it shows seven icons or change cities until you get a full seven-day forecast

OBSERVED RESULT
All contents in the popout are clipped off to the right when it displays seven days in the forecast.

EXPECTED RESULT
The popout would have its width adjusted to fit the contents nicely.

SOFTWARE/OS VERSIONS
Operating System: Manjaro Linux
KDE Plasma Version: 5.22.4
KDE Frameworks Version: 5.84.0
Qt Version: 5.15.2
Kernel Version: 5.13.6-1-MANJARO (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 1600 Six-Core Processor
Memory: 15.6 GiB of RAM
Graphics Processor: Radeon RX 570 Series

ADDITIONAL INFORMATION
I use Materia (https://store.kde.org/p/1462622/) for the Plasma Look-and-Feel theme and Papirus (https://store.kde.org/p/1166289) for icons, but I believe this can also be reproducible with the default Breeze theme. The attached screenshot was taken on the upper right corner of the screen.

This behavior was not introduced in 5.22; I've encountered this since I started using Plasma with 5.17, and it's possible that this started with a much earlier version of Plasma 5.
Comment 1 Timothy B 2021-08-10 22:11:07 UTC 
Created attachment 140646 [details]
Weather Report popout with long station/city name

It turns out that you don't have to get a full seven-day forecast to reproduce this bug. When I changed to a weather station with a very long name, the popup exhibited the same exact behavior as I initially posted. This time, however, the title is what's causing the clipped overflow here, so as a result, you can't find the temperature on the righthand side of the popout.

Also for this screenshot, I used a different system running Manjaro KDE on a fairly new laptop with Breeze as its overall Plasma theme to make sure this happens regardless of the theme being applied. I've changed the title of the bug accordingly to reflect this discovery.
Comment 2 Nate Graham 2021-08-11 14:55:43 UTC 
Can sometimes reproduce.
Comment 3 Bug Janitor Service 2021-11-11 10:22:53 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kdeplasma-addons/-/merge_requests/98
Comment 4 Bharadwaj Raju 2021-11-11 10:27:01 UTC 
This MR fixes the long-city-name case, but I haven't been able to test the 7-day-forecast case. Once I can test it I'll post an MR for that too.
Comment 5 Bharadwaj Raju 2021-11-11 13:02:30 UTC 
Added fix for long forecasts in that MR too.