Skip to content

Drawer Navigation and Branding#1129

Open
kevinherdez wants to merge 1 commit intoNWACus:kevinherdez/mapProjectfrom
kevinherdez:kevinherdez/drawerNavigationHeaderChanges
Open

Drawer Navigation and Branding#1129
kevinherdez wants to merge 1 commit intoNWACus:kevinherdez/mapProjectfrom
kevinherdez:kevinherdez/drawerNavigationHeaderChanges

Conversation

@kevinherdez
Copy link
Collaborator

This PR address the following issues:

Overview

This pulls in @react-navigation/drawer to move the Menu in the drawer. As there are some big navigation stack refactoring to achieve the results that we want.

Navigation Stack Overview and Refactor

To achieve the behavior we want there are now 2 new levels to the Navigation Stack

RootStackNavigation
|
DrawerNavigator
|
BottomTabNavigator

RootStackNavigator

This contains the Drawer and any views that we want to present on top of the drawer. If you include these screens inside the DrawerNavigator itself, they replace the BottomTabs which isn't necessarily what we want

DrawerNavigator

This only contains the BottomTabs as its main screen and it contains the UI for the drawer itself

BottomTabs

This is the old TabNavigator that was in App.tsx

Design changes

Branding

This PR always shows the NavigationHeader and changes it so that it renders the center name with their icon. It does not display any color branding.

ForecastScreen

This PR removes the ability to change zones when viewing the full avalanche forecast as per designs

Recording with the changes in action

Simulator.Screen.Recording.-.iPhone.17.Pro.-.2026-03-11.at.11.58.51.mov

Stack/Header changes missing from this

  • Move the filter button on the Obs/Weather map up into the nav bar
  • Rearrange the stack navigation to hide the bottom tab in certain views

import {RequestedTime} from 'utils/date';

const RootStack = createNativeStackNavigator<RootStackParamList>();
export const RooStackNavigator: React.FunctionComponent<{
Copy link
Collaborator

@yuliadub yuliadub Mar 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo 🙂 I think

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