-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Add setting to allow forcing the game to draw over notches / cameras #20983
Conversation
6d2e47f
to
f9d2c30
Compare
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.
code seems sane - mostly tested via test scene, not tested on device, so there's a chance the toggle may not appear but looks like it should
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.
Tested on iOS, works as promised.
if (host.Window?.SafeAreaPadding.Value.Total != Vector2.Zero) | ||
safeAreaConsiderationsCheckbox.Show(); | ||
else | ||
safeAreaConsiderationsCheckbox.Hide(); |
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.
Slightly safer to update visibility based on value changes to host.Window.SafeAreaPadding
in case it gets updated later than the display.
safeAreaConsiderationsCheckbox = new SettingsCheckbox | ||
{ | ||
LabelText = "Shrink game to avoid cameras and notches", | ||
Current = osuConfig.GetBindable<bool>(OsuSetting.SafeAreaConsiderations), | ||
}, |
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.
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.
Disagree. This only shows on mobile. It makes sense for it to be the topmost setting there for me.
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.
Perhaps the padding is what makes it look weird. Dunno.
Setting will only show up when running on a device that exposes safe areas.
Addresses a common point of user contention as seen by the following discussions (and probably more):
#20970
#15506
#19737
#17769
#18395