Skip to content

Commit

Permalink
Merge pull request Expensify#54368 from rushatgabhane/copilot-margin
Browse files Browse the repository at this point in the history
Hide margin if no delegators
  • Loading branch information
techievivek authored Dec 23, 2024
2 parents c0db276 + c2b0ad6 commit c61f3a1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/pages/settings/Security/SecuritySettingsPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ function SecuritySettingsPage() {
title: personalDetail?.displayName ?? formattedEmail,
description: personalDetail?.displayName ? formattedEmail : '',
badgeText: translate('delegate.role', {role}),
avatarID: personalDetail?.accountID ?? -1,
avatarID: personalDetail?.accountID ?? CONST.DEFAULT_NUMBER_ID,
icon: personalDetail?.avatar ?? FallbackAvatar,
iconType: CONST.ICON_TYPE_AVATAR,
numberOfLinesDescription: 1,
Expand Down Expand Up @@ -195,7 +195,7 @@ function SecuritySettingsPage() {
title: personalDetail?.displayName ?? formattedEmail,
description: personalDetail?.displayName ? formattedEmail : '',
badgeText: translate('delegate.role', {role}),
avatarID: personalDetail?.accountID ?? -1,
avatarID: personalDetail?.accountID ?? CONST.DEFAULT_NUMBER_ID,
icon: personalDetail?.avatar ?? FallbackAvatar,
iconType: CONST.ICON_TYPE_AVATAR,
numberOfLinesDescription: 1,
Expand Down Expand Up @@ -303,7 +303,7 @@ function SecuritySettingsPage() {
icon={Expensicons.UserPlus}
onPress={() => Navigation.navigate(ROUTES.SETTINGS_ADD_DELEGATE)}
shouldShowRightIcon
wrapperStyle={[styles.sectionMenuItemTopDescription, styles.mb6]}
wrapperStyle={[styles.sectionMenuItemTopDescription, hasDelegators && styles.mb6]}
/>
)}
{hasDelegators && (
Expand Down

0 comments on commit c61f3a1

Please sign in to comment.