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.

Sign up for GitHub

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

Jump to bottom

Support drawing around notches on Android displays. Fixes #12261 #12779

Merged
merged 1 commit into from
Mar 31, 2020
Merged

Support drawing around notches on Android displays. Fixes #12261 #12779

merged 1 commit into from
Mar 31, 2020

Conversation

hrydgard
Copy link
Owner

Finally finished this up, but I'm struggling to see the point - all phones with notches are so wide anyway that you don't get any additional PSP display area, and do you really want to put controls out there in the limit? You'll just smudge up the camera. Anyway, it's not really any worse so...

Here's what it looks like on a Pocophone F1:

20200331_004144

@hrydgard hrydgard added this to the v1.10.0 milestone Mar 30, 2020
@hrydgard hrydgard mentioned this pull request Mar 30, 2020
Closed
@Panderner
Copy link
Contributor

Panderner commented Mar 30, 2020 •
edited
Loading

How about iPhone X, XS, XR and 11?

unknownbrackets
unknownbrackets approved these changes Mar 31, 2020
View reviewed changes
Copy link
Collaborator

@unknownbrackets unknownbrackets left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean, the menu looks better with the background behind it, at least.

-[Unknown]

@unknownbrackets unknownbrackets merged commit 3965341 into master Mar 31, 2020
@unknownbrackets
Copy link
Collaborator

This makes it easier to implement any similar change for iOS devices, but doesn't actually get the notch information from iOS. Contributions welcome.

-[Unknown]

@Panderner
Copy link
Contributor

Reverse landscape mode doesn't seem to be right

@hrydgard
Copy link
Owner Author

@Panderner can you take a picture?

@hrydgard hrydgard deleted the notch-support branch March 31, 2020 13:21
@hrydgard
Copy link
Owner Author

Ah, I see, I could repro. When going directly from landscape to reverse landscape, it doesn't update the margins properly. I guess we might be doing the checks in the wrong place, I'll get this fixed.

hrydgard added a commit that referenced this pull request Mar 31, 2020
@hrydgard
Fix landscape/reverse landscape cutout padding issue, see #12779
7dfe895
@hrydgard
Copy link
Owner Author

Pushed a fix, should be ok now.

@Panderner
Copy link
Contributor

Panderner commented Mar 31, 2020 •
edited
Loading

And how about Touchscreen test and GPU Driver Test at the developer tools menu? The letters blocked by the notch.

@hrydgard
Copy link
Owner Author

Oh, right, thanks for reminding me about those. They indeed don't properly take the notch into account, I'll fix them up later.

hrydgard referenced this pull request Mar 31, 2020
@hrydgard
Additional rendering fixes for notch
f01a4a3
@hrydgard
Copy link
Owner Author

Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@unknownbrackets unknownbrackets unknownbrackets approved these changes

Assignees
No one assigned
Labels
Platform-specific (Android)
Projects
None yet
Milestone
v1.10.0
Development

Successfully merging this pull request may close these issues.
3 participants
@hrydgard @Panderner @unknownbrackets