Fixed touch area of bottom navigation bar to be filled.#22685
Fixed touch area of bottom navigation bar to be filled.#22685hyochan wants to merge 3 commits intoflutter:masterfrom crossplatformkorea:master
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.
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:
