Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
Everest10 opened this issue Oct 16, 2022 · 19 comments
Open

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

Everest10 opened this issue Oct 16, 2022 · 19 comments
Assignees
Labels
bug Something isn't working

Comments

@Everest10
Copy link

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

@Nowruz
Copy link

Nowruz commented Oct 17, 2022

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

@blackadderkate
Copy link
Owner

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

@fkru
Copy link

fkru commented Oct 17, 2022

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
Copy link

fkru commented Oct 17, 2022

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?

@blackadderkate
Copy link
Owner

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?

@Everest10
Copy link
Author

Everest10 commented Oct 18, 2022

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

@fkru
Copy link

fkru commented Oct 18, 2022

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?

@kppr0
Copy link
Contributor

kppr0 commented Oct 19, 2022

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

@blackadderkate
Copy link
Owner

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
Copy link
Owner

blackadderkate commented Oct 19, 2022

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]

@ZWindL
Copy link

ZWindL commented Oct 21, 2022

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

@Everest10
Copy link
Author

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

@blackadderkate
Copy link
Owner

blackadderkate commented Oct 22, 2022

@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 blackadderkate self-assigned this Oct 22, 2022
@blackadderkate blackadderkate added the bug Something isn't working label Oct 22, 2022
@Grief
Copy link

Grief commented Nov 14, 2022

Same problem, widget appears too big with overlapped labels
image

@Grief
Copy link

Grief commented Nov 14, 2022

@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

@blackadderkate
Copy link
Owner

@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...

@Grief
Copy link

Grief commented Nov 14, 2022

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

@ArchangeGabriel
Copy link

The DPI change is borking the display on HiDPI monitors indeed: #127 (comment)

@phyrz91
Copy link

phyrz91 commented Jan 23, 2023

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

Version: 2.3.4.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

9 participants