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

feat: adjust BottomNavigation #3083

Merged
merged 8 commits into from
May 11, 2022
Merged

Conversation

lukewalczak
Copy link
Member

@lukewalczak lukewalczak commented Feb 7, 2022

Summary

PR:

  • adjusts BottomNavigation into Material You.
  • introduced prop compact indicating whether tabs should be spread across the entire width.
  • couple changes in navigationState.routes:
    • renamed from icon to focusedIcon
    • introduced unfocusedIcon
    • deprecated color

Test plan

  • check application in both themes, if BottomNavigation works and have correct colors in both themes on both platforms
  • rotate the phone to confirm whether BottomNavigation works properly in horizontal mode
  • snapshots indicate that BottomNavigation from theme version 2 wasn't changed.

@callstack-bot
Copy link

callstack-bot commented Feb 7, 2022

Hey @lukewalczak, thank you for your pull request 🤗. The documentation from this branch can be viewed here.

@lukewalczak lukewalczak force-pushed the feat/adjust-bottom-navigation branch from 0ec3fba to 9936468 Compare February 7, 2022 20:01
@lukewalczak lukewalczak force-pushed the feat/adjust-bottom-navigation branch from 4b3e3a4 to dbf7660 Compare March 9, 2022 21:08
@github-actions
Copy link

github-actions bot commented Mar 9, 2022

The mobile version of example app from this branch is ready! You can see it here

.

@lukewalczak lukewalczak force-pushed the feat/adjust-bottom-navigation branch from dbf7660 to 96a0ee9 Compare March 10, 2022 12:29
@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

2 similar comments
@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@lukewalczak lukewalczak force-pushed the feat/adjust-bottom-navigation branch from d4277db to 4aee358 Compare March 26, 2022 13:48
@lukewalczak lukewalczak marked this pull request as ready for review March 26, 2022 13:52
@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@github-actions
Copy link

The mobile version of example app from this branch is ready! You can see it here

.

@lukewalczak lukewalczak merged commit c36bd99 into next May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants