Skip to content

Commit e05152d

Browse files
authored
Merge pull request Expensify#90210 from rushatgabhane/a11y-remove-participant
Announce remove action on selected participant
2 parents ae9cd88 + aee1e8a commit e05152d

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

src/components/SelectionList/ListItem/UserListItemContent.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {ListItemFocusContext} from '@components/SelectionList/ListItemFocusConte
88
import getAccessibilityLabel from '@components/SelectionList/utils/getAccessibilityLabel';
99
import TextWithTooltip from '@components/TextWithTooltip';
1010
import {useMemoizedLazyExpensifyIcons} from '@hooks/useLazyAsset';
11+
import useLocalize from '@hooks/useLocalize';
1112
import useOnyx from '@hooks/useOnyx';
1213
import useStyleUtils from '@hooks/useStyleUtils';
1314
import useTheme from '@hooks/useTheme';
@@ -53,6 +54,7 @@ function UserListItemContent<TItem extends ListItem>({
5354
const styles = useThemeStyles();
5455
const theme = useTheme();
5556
const StyleUtils = useStyleUtils();
57+
const {translate} = useLocalize();
5658

5759
const focusedBackgroundColor = styles.sidebarLinkActive.backgroundColor;
5860
const subscriptAvatarBorderColor = isFocused ? focusedBackgroundColor : theme.sidebar;
@@ -72,12 +74,16 @@ function UserListItemContent<TItem extends ListItem>({
7274
const policyID = isThereOnlyWorkspaceIcon && shouldUseIconPolicyID ? String(item.icons?.at(0)?.id) : item.policyID;
7375

7476
const isHovered = hovered && !shouldDisableHoverStyle;
75-
const contactAccessibilityLabel = getAccessibilityLabel(item);
77+
const baseAccessibilityLabel = getAccessibilityLabel(item);
78+
const accessibilityLabel =
79+
shouldDisableAccessibleGrouping && item.isSelected !== undefined
80+
? `${translate(item.isSelected ? 'common.deselect' : 'common.select')}, ${baseAccessibilityLabel}`
81+
: baseAccessibilityLabel;
7682

7783
return (
7884
<View
7985
accessible={shouldDisableAccessibleGrouping || undefined}
80-
accessibilityLabel={shouldDisableAccessibleGrouping ? contactAccessibilityLabel : undefined}
86+
accessibilityLabel={shouldDisableAccessibleGrouping ? accessibilityLabel : undefined}
8187
role={shouldDisableAccessibleGrouping ? CONST.ROLE.BUTTON : undefined}
8288
style={[styles.flex1, styles.flexRow, styles.alignItemsCenter]}
8389
>

0 commit comments

Comments
 (0)