From 1f3d2a91dd404bf790663bd9bde5cbe460a68b83 Mon Sep 17 00:00:00 2001
From: Chikaodi Adibemma <53785439+cadibemma@users.noreply.github.com>
Date: Wed, 18 Sep 2024 17:30:23 -0400
Subject: [PATCH] 
 bug/9269-Chika-NonActionableBoxesFlaggedAsInteractiveWithVoiceControl (#9528)

---
 VAMobile/src/components/BaseListItem.tsx                      | 2 +-
 .../DisabilityRatingsScreen/DisabilityRatingsScreen.tsx       | 3 ++-
 .../Pharmacy/PrescriptionDetails/DetailsTextSections.tsx      | 4 ++--
 .../SecureMessaging/NoInboxMessages/NoInboxMessages.tsx       | 2 +-
 .../DirectDepositScreen/DirectDepositScreen.tsx               | 2 +-
 5 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/VAMobile/src/components/BaseListItem.tsx b/VAMobile/src/components/BaseListItem.tsx
index e4654023dc8..dcf68414355 100644
--- a/VAMobile/src/components/BaseListItem.tsx
+++ b/VAMobile/src/components/BaseListItem.tsx
@@ -235,7 +235,6 @@ const BaseListItem: FC<BaseListItemProps> = (props) => {
     onPressIn: _onPressIn,
     onPressOut: _onPressOut,
     accessible: true,
-    accessibilityRole,
     disabled: decorator === ButtonDecoratorType.RadioDisabled,
   }
 
@@ -258,6 +257,7 @@ const BaseListItem: FC<BaseListItemProps> = (props) => {
     ...a11yValueProp(a11yValue ? { text: a11yValue } : {}),
     accessibilityState: a11yState ? a11yState : {},
     accessibilityLabel: testId,
+    accessibilityRole: onPress ? accessibilityRole : 'text',
   }
 
   if (isSwitchRow && decoratorProps) {
diff --git a/VAMobile/src/screens/BenefitsScreen/DisabilityRatingsScreen/DisabilityRatingsScreen.tsx b/VAMobile/src/screens/BenefitsScreen/DisabilityRatingsScreen/DisabilityRatingsScreen.tsx
index 640dd7708b1..4a7b250ab23 100644
--- a/VAMobile/src/screens/BenefitsScreen/DisabilityRatingsScreen/DisabilityRatingsScreen.tsx
+++ b/VAMobile/src/screens/BenefitsScreen/DisabilityRatingsScreen/DisabilityRatingsScreen.tsx
@@ -98,7 +98,8 @@ function DisabilityRatingsScreen() {
         backgroundColor={'list'}
         borderStyle="solid"
         borderColor="primary"
-        accessible={true}>
+        accessible={true}
+        accessibilityRole={'text'}>
         <Box mx={theme.dimensions.gutter} my={theme.dimensions.buttonPadding} flexDirection="column" accessible={false}>
           {percentageText && (
             <TextView variant={'MobileBodyBold'} testID={percentageText} accessible={false}>
diff --git a/VAMobile/src/screens/HealthScreen/Pharmacy/PrescriptionDetails/DetailsTextSections.tsx b/VAMobile/src/screens/HealthScreen/Pharmacy/PrescriptionDetails/DetailsTextSections.tsx
index 877b2f94aa5..c7095a97af9 100644
--- a/VAMobile/src/screens/HealthScreen/Pharmacy/PrescriptionDetails/DetailsTextSections.tsx
+++ b/VAMobile/src/screens/HealthScreen/Pharmacy/PrescriptionDetails/DetailsTextSections.tsx
@@ -73,11 +73,11 @@ function DetailsTextSections({
       borderTopColor={'prescriptionDivider'}
       pt={standardMarginBetween}>
       <Box {...containerProps}>
-        <Box accessible={hasRightText ? true : false}>
+        <Box accessible={hasRightText ? true : false} accessibilityRole={'text'}>
           {getTextElements(leftSectionTitle, leftSectionValue, leftSectionTitleLabel, leftSectionValueLabel)}
         </Box>
         {hasRightText && (
-          <Box alignItems="flex-end" accessible={true}>
+          <Box alignItems="flex-end" accessible={true} accessibilityRole={'text'}>
             {getTextElements(rightSectionTitle, rightSectionValue, rightSectionTitleLabel, rightSectionValueLabel)}
           </Box>
         )}
diff --git a/VAMobile/src/screens/HealthScreen/SecureMessaging/NoInboxMessages/NoInboxMessages.tsx b/VAMobile/src/screens/HealthScreen/SecureMessaging/NoInboxMessages/NoInboxMessages.tsx
index e5f72d10f5e..966605dd60b 100644
--- a/VAMobile/src/screens/HealthScreen/SecureMessaging/NoInboxMessages/NoInboxMessages.tsx
+++ b/VAMobile/src/screens/HealthScreen/SecureMessaging/NoInboxMessages/NoInboxMessages.tsx
@@ -21,7 +21,7 @@ function NoInboxMessages() {
           {t('secureMessaging.inbox.noMessages.title')}
         </TextView>
       </Box>
-      <Box accessible={true}>
+      <Box accessible={true} accessibilityRole={'text'}>
         <TextView variant="MobileBody" textAlign="center" my={theme.dimensions.standardMarginBetween}>
           {t('secureMessaging.inbox.noMessages.body')}
         </TextView>
diff --git a/VAMobile/src/screens/PaymentsScreen/DirectDepositScreen/DirectDepositScreen.tsx b/VAMobile/src/screens/PaymentsScreen/DirectDepositScreen/DirectDepositScreen.tsx
index 1970b110465..9a30e7fbfd2 100644
--- a/VAMobile/src/screens/PaymentsScreen/DirectDepositScreen/DirectDepositScreen.tsx
+++ b/VAMobile/src/screens/PaymentsScreen/DirectDepositScreen/DirectDepositScreen.tsx
@@ -116,7 +116,7 @@ function DirectDepositScreen({ navigation }: DirectDepositScreenProps) {
             </TextView>
           </Box>
           <DefaultList items={getButtonTextList()} title={t('directDeposit.information')} />
-          <Box mx={gutter} my={theme.paragraphSpacing.spacing20FontSize} accessible={true}>
+          <Box mx={gutter} my={theme.paragraphSpacing.spacing20FontSize} accessible={true} accessibilityRole={'text'}>
             <TextView>
               <TextView variant="MobileBodyBold">{t('directDeposit.bankFraudNote') + ' '}</TextView>
               <TextView variant="MobileBody">{t('directDeposit.bankFraudText')}</TextView>