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

Incorrect UI scaling #6

Closed
dromer opened this issue Dec 24, 2023 · 9 comments
Closed

Incorrect UI scaling #6

dromer opened this issue Dec 24, 2023 · 9 comments

Comments

@dromer
Copy link

dromer commented Dec 24, 2023

At the moment the default size is rather huge, which makes the entire UI very difficult to view and navigate.
For the graph we can zoom out, but the rest of the UI is still very bulky this way.

Please add an option to change the default font size, so we can adjust the way the entire UI gets sized.

@dimtpap
Copy link
Owner

dimtpap commented Dec 24, 2023

This may be a bug, can you provide a screenshot?

@dromer
Copy link
Author

dromer commented Dec 24, 2023

Sure:

@dimtpap
Copy link
Owner

dimtpap commented Dec 28, 2023

I tried on latest Ubuntu X11 and got the same behavior. Linux Mint X11 though had no problems. What distro and window system are you running?

@dromer
Copy link
Author

dromer commented Dec 28, 2023

Debian 12, X11 and Fluxbox.

Not having any fancy Hi-DPI going btw.

@dimtpap
Copy link
Owner

dimtpap commented Dec 28, 2023

The scaling is set based on Xft.dpi, maybe it's not set correctly in your setup. As a workaround you can run coppwr with the WINIT_X11_SCALE_FACTOR environment variable set to your display's scale factor, or whatever works best. For example WINIT_X11_SCALE_FACTOR=1 coppwr

@dromer
Copy link
Author

dromer commented Dec 28, 2023

Indeed WINIT_X11_SCALE_FACTOR is not set. If I set it to 1 everything looks good!

So maybe default to 1 when the env-var is not used?

@dimtpap
Copy link
Owner

dimtpap commented Dec 28, 2023

It may be a bug in winit, the windowing library used by coppwr, that should be reported. I'll try messing with Xft.dpi to see if it's respected. If it is, it's an end-user X11 configuration issue, otherwise it's a bug in winit

@dimtpap dimtpap changed the title Add setting for default font size Incorrect UI scaling Dec 28, 2023
@dromer
Copy link
Author

dromer commented Dec 28, 2023

Hmm, I don't think expecting unused envvars to be set makes a lot of sense.
This is actually the first time I hear about this one.

Can you check with the winit devs then? This is a bit out of my league (I'm just a poor user ;) ).

Maybe related? -> rust-windowing/winit#2231

@dimtpap
Copy link
Owner

dimtpap commented Dec 28, 2023

Maybe related? -> rust-windowing/winit#2231

Yes, winit falls back to a manual calculation of the factor based on the display's properties if neither Xft.dpi nor the env var are set. Thanks for linking it

@dimtpap dimtpap closed this as completed Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants