-
-
Notifications
You must be signed in to change notification settings - Fork 815
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
Scaling stars on a HiDPI devices #2846
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files |
Why do we even need a flag? Shouldn't it be always-on? |
I cannot test it. Have a 3880x1440 monitor which is not regarded HiDPI. If this is a hardware property which is either 1 or 2, I agree with @10110111 that this should be set automatically. |
No, it's rather an OS property which can be anything from 100% to 500% or something like that. |
OK, I was reading about QScreen::devicePixelRatio(). Is this what you set (on Windows) as gui scaling ("100-500, 500 not recommended")? Checking always requires a logout/login for me, so I don't like to even test it. |
Yes, exactly this setting.
That's strange, at least on Windows 11 it's applied on the fly (Stellarium needs a relaunch). |
I remember it worked on the fly with my previous screen. It may depend on what screen is attached, HDMI or Thunderbolt. Only the latter has the full resolution, though. |
This reverts commit cba1f8e.
OK, it's always on now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine for me.
Hello @alex-w! Please check the fresh version (development snapshot) of Stellarium: |
Hello @alex-w! Please check the latest stable version of Stellarium: |
Description
This PR introducing configurable option to enabling scaling stars on a HiDPI devices to better visualisation of a faint stars
Fixes #2840 (issue)
Screenshots (if appropriate):
Before:
After:
Type of change
Checklist: