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

[NavigationRailView] Start insets use the system bars instead of the display cutout #3990

Open
t895 opened this issue Jan 11, 2024 · 0 comments · May be fixed by #3991
Open

[NavigationRailView] Start insets use the system bars instead of the display cutout #3990

t895 opened this issue Jan 11, 2024 · 0 comments · May be fixed by #3991
Labels
bug Widget: NavigationRail

Comments

@t895
Copy link

t895 commented Jan 11, 2024

Description: The navigation rail applies start insets based on the system bars. This results in the rail appearing behind display cutouts.

Expected behavior: Apply start insets based on the display cutout

Source code:

material-components-android/lib/java/com/google/android/material/navigationrail/NavigationRailView.java

Line 216 in a2309ce

ViewUtils.isLayoutRtl(view) ? systemBarInsets.right : systemBarInsets.left;

Minimal sample app repro: https://github.com/t895/yuzu/tree/material-update

Android API version: API 34

Material Library version: 1.10.0+ The bug was introduced with this commit

Device: Samsung Galaxy S21 and Google Pixel 7 both running their latest stable software updates as of 1/11/24
@t895 t895 added the bug label Jan 11, 2024
@t895 t895 linked a pull request Jan 11, 2024 that will close this issue
Open
rtiangha added a commit to rtiangha/Lime3DS that referenced this issue May 24, 2024
@rtiangha
Android: revert material to 1.9.0
ed69cc9 
There's been an unresolved bug since version 1.10.0
Details: material-components/material-components-android#3990
OpenSauce04 pushed a commit to Lime3DS/Lime3DS that referenced this issue May 25, 2024
@rtiangha @OpenSauce04
Android: revert material to 1.9.0
7b25c1e 
There's been an unresolved bug since version 1.10.0
Details: material-components/material-components-android#3990
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Widget: NavigationRail
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[NavigationRailView] Use display cutout insets for start padding t895/material-components-android
2 participants
@leticiarossi @t895