Skip to content

Commit

Permalink
Merge pull request #20 from software-mansion-labs/@kosmydel/ideal-nav…
Browse files Browse the repository at this point in the history
…-small-fixes

fix back button
  • Loading branch information
MaciejSWM committed Jan 4, 2024
2 parents f81f7f2 + b3fd73e commit bf89a11
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
3 changes: 0 additions & 3 deletions src/components/HeaderWithBackButton/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -108,9 +108,6 @@ type HeaderWithBackButtonProps = Partial<ChildrenProps> & {

/** Whether we should enable detail page navigation */
shouldEnableDetailPageNavigation?: boolean;

/** TEMP: Will be removed in my other PR */
shouldUseCentralPaneView: boolean;
};

export default HeaderWithBackButtonProps;
2 changes: 1 addition & 1 deletion src/components/MenuItem.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ type MenuItemProps = (ResponsiveProps | UnresponsiveProps) &
contentFit?: ImageContentFit;

/** Is this in the Pane */
isPaneMenu: false;
isPaneMenu?: false;
};

function MenuItem(
Expand Down
5 changes: 2 additions & 3 deletions src/pages/workspace/WorkspacesListPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -298,7 +298,7 @@ function WorkspacesListPage({policies, allPolicyMembers, reimbursementAccount, c
onBackButtonPress={() => Navigation.goBack(ROUTES.SETTINGS.ROOT)}
title={translate('common.workspaces')}
style={!isSmallScreenWidth && styles.alignItemsCenter}
shouldUseCentralPaneView
shouldShowBackButton={isSmallScreenWidth}
footer={
isSmallScreenWidth && (
<Button
Expand Down Expand Up @@ -336,8 +336,7 @@ function WorkspacesListPage({policies, allPolicyMembers, reimbursementAccount, c
<View style={{flex: 1}}>
<HeaderWithBackButton
title={translate('common.workspaces')}
shouldShowBorderBottom
shouldUseCentralPaneView
shouldShowBackButton={isSmallScreenWidth}
>
<Button
accessibilityLabel={translate('workspace.new.newWorkspace')}
Expand Down

0 comments on commit bf89a11

Please sign in to comment.