-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Fixed touch area of bottom navigation bar to be filled. #22685
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
Conversation
Previously, the top of touch area of BottomNavigationBarItem is slightly cutted off.
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
|
I've submitted |
|
CLAs look good, thanks! |
tvolkert
left a comment
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.
Thanks for the contribution @dooboolab!
Could you add a test in bottom_navigation_bar_test.dart that would have caught this problem? It'll ensure we don't accidentally regress it in the future.
|
/cc @HansMuller |
Since, giving additionalBottomPadding with symmetric, we need to multiply the value by 2.
|
@tvolkert I've like to write up test for this too. However, how can I test if my test code is working correctly? I could fix this problem in Could you provide me some guideline? |
|
Thanks for your contribution. The before and after animated GIFs are superb! Unfortunately, we need to address two problems before landing this fix:
This is no longer a simple change however if you'd like to attempt it, I can help. |
|
I'll try to manage above when #22956 is merged because current |
|
I was going to continue working on this but it seems that this has been taken in #28159. |
The ripple effect still does not match the Material spec when I use it. Are there any examples of this working in the wild? |

Previously, the top of the touch area of BottomNavigationBarItem is slightly cut off.
Previous result:

Fixed result:
