remove shouldPreventDisableCheckboxIfDisabled prop

Expensify · Dec 9, 2024 · 00f2fb2 · 00f2fb2
1 parent 072684d
commit 00f2fb2
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 5 deletions.
diff --git a/src/components/SelectionList/TableListItem.tsx b/src/components/SelectionList/TableListItem.tsx
@@ -87,7 +87,7 @@ function TableListItem<TItem extends ListItem>({
                             accessibilityLabel={item.text ?? ''}
                             role={CONST.ROLE.BUTTON}
                             // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
-                            disabled={(isDisabled && !item.shouldPreventDisableCheckboxIfDisabled) || item.isDisabledCheckbox}
+                            disabled={isDisabled || item.isDisabledCheckbox}
                             onPress={handleCheckboxPress}
                             style={[styles.cursorUnset, StyleUtils.getCheckboxPressableStyle(), item.isDisabledCheckbox && styles.cursorDisabled, styles.mr3, item.cursorStyle]}
                         >

diff --git a/src/components/SelectionList/types.ts b/src/components/SelectionList/types.ts
@@ -191,9 +191,6 @@ type ListItem = {
 
     /** The style to override the default appearance */
     itemStyle?: StyleProp<ViewStyle>;
-
-    /** Whether to prevent checkbox from being disabled */
-    shouldPreventDisableCheckboxIfDisabled?: boolean;
 };
 
 type TransactionListItemType = ListItem &

diff --git a/src/pages/workspace/WorkspaceMembersPage.tsx b/src/pages/workspace/WorkspaceMembersPage.tsx
@@ -375,7 +375,6 @@ function WorkspaceMembersPage({personalDetails, route, policy, currentUserPerson
                 isDisabledCheckbox: !(isPolicyAdmin && accountID !== policy?.ownerAccountID && accountID !== session?.accountID),
                 isDisabled: isPendingDeleteOrError,
                 isInteractive: !details.isOptimisticPersonalDetail,
-                shouldPreventDisableCheckboxIfDisabled: !isPendingDeleteOrError,
                 cursorStyle: details.isOptimisticPersonalDetail ? styles.cursorDefault : {},
                 text: formatPhoneNumber(PersonalDetailsUtils.getDisplayNameOrDefault(details)),
                 alternateText: formatPhoneNumber(details?.login ?? ''),