From 0ff9195eefa49035ac65305735ae5f96099be353 Mon Sep 17 00:00:00 2001 From: situchan Date: Wed, 17 Dec 2025 17:45:20 +0600 Subject: [PATCH 1/2] Revert "Merge pull request #76184 from situchan/revert-74667-fix/74496" This reverts commit 294c3c1516aa1c42c85af1268fc3152c77eddcdb, reversing changes made to 70ee2f51e96074a20ffaf6f3febb7b7a57a1bbaa. --- src/components/Section/index.tsx | 2 +- src/pages/workspace/WorkspaceOverviewPage.tsx | 81 +++++++++---------- 2 files changed, 39 insertions(+), 44 deletions(-) diff --git a/src/components/Section/index.tsx b/src/components/Section/index.tsx index 139c9ff224b40..9b1fa58eb3ab8 100644 --- a/src/components/Section/index.tsx +++ b/src/components/Section/index.tsx @@ -186,7 +186,7 @@ function Section({ /> )} - {renderTitle ? renderTitle() : {title}} + {renderTitle ? renderTitle() : !!title && {title}} {cardLayout === CARD_LAYOUT.ICON_ON_RIGHT && ( = shouldUseNarrowLayout ? [styles.mhv12, styles.mhn5, styles.mbn5] : [styles.mhv8, styles.mhn8, styles.mbn5]; const shouldShowAddress = !readOnly || !!formattedAddress; const {isAccountLocked, showLockedAccountModal} = useContext(LockedAccountContext); const [lastPaymentMethod] = useOnyx(ONYXKEYS.NVP_LAST_PAYMENT_METHOD, {canBeMissing: true}); @@ -385,20 +383,27 @@ function WorkspaceOverviewPage({policyDraft, policy: policyProp, route}: Workspa }; const renderDropdownMenu = (options: Array>) => ( - - {}} - shouldAlwaysShowDropdownMenu - customText={translate('common.more')} - options={options} - isSplitButton={false} - wrapperStyle={styles.flexGrow1} - /> - + {}} + shouldAlwaysShowDropdownMenu + customText={translate('common.more')} + options={options} + isSplitButton={false} + wrapperStyle={isPolicyAdmin ? styles.flexGrow0 : styles.flexGrow1} + /> ); + const handleInvitePress = useCallback(() => { + if (isAccountLocked) { + showLockedAccountModal(); + return; + } + clearInviteDraft(route.params.policyID); + Navigation.navigate(ROUTES.WORKSPACE_INVITE.getRoute(route.params.policyID, Navigation.getActiveRouteWithoutParams())); + }, [isAccountLocked, showLockedAccountModal, route.params.policyID]); + const getHeaderButtons = () => { const secondaryActions: Array> = []; const canLeave = !isOwner; @@ -416,21 +421,6 @@ function WorkspaceOverviewPage({policyDraft, policy: policyProp, route}: Workspa return null; } - if (isPolicyAdmin) { - secondaryActions.push({ - value: 'invite', - text: translate('common.invite'), - icon: expensifyIcons.UserPlus, - onSelected: () => { - if (isAccountLocked) { - showLockedAccountModal(); - return; - } - clearInviteDraft(route.params.policyID); - Navigation.navigate(ROUTES.WORKSPACE_INVITE.getRoute(route.params.policyID, Navigation.getActiveRouteWithoutParams())); - }, - }); - } secondaryActions.push({ value: 'share', text: translate('common.share'), @@ -467,7 +457,22 @@ function WorkspaceOverviewPage({policyDraft, policy: policyProp, route}: Workspa }); } - return renderDropdownMenu(secondaryActions); + return ( + + {isPolicyAdmin && ( +