Skip to content

Commit b091c01

Browse files
authored
Merge pull request Expensify#90332 from callstack-internal/refactor/option-row-avatar
[No QA] refactor: rename OptionRowAvatar to OptionRow/Avatar
2 parents 8cf2a25 + dec3296 commit b091c01

2 files changed

Lines changed: 9 additions & 9 deletions

File tree

src/components/LHNOptionsList/OptionRowLHN/OptionRowAvatar.tsx renamed to src/components/LHNOptionsList/OptionRowLHN/OptionRow/Avatar.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@ import {getDelegateAccountIDFromReportAction} from '@libs/ReportActionsUtils';
77
import type {OptionData} from '@libs/ReportUtils';
88
import CONST from '@src/CONST';
99

10-
type OptionRowAvatarProps = {
10+
type AvatarProps = {
1111
optionItem: OptionData;
1212
isInFocusMode: boolean;
1313
subscriptAvatarBorderColor: ColorValue;
1414
secondaryAvatarBackgroundColor: ColorValue;
1515
singleAvatarContainerStyle: ViewStyle[];
1616
};
1717

18-
function OptionRowAvatarInner({optionItem, isInFocusMode, subscriptAvatarBorderColor, secondaryAvatarBackgroundColor, singleAvatarContainerStyle}: OptionRowAvatarProps) {
18+
function AvatarInner({optionItem, isInFocusMode, subscriptAvatarBorderColor, secondaryAvatarBackgroundColor, singleAvatarContainerStyle}: AvatarProps) {
1919
const personalDetails = usePersonalDetails();
2020

2121
const delegateAccountID = getDelegateAccountIDFromReportAction(optionItem?.parentReportAction);
@@ -64,15 +64,15 @@ function OptionRowAvatarInner({optionItem, isInFocusMode, subscriptAvatarBorderC
6464
);
6565
}
6666

67-
OptionRowAvatarInner.displayName = 'OptionRowAvatarInner';
67+
AvatarInner.displayName = 'OptionRow.AvatarInner';
6868

69-
function OptionRowAvatar({optionItem, isInFocusMode, subscriptAvatarBorderColor, secondaryAvatarBackgroundColor, singleAvatarContainerStyle}: OptionRowAvatarProps) {
69+
function Avatar({optionItem, isInFocusMode, subscriptAvatarBorderColor, secondaryAvatarBackgroundColor, singleAvatarContainerStyle}: AvatarProps) {
7070
// Bail out before subscribing to personal details when the row has no avatar to render.
7171
if (!optionItem.icons?.length || !optionItem.icons.at(0)) {
7272
return null;
7373
}
7474
return (
75-
<OptionRowAvatarInner
75+
<AvatarInner
7676
optionItem={optionItem}
7777
isInFocusMode={isInFocusMode}
7878
subscriptAvatarBorderColor={subscriptAvatarBorderColor}
@@ -82,6 +82,6 @@ function OptionRowAvatar({optionItem, isInFocusMode, subscriptAvatarBorderColor,
8282
);
8383
}
8484

85-
OptionRowAvatar.displayName = 'OptionRowAvatar';
85+
Avatar.displayName = 'OptionRow.Avatar';
8686

87-
export default OptionRowAvatar;
87+
export default Avatar;

src/components/LHNOptionsList/OptionRowLHN/OptionRowLHNCore.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import useTheme from '@hooks/useTheme';
1212
import useThemeStyles from '@hooks/useThemeStyles';
1313
import {shouldUseBoldText} from '@libs/OptionsListUtils';
1414
import CONST from '@src/CONST';
15+
import Avatar from './OptionRow/Avatar';
1516
import DescriptiveText from './OptionRow/DescriptiveText';
1617
import DraftIndicator from './OptionRow/DraftIndicator';
1718
import ErrorBadge from './OptionRow/ErrorBadge';
@@ -23,7 +24,6 @@ import ProductTrainingTooltip from './OptionRow/ProductTrainingTooltip';
2324
import Status from './OptionRow/Status';
2425
import Subtitle from './OptionRow/Subtitle';
2526
import Title from './OptionRow/Title';
26-
import OptionRowAvatar from './OptionRowAvatar';
2727
import OptionRowPressable from './OptionRowPressable';
2828

2929
function OptionRowLHN({isOptionFocused = false, onSelectRow = () => {}, optionItem, viewMode = 'default', style, onLayout = () => {}, hasDraftComment, testID}: OptionRowLHNProps) {
@@ -107,7 +107,7 @@ function OptionRowLHN({isOptionFocused = false, onSelectRow = () => {}, optionIt
107107
<>
108108
<View style={sidebarInnerRowStyle}>
109109
<View style={[styles.flexRow, styles.alignItemsCenter]}>
110-
<OptionRowAvatar
110+
<Avatar
111111
optionItem={optionItem}
112112
isInFocusMode={isInFocusMode}
113113
subscriptAvatarBorderColor={hovered && !isOptionFocused ? hoveredBackgroundColor : subscriptAvatarBorderColor}

0 commit comments

Comments
 (0)