Bug 394006 - Weather widget applet window too small by default
Summary: Weather widget applet window too small by default
Status: RESOLVED FIXED
Alias: None
Product: kdeplasma-addons
Classification: Plasma
Component: Weather (show other bugs)
Version: 5.14.2
Platform: Debian stable Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2018-05-08 14:35 UTC by Qasim Ali
Modified: 2019-11-14 03:39 UTC (History)
2 users (show)

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


Attachments
Weather widget popup default size (107.29 KB, image/png)
2018-05-08 14:35 UTC, Qasim Ali
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Qasim Ali 2018-05-08 14:35:09 UTC
Created attachment 112503 [details]
Weather widget popup default size

The default size for the forecast window appears to be too small, causing text and labels to be placed too close to each other. Once resized, the widget appears to be sized properly.
Comment 1 Friedrich W. H. Kossebau 2018-05-09 22:34:51 UTC
Hi, thanks for the report. This is a known issue with the old layout code of the weather widget for the full presentation mode, which has a hard-coded random initial size. not taking into account any possible content.
And that size not only depends on chosen font sizes, but also on data provided by the selected weather service, where some services have lots of days in the forecast, others only a few. 

And with EnvCan there is additionally this glitch that the forecast reports are split into day & night, screwing over the UI concept and doubling the need-for-width.

Myself I have had only adapted this code as maintainer for Plasma 5 and since done some porting and brush over, but did not yet get around to adapt the general Plasma dataengine forecast data model to deal with that.

But at least I already have rewritten the UI layout code, which now adapts to the content (few glitches still, but mainly done).

Sadly this will be only available for Plasma 5.13 and later, given this also was done based on other refactoring done meanwhile. So for the Plasma LTS versions 5.8 & 5.12 this would be too much of a rewrite for this annoying, but relativly minor bug.

Keeping this bug report open for now, but will close it once I am done with the last glitches in the new layout code.
Comment 2 Qasim Ali 2018-05-10 00:52:39 UTC
Understood, thank you for the quick response!
Comment 3 Nate Graham 2019-11-14 03:39:11 UTC
Friedrich is no longer the maintainer, and from my usage of this widget, the layout always works. Let's close this and open new targeted bugs for specific remaining issues.