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

navigation background color and bottom tab ripple #4118

Open
FouadMagdy01 opened this issue Oct 7, 2023 · 5 comments
Open

navigation background color and bottom tab ripple #4118

FouadMagdy01 opened this issue Oct 7, 2023 · 5 comments

Comments

@FouadMagdy01
Copy link

FouadMagdy01 commented Oct 7, 2023

According to google's official apps that uses material you, there is some differences between the react native paper and the official material you design

1- in the bottom tabs, the ripple effect in the official design is only affect the bottom tab icon, not the whole button

2-when you adapt the react navigation theme, the background color is different from google's official apps background color,
I think the problem is that the background color property in the navigation theme should be the same value as the card property.

I say this because in the example app, the drawer background color have the same color of the google's main apps background color.

Thanks in advance

@lukewalczak
Copy link
Member

Hey @FouadMagdy01, could you please attach the screenshots presenting the differences to visualize them?

@FouadMagdy01
Copy link
Author

FouadMagdy01 commented Oct 7, 2023

Sure @lukewalczak.
Here is the difference
Screenshot_2023-10-07-18-32-34-851_com google android apps docs

This is a screen from google drive app, you see the background color

Screenshot_2023-10-07-18-32-46-186_com callstack reactnativepaperexample

Here is a screen from the example app, the background color is not the same

Screenshot_2023-10-07-18-11-36-890_com callstack reactnativepaperexample

The navigation background color should be similar to the drawer background color

@FouadMagdy01
Copy link
Author

Screenrecorder-2023-10-07-18-35-36-5091.mp4

This is the ripple effect behavior in google drive app

Screenrecorder-2023-10-07-18-35-50-4411.mp4

And this is the behavior in the example app

There should be two changes

1- ripple color should be changed
2- ripple effect should be moved from the whole tab button to be only in the icon

@lukewalczak
Copy link
Member

1- in the bottom tabs, the ripple effect in the official design is only affect the bottom tab icon, not the whole button

as far as I remember initially bottom navigation even doesn't have the ripple effect, that why the rippleColor is set to transparent.

The bug that you've observed is related to the Android 13, it was already reported in the separate issue, however I've received the message it should be fixed.

Anyway I'm going the check the documentation, whether there should be a ripple effect on the icon in the bottom tabs.

2-when you adapt the react navigation theme, the background color is different from google's official apps background color,
I think the problem is that the background color property in the navigation theme should be the same value as the card property.

The issue is not related to the component itself, we are using there just a plain View which doesn't have a specified background color, so it takes some default. I will adjust it to be consistent.

@FouadMagdy01
Copy link
Author

Thanks so much for your great work and active replies

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants