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.

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 overlay for Xiaomi Redmi Note 10 #605

Merged
merged 7 commits into from Sep 16, 2022
Merged

Add overlay for Xiaomi Redmi Note 10 #605

merged 7 commits into from Sep 16, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jul 26, 2022

Tested

Prince Cooper added 6 commits July 27, 2022 06:38
Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
causes screen flashes while switching between landscape & portrait

Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
@ghost
Copy link
Author

ghost commented Aug 1, 2022

@azan-n
Copy link

azan-n commented Aug 13, 2022

#535 might help since it is essentially the same device.

The SystemUI definitely needs overlaying. Manually setting the rounded corners fixes truncation but the status bar height needs to be set for the notch to work correctly.

@ghost
Copy link
Author

ghost commented Aug 13, 2022

@azan-n I've tested all the scenarios of portrait & landcapes & did not found a single notch issue. You can literally take example of RN11 where he didn't even include any dimensional values.

Manually setting the rounded corners fixes truncation

That's the reason I atleast included rounded corner dimens.

The SystemUI definitely needs overlaying

I don't think so. System UI attributes aren't really necessary as they're just purely aesthetic paddings & give no extra functionality. There aren't any issues whatsoever with the existing strings tbh.

@phhusson
Copy link
Owner

what's the status here? is this pr mergeable?

@ghost
Copy link
Author

ghost commented Aug 24, 2022

what's the status here? is this pr mergeable?

Yes go ahead

reverts mistakenly modified RN10P strings by me

Signed-off-by: Prince Cooper <8982359-pc00per@users.noreply.gitlab.com>
@phhusson phhusson merged commit 76cd5b0 into phhusson:pie Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants