From cc1ca4e95954e39ec0d4bda8e6f20bc3ff4d85d0 Mon Sep 17 00:00:00 2001
From: Shubham Agrawal <58412969+shubham1206agra@users.noreply.github.com>
Date: Wed, 13 Mar 2024 20:42:04 +0530
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: Carlos Martins <luacmartins@gmail.com>
Co-authored-by: Ishpaul Singh <104348397+ishpaul777@users.noreply.github.com>
---
 src/components/SelectionList/BaseListItem.tsx | 6 +++---
 src/components/SelectionList/types.ts         | 4 ++--
 src/pages/workspace/WorkspaceInvitePage.tsx   | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/components/SelectionList/BaseListItem.tsx b/src/components/SelectionList/BaseListItem.tsx
index 0ea868d6d24e..a7dafc37e467 100644
--- a/src/components/SelectionList/BaseListItem.tsx
+++ b/src/components/SelectionList/BaseListItem.tsx
@@ -24,7 +24,7 @@ function BaseListItem<TItem extends ListItem>({
     onCheckboxPress,
     onDismissError = () => {},
     rightHandSideComponent,
-    checkmarkPosition = 'left',
+    checkmarkPosition = CONST.DIRECTION.LEFT,
     keyForList,
     errors,
     pendingAction,
@@ -78,7 +78,7 @@ function BaseListItem<TItem extends ListItem>({
                 style={pressableStyle}
             >
                 <View style={wrapperStyle}>
-                    {canSelectMultiple && checkmarkPosition === 'left' && (
+                    {canSelectMultiple && checkmarkPosition === CONST.DIRECTION.LEFT && (
                         <PressableWithFeedback
                             accessibilityLabel={item.text ?? ''}
                             role={CONST.ROLE.BUTTON}
@@ -101,7 +101,7 @@ function BaseListItem<TItem extends ListItem>({
 
                     {typeof children === 'function' ? children(hovered) : children}
 
-                    {canSelectMultiple && checkmarkPosition === 'right' && (
+                    {canSelectMultiple && checkmarkPosition === CONST.DIRECTION.RIGHT && (
                         <PressableWithFeedback
                             onPress={handleCheckboxPress}
                             disabled={isDisabled}
diff --git a/src/components/SelectionList/types.ts b/src/components/SelectionList/types.ts
index 5851f90276e4..11c10021bc78 100644
--- a/src/components/SelectionList/types.ts
+++ b/src/components/SelectionList/types.ts
@@ -35,7 +35,7 @@ type CommonListItemProps<TItem> = {
     rightHandSideComponent?: ((item: TItem) => ReactElement<TItem>) | ReactElement | null;
 
     /** Direction of checkmark to show */
-    checkmarkPosition?: 'left' | 'right';
+    checkmarkPosition?: ValueOf<typeof CONST.DIRECTION>;
 
     /** Styles for the pressable component */
     pressableStyle?: StyleProp<ViewStyle>;
@@ -271,7 +271,7 @@ type BaseSelectionListProps<TItem extends ListItem> = Partial<ChildrenProps> & {
     rightHandSideComponent?: ((item: ListItem) => ReactElement<ListItem>) | ReactElement | null;
 
     /** Direction of checkmark to show */
-    checkmarkPosition?: 'left' | 'right';
+    checkmarkPosition?: ValueOf<typeof CONST.DIRECTION>;
 
     /** Whether to show the loading indicator for new options */
     isLoadingNewOptions?: boolean;
diff --git a/src/pages/workspace/WorkspaceInvitePage.tsx b/src/pages/workspace/WorkspaceInvitePage.tsx
index afca6c29aee8..00d7eaa33f6b 100644
--- a/src/pages/workspace/WorkspaceInvitePage.tsx
+++ b/src/pages/workspace/WorkspaceInvitePage.tsx
@@ -321,7 +321,7 @@ function WorkspaceInvitePage({
                     showScrollIndicator
                     showLoadingPlaceholder={!didScreenTransitionEnd || !OptionsListUtils.isPersonalDetailsReady(personalDetailsProp)}
                     shouldPreventDefaultFocusOnSelectRow={!DeviceCapabilities.canUseTouchScreen()}
-                    checkmarkPosition="right"
+                    checkmarkPosition={CONST.DIRECTION.RIGHT}
                 />
                 <View style={[styles.flexShrink0]}>
                     <FormAlertWithSubmitButton