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
Use config settings screen_* in physical (unscaled) pixels #3077
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. |
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.
Thanks!
Tested on macOS - everything OK |
This again scales the custom-sized screenshots. Is this thought as precondition for #3071? |
Yes, this doesn't affect screenshot sizes. |
It actually does. Screenshot from Fullscreen has screen pixel count, from windowed mode has configured pixel count from config.ini, but custom-size screenshot deviates from configured size. If this is fixed in #3071, we can continue discussion there after merge of this. |
This commit only affects the correspondence between |
Hello @10110111! The enhancement or feature has been merged into source code and you may test it via building Stellarium from source code or wait the weekly development snapshot... |
Hello @10110111! The fix has been merged into source code and you may test it via building Stellarium from source code or wait the weekly development snapshot... |
Hello @10110111! Please check the fresh version (development snapshot) of Stellarium: |
Hello @10110111! Please check the latest stable version of Stellarium: |
Description
This makes the config settings
screen_{w,h,x,y}
work as defined in units of real pixels—the only sensible user-visible units for any screen. Previously, due to the Qt approach to GUI scaling, these settings were stored and read in units of virtual pixels.I hope this change will relieve the debates about PR #3071.
Type of change
How Has This Been Tested?
Test Configuration:
Checklist: