[kdeplasma-addons] [Bug 395496] Weather applet layout offset to the right and not visible with long city names

2019-11-24 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395496

Nate Graham  changed:

   What|Removed |Added

 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Nate Graham  ---
This has since been fixed.

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdeplasma-addons] [Bug 395496] Weather applet layout offset to the right and not visible with long city names

2019-10-21 Thread Friedrich W. H. Kossebau
https://bugs.kde.org/show_bug.cgi?id=395496

Friedrich W. H. Kossebau  changed:

   What|Removed |Added

   Assignee|kosse...@kde.org|plasma-b...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdeplasma-addons] [Bug 395496] Weather applet layout offset to the right and not visible with long city names

2019-03-13 Thread Adel Q
https://bugs.kde.org/show_bug.cgi?id=395496

--- Comment #4 from Adel Q  ---
(In reply to Christoph Feck from comment #1)
> Is it caused by the long city name? Does it reproduce with a shorter name?
> Which weather provider are you using?

Yes, it is caused by the long city name. When using a shorter city name, the
tabs are shown fine. I am using the NOAA weather provider. The location to
reproduce is "Detroit, Detroit Metropolitan Wayne County Airport, MI (noaa)".

(In reply to Duncan from comment #3)
> This second method, adding the weather-report plasmoid directly to the panel
> (or desktop if preferred), instead of nesting it inside the system-tray
> plasmoid, avoids the popup-window size constraints of the system-tray, thus
> avoiding the problem. =:^)

I can confirm that this happens in my case as well -- the plasmoid resizes to
the full width of the city name when placed as an independent plasmoid rather
than in the system-tray. I think the preferred behavior should be truncating
the location name with "..." rather than overextending. For an extreme example,
try the "Bloomington/Normal, Central Illinois Regional Airport at
Bloomington-Normal" NOAA station, the weather widget shouldn't take up half the
screen.

For testing purposes of different stations, you can use the following to get
the list of NOAA stations sorted by length:

wget https://w1.weather.gov/xml/current_obs/index.xml
xmllint --xpath '//station/station_name/text()' index.xml | awk '{ print
length, $0 }' | sort -rns | less

Bug is still reproducible in the most current releases.

Distro: Arch Linux (up to date)
Qt Version: 5.12.1
KDE Plasma Version: 5.15.3
KDE Frameworks Version: 5.56.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdeplasma-addons] [Bug 395496] Weather applet layout offset to the right and not visible with long city names

2019-03-12 Thread Duncan
https://bugs.kde.org/show_bug.cgi?id=395496

Duncan <1i5t5.dun...@cox.net> changed:

   What|Removed |Added

 CC||1i5t5.dun...@cox.net

--- Comment #3 from Duncan <1i5t5.dun...@cox.net> ---
Same problem here for some time, on gentoo, running live-git (live-git ebuilds
from the gentoo/kde overlay) of pretty much everything kde related including
all the frameworks and plasma, thus including kdeplasma-addons.

(In reply to Friedrich W. H. Kossebau from comment #2)
> It is caused by the weather applet being used here inside the systemtray
> popup container, which has a hardcoded(?) aize. While the weather applet now
> (>= Plasma 5.13) assumes it can grow to the size it needs, always trying to
> show e.g. all forecast days.

The apparently hard-coded size does indeed seem to be the problem.

> A fix will need to have the applet adapt to the available size in some way.
> 
> No idea yet, needs me to sit down some longer time for it.

The hints here (length of city name and system-tray hard-coded size limits)
*did* suggest a workaround, however, which I can confirm works, having just
tried it after reading this bug. =:^)

Note that there's *TWO* ways to add a weather-report plasmoid, either nested
inside the system-tray plasmoid, which in turn can be placed in a panel (as is
traditional) or directly on the desktop activity (when it's in desktop/widgets
mode, not folderview mode).

Adding it nested in the system-tray is done via simple checkmark in the
system-tray config, and nested in the system-tray is where the popup sizes are
constrained, so this is the problem location.

But, it's also possible (with widgets unlocked) to select add-widgets, either
from the (desktop-mode) desktop config menu, or from the panel config menu,
then in the popup window with all the widgets/plasmoids displayed, scroll
toward the bottom and find the weather report widget/plasmoid, and drag it to
the desired location on the desktop or in the panel.

This second method, adding the weather-report plasmoid directly to the panel
(or desktop if preferred), instead of nesting it inside the system-tray
plasmoid, avoids the popup-window size constraints of the system-tray, thus
avoiding the problem. =:^)

Thanks for the hints that allowed me to figure that out.  This had been
frustrating me for quite some time, and now at least I have a workaround while
we wait for a fix to the system-tray nesting constraints, putting it directly
in the panel instead of nested in the system-tray. =:^)

-- 
You are receiving this mail because:
You are watching all bug changes.

[kdeplasma-addons] [Bug 395496] Weather applet layout offset to the right and not visible with long city names

2019-01-08 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=395496

Nate Graham  changed:

   What|Removed |Added

Summary|Weather applet layout   |Weather applet layout
   |offset to the right and not |offset to the right and not
   |visible |visible with long city
   ||names

-- 
You are receiving this mail because:
You are watching all bug changes.