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

[nav] Basic bottom nav implementation #12

Merged
merged 1 commit into from
Oct 2, 2023
Merged

Conversation

emilysunaryo
Copy link
Contributor

What's new in this PR

Added Material Bottom Navigation Bar package from React Native and added basic routing functionality as well as a icon changes. In the process of adding a separate icons.tsx file to be able to have cleaner code although this is not fully implemented and the icons are hard coded currently

Relevant Links

Notion Sprint Task

https://www.notion.so/calblueprint/b885547588f4406bb7f6f8153be99bb9?v=6572442cb5c644a9bd1c298d53a85eec&p=5b85a042022c4bdbbd0199001ff28f13&pm=s

Online sources

Related PRs

How to review

Next steps

the icons.tsx file under assets is still something that I need to fully implement so we can start importing an Icon component rather than hard coding the icons directly. I also need to add styling for the bottom nav bar as well. (figuring out how to get rid of the purple hue )

Tests Performed, Edge Cases

Screenshots

Screenshot 2023-10-01 at 9 48 18 PM

CC: @akshaynthakur

@akshaynthakur
Copy link
Collaborator

LGTM!!!

@akshaynthakur akshaynthakur changed the title basic bottom nav implementation [nav] basic bottom nav implementation Oct 2, 2023
@akshaynthakur akshaynthakur changed the title [nav] basic bottom nav implementation [nav] Basic bottom nav implementation Oct 2, 2023
@akshaynthakur akshaynthakur merged commit 41ea77a into main Oct 2, 2023
2 checks passed
@akshaynthakur akshaynthakur deleted the emily/bottomnavbar branch October 3, 2023 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants