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.

Sign up for GitHub

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

Jump to bottom

The coordinate graph of temperature size is too small #107

Open
linunixer opened this issue Sep 7, 2022 · 14 comments
Open

The coordinate graph of temperature size is too small #107

linunixer opened this issue Sep 7, 2022 · 14 comments
Assignees
@blackadderkate
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@linunixer
Copy link

The coordinate graph of temperature size is too small after upgrade to version 2.3.0
weather-widget-2
Appreciate your work.
OS: Archlinux
DE: KDE Plasma 5.25.5/KDE Framework 5.97.0/Qt version: 5.15.5
Linux Kernel: 5.19.7
Graphics platform: X11
@blackadderkate blackadderkate self-assigned this Sep 7, 2022
@blackadderkate blackadderkate added bug Something isn't working help wanted Extra attention is needed labels Sep 7, 2022
@blackadderkate
Copy link
Owner

The translations are wrong too!

I've had two bugs filed for Archlinux in the last two days - it looks like my widget is incompatible with either Plasma 5.25.5 or KDE Framework 5.97.0.

I develop using Kubuntu 22.04LTS so this is going be difficult for me to fix. I'm going to have to install VitrualBox, and setup a VM before I can even think about trying to fix this bug.

I'd suggest uninstalling 2.3 and going back to the previous version if you can...

@linunixer
Copy link
Author

Thank you @blackadderkate
I downgraded to 2.2.4 and looks good now.
I have an extra question for the best practice to install the weather widget 2:
Is it better to install weather widget 2 through KDE desktop widget manager or is it better to install weather widget 2 through OS's default package manager like apt/rpm/pacman?

@blackadderkate
Copy link
Owner

There are lots of ways - I've put them on the project wiki.

IMHO, the best method is to using the Add Widget method.
This pulls the widget from the KDE store, which is only updated when I've got plenty of free time to fix all the bugs that get reported!

I have just installed VirtualBox in the hope I can install ArchLinux on a VM, to try to get this bug fixed...

@linunixer
Copy link
Author

Thank you @blackadderkate
Here is a background:
I was using KDE store to install the weather widget 2 before, but it's broken because I removed qt5-xmlpatterns accidently(qt5-xmlpatterns was removed because I removed the education package group from KDE). Then someone from Archlinux community recommeded to install from Archlinux AUR.
Again, appreciate your effort on this project! It's really good for use!

@blackadderkate
Copy link
Owner

My attempts to install ArchLinux are not going well.
Downloaded packages keep failing integrity tests, suffering from very slow download speeds, and crashes/errors installing the packages when they download - including a crash right at the end of the install whilst compiling the kernel / installing grub.

Very frustrating.

@linunixer
Copy link
Author

hi @blackadderkate
I am sorry to hear that you were not lucky to install Archlinux VM. I will recommend EndeavoursOS as an alternative, which is a 'almost pure' Archlinux with easy-to-install interface.
If you don't mind, you can install and try EndeavoursOS, which 99% the same as Archlinux.
Anyway, I am happy with weather widget 2 version 2.2.4 now. Just take your time.

@blackadderkate
Copy link
Owner

I finally have ArchLinux installed and working.

OS: Archlinux
DE: KDE Plasma 5.25.5/KDE Framework 5.97.0/Qt version: 5.15.6
Linux Kernel: 5.19.7-arch1-1
Graphics platform: X11

I installed widget version 2.3.0 and it works for me (en_GB locale)...
I shall try to install some different user locales and test further.

Just out of interest, what screen resolution and DPI settings are you running?

@linunixer
Copy link
Author

linunixer commented Sep 12, 2022 •
edited
Loading

my locale is zh_CN
DPI scale 150%
resolution 2560*1440
sent from phone. will provide more detail later

@blackadderkate
Copy link
Owner

my locale is zh_CN DPI scale 150% resolution 2560*1440 sent from phone. will provide more detail later

I have a feeling this is due to your 150% DPI scale setting, and maybe caused by issue #104.

@linunixer
Copy link
Author

my locale is zh_CN DPI scale 150% resolution 2560*1440 sent from phone. will provide more detail later

I have a feeling this is due to your 150% DPI scale setting, and maybe caused by issue #104.

Thank you Kate!
That makes sense. I am glad that the root cause is located. I will keep tuned if there is any fix completed for this issue.
I am good with verision 2.2.4 using. Please take you time.
Again, thank you for your effort and patience!

blackadderkate added a commit that referenced this issue Sep 13, 2022
@blackadderkate
Fix for issue #104 and issue #107
2eb78c6 
Fixes for issue #104 and issue #107.

Change units.devicePixelRatio to Screen.devicePixelRatio and import
QtQuick.Window 2.5 where necessary
@blackadderkate
Copy link
Owner

I have just uploaded version 2.3.1 which fixes the problem on my development Virtual Machine.

@linunixer
Copy link
Author

Thank you @blackadderkate !
I just upgraded to version 2.3.1 and test it.
Everything looks good, except a little bit fonts overlap on the 'date line'
see this screenshot
2022-09-13_09-20
And the font looks a little bit smaller than version 2.2.4(just FYI, it's not a big problem)

@blackadderkate
Copy link
Owner

The font being slightly smaller is something to do with the change they made to DPI scaling.
I've added a line to get the dates so they shrink-to-fit the available area - the font will be smaller, but it won't run together.
I'll release this in version 2.3.2.2 shortly...

@linunixer
Copy link
Author

The font being slightly smaller is something to do with the change they made to DPI scaling. I've added a line to get the dates so they shrink-to-fit the available area - the font will be smaller, but it won't run together. I'll release this in version 2.3.2.2 shortly...

Thank you Kate!
Looks my archlinux have upgraded the weather-widget to version 2.3.2.2
https://aur.archlinux.org/packages/plasma5-applets-weather-widget-2
Just as a testing result, I tried it and looks the problem is mitigated but is still a little bit overlap.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blackadderkate blackadderkate

Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@linunixer @blackadderkate