Bug 440843 - Popup sometimes cuts off contents wider than itself
Summary: Popup sometimes cuts off contents wider than itself
Status: RESOLVED FIXED
Alias: None
Product: kdeplasma-addons
Classification: Plasma
Component: Weather (show other bugs)
Version: 5.22.4
Platform: Manjaro Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-10 17:53 UTC by Timothy B
Modified: 2023-10-12 18:38 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Screenshot of Weather Report popout with full 7-day forecast (34.64 KB, image/png)
2021-08-10 17:53 UTC, Timothy B
Details
Weather Report popout with long station/city name (90.91 KB, image/png)
2021-08-10 22:11 UTC, Timothy B
Details

Note You need to log in before you can comment on or make changes to this bug.
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.