Skip to content

feat: adjust BottomNavigation#3083

Merged
lukewalczak merged 8 commits intonextfrom
feat/adjust-bottom-navigation
May 11, 2022
Merged

feat: adjust BottomNavigation#3083
lukewalczak merged 8 commits intonextfrom
feat/adjust-bottom-navigation

Conversation

@lukewalczak
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

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
Copy Markdown

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

.

2 similar comments
@github-actions
Copy link
Copy Markdown

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

.

@github-actions
Copy link
Copy Markdown

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
Copy Markdown

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

.

@github-actions
Copy link
Copy Markdown

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