weather-widget-2 icon indicating copy to clipboard operation
weather-widget-2 copied to clipboard

Bug: Widget width too small on KDE Plasma Version: 5.26.0

Open Everest10 opened this issue 2 years ago • 19 comments

Operating System: Fedora Linux 37 KDE Plasma Version: 5.26.0 KDE Frameworks Version: 5.99.0 Qt Version: 5.15.6 Graphics Platform: X11

Bug: Widget width too small.

image

Everest10 avatar Oct 16 '22 20:10 Everest10

Same issue here. And it's the same on Wayland.

Operating System: Arch Linux KDE Plasma Version: 5.26.0 KDE Frameworks Version: 5.99.0 Qt Version: 5.15.6 Kernel Version: 6.0.2-arch1-1 (64-bit) Graphics Platform: X11

Nowruz avatar Oct 17 '22 09:10 Nowruz

Duplicate of issue #109 . Fixed in Version 2.3.3 (now also available from the KDE Store)

blackadderkate avatar Oct 17 '22 17:10 blackadderkate

Duplicate of issue #109 . Fixed in Version 2.3.3 (now also available from the KDE Store)

Just installed 2.3.3, still having this issue. On top of that, KDE Store's version deleted all my local settings.

fkru avatar Oct 17 '22 18:10 fkru

Duplicate of issue #109 . Fixed in Version 2.3.3 (now also available from the KDE Store)

Just installed 2.3.3, still having this issue. On top of that, KDE Store's version deleted all my local settings.

Strange, after renaming /usr/share/plasma/plasmoids/org.kde.plasma.weather/contents/ui/FullRepresentaion.qml it works as expected and the issue is gone. What do I miss?

fkru avatar Oct 17 '22 18:10 fkru

Duplicate of issue #109 . Fixed in Version 2.3.3 (now also available from the KDE Store)

Just installed 2.3.3, still having this issue. On top of that, KDE Store's version deleted all my local settings.

Regarding the deleting of your settings: There's virtually no documentation on how to publish things on the KDE Store (pling.com) - I think maybe I did something wrong in the publishing process and Discover is treating it as a new download rather than an upgrade?

blackadderkate avatar Oct 18 '22 12:10 blackadderkate

Uninstalled the old version and deleted the /usr/share/plasma/plasmoids/org.kde.plasma.weather still same problem in 2.3.3

Everest10 avatar Oct 18 '22 19:10 Everest10

Duplicate of issue #109 . Fixed in Version 2.3.3 (now also available from the KDE Store)

Just installed 2.3.3, still having this issue. On top of that, KDE Store's version deleted all my local settings.

Regarding the deleting of your settings: There's virtually no documentation on how to publish things on the KDE Store (pling.com) - I think maybe I did something wrong in the publishing process and Discover is treating it as a new download rather than an upgrade?

Maybe, unfortunately, I don't have insight into KDE Store. By the way, where are the local settings such as location etc. stored?

fkru avatar Oct 18 '22 20:10 fkru

Try uninstall, delete: /home/your_account/.local/share/plasma/plasmoids/org.kde.weatherWidget-2, restart and install new widget version.

kppr0 avatar Oct 19 '22 10:10 kppr0

Uninstalled the old version and deleted the /usr/share/plasma/plasmoids/org.kde.plasma.weather still same problem in 2.3.3

@Everest10: The location depends on how the widget was installed...

If installed by "Get Widget" it will install your home folder in the following location: $HOME/.local/share/plasma/plasmoids/org.kde.weatherWidget-2

If installed using AUR/pacman it will install for all users in /usr/share/plasma/plasmoids/org.kde.weatherWidget-2

Make sure both those folders are gone, download the plasmoid again and install using plasmapkg -i ...

blackadderkate avatar Oct 19 '22 15:10 blackadderkate

By the way, where are the local settings such as location etc. stored?

@fkru they are stored in $HOME/config/plasma-org.kde.plasma.desktop-appletsrc Look for a section starting something like this (although the [2] and [38] will vary according to how you have configured your desktop):

[Containments][2][Applets][38]
immutability=1
plugin=org.kde.weatherWidget-2

Location settings are in this section: [Containments][2][Applets][38][Configuration][General]

blackadderkate avatar Oct 19 '22 15:10 blackadderkate

It appears too big on my laptop now. Distro: Arch Linux KDE Plasma Version: 5.26.1 KDE Frameworks Version: 5.99.0 QT Version: 5.15.6 Graphics Platform: X11 UI Scale: 250% Resolution: 3840x2160 DPI: 210

This package is installed from AUR, and I tried to remove widget -> uninstall package -> install again -> add widget, still that big.

It fills the whole screen, like this.

Screenshot_20221021_161723

ZWindL avatar Oct 21 '22 08:10 ZWindL

It's ok right now. Fedora 37, Plasma 5.26.1

Everest10 avatar Oct 22 '22 00:10 Everest10

@ZWindL OK, I'll need to test with different UI Scale and DPI settings on my development Virtual Machine, to see if I can figure out what is going on.

blackadderkate avatar Oct 22 '22 19:10 blackadderkate

Same problem, widget appears too big with overlapped labels image

Grief avatar Nov 14 '22 11:11 Grief

@blackadderkate I've removed multiplications by devicePixelRatio all across the code and it started to look adequate. Could you please add an option, like ☑ Ignore system DPI into the appearance settings tab which would make those multiplications conditional?

Or maybe those multiplications could be just removed since they are not working, as you can see from the screenshot above. Some values are multiplied and some are not which leads to the overlapping of the elements image

Grief avatar Nov 14 '22 12:11 Grief

@Grief On my development machine, devicePixelRatio is 1 regardless of what I set DPI or the Global Scale to. Presumably this setting only changes if you have an Ultra HD monitor, which I don't have.

I'll remove the multipliers as you suggested...

Longer term, I'd like to implement some kind of scaling setting, so that the Meteogram can be made bigger on very large screens... I'll raise a new issue for that now...

blackadderkate avatar Nov 14 '22 14:11 blackadderkate

You are correct, devicePixelRatio is for monitor's dpi (my screen is 4K), however there are a lot of tricks with it: https://doc.qt.io/qt-5/highdpi.html

Grief avatar Nov 14 '22 15:11 Grief

The DPI change is borking the display on HiDPI monitors indeed: https://github.com/blackadderkate/weather-widget-2/issues/127#issuecomment-1321099183

ArchangeGabriel avatar Nov 27 '22 08:11 ArchangeGabriel

Hi, same problem: On X11 there is no problem. wwidget2.png Screenshot_20230123_125151.md.png

Version: 2.3.4.3

phyrz91 avatar Jan 23 '23 11:01 phyrz91