-
Notifications
You must be signed in to change notification settings - Fork 97
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
Added multiple aspect ratios #301
Conversation
This is aspect ratio, not screen resolution. |
this is a great point, let me fix this |
@Makkkkus does this look good? |
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.
Good idea, but I don't think this is safe enough yet.
This might also break checking for entities on screen, and other quirks like that.
@Makkkkus does this look good? |
Very good changes! |
Any other questions? |
No |
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.
Very good
Is the start of dealing with #277. We probably need to rethink some of the menus but this works good enough as a start for now.
The default of 4x3 makes sense until we adjust menus for both 4x3 and 16x9.
The "Native" screen resolution option adjusts the screen resolution to that of the main monitor. On a Mac, it'd be 16x10. On most monitors, it'd be 16x9.