From 05a2bc37371f95987ccfafdef153cc138b2c5ae4 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 7 Apr 2022 04:06:57 +0800
Subject: [PATCH 01/52] draft

---
 Libraries/Components/Button.js                |   5 +-
 .../Touchable/TouchableNativeFeedback.js      |  10 +-
 .../js/examples/Button/ButtonExample.js       | 213 ++++--------------
 3 files changed, 59 insertions(+), 169 deletions(-)

diff --git a/Libraries/Components/Button.js b/Libraries/Components/Button.js
index 89ec384fe2c1b7..1170e8766000f4 100644
--- a/Libraries/Components/Button.js
+++ b/Libraries/Components/Button.js
@@ -317,14 +317,15 @@ class Button extends React.Component<ButtonProps> {
 
     return (
       <Touchable
+        /*
         accessible={accessible}
         accessibilityActions={accessibilityActions}
         onAccessibilityAction={onAccessibilityAction}
-        accessibilityLabel={accessibilityLabel}
         accessibilityHint={accessibilityHint}
         accessibilityLanguage={accessibilityLanguage}
-        accessibilityRole="button"
         accessibilityState={accessibilityState}
+        */
+        accessibilityRole="button"
         hasTVPreferredFocus={hasTVPreferredFocus}
         nextFocusDown={nextFocusDown}
         nextFocusForward={nextFocusForward}
diff --git a/Libraries/Components/Touchable/TouchableNativeFeedback.js b/Libraries/Components/Touchable/TouchableNativeFeedback.js
index 910a51c5e30994..032c429f3db1ed 100644
--- a/Libraries/Components/Touchable/TouchableNativeFeedback.js
+++ b/Libraries/Components/Touchable/TouchableNativeFeedback.js
@@ -238,6 +238,14 @@ class TouchableNativeFeedback extends React.Component<Props, State> {
   render(): React.Node {
     const element = React.Children.only(this.props.children);
     const children = [element.props.children];
+    let title = '';
+    // if !props.accessibilityLabel
+    React.Children.map(children, child => {
+      if (!child?.props?.accessible && !child?.props?.focusable) {
+        title += ', ' + child.props.children;
+      }
+    });
+    console.log('TouchableNativeFeedback title', title);
     if (__DEV__) {
       if (element.type === View) {
         children.push(
@@ -272,7 +280,7 @@ class TouchableNativeFeedback extends React.Component<Props, State> {
         accessible: this.props.accessible !== false,
         accessibilityHint: this.props.accessibilityHint,
         accessibilityLanguage: this.props.accessibilityLanguage,
-        accessibilityLabel: this.props.accessibilityLabel,
+        accessibilityLabel: title,
         accessibilityRole: this.props.accessibilityRole,
         accessibilityState: accessibilityState,
         accessibilityActions: this.props.accessibilityActions,
diff --git a/packages/rn-tester/js/examples/Button/ButtonExample.js b/packages/rn-tester/js/examples/Button/ButtonExample.js
index dd97ad5a78f414..e7be8a10f0b655 100644
--- a/packages/rn-tester/js/examples/Button/ButtonExample.js
+++ b/packages/rn-tester/js/examples/Button/ButtonExample.js
@@ -12,7 +12,15 @@
 
 const React = require('react');
 
-const {Alert, Button, View, StyleSheet} = require('react-native');
+const {
+  Alert,
+  Button,
+  View,
+  StyleSheet,
+  Text,
+  Pressable,
+  TouchableNativeFeedback,
+} = require('react-native');
 const {RNTesterThemeContext} = require('../../components/RNTesterTheme');
 
 function onButtonPress(buttonName) {
@@ -26,180 +34,53 @@ exports.title = 'Button';
 exports.documentationURL = 'https://reactnative.dev/docs/button';
 exports.description = 'Simple React Native button component.';
 
+function TouchableExample(props) {
+  return (
+    <TouchableNativeFeedback
+      accessible={true}
+      style={{height: 200, width: 400, backgroundColor: 'red'}}
+      importantForAccessibility="yes"
+      accessibilityRole="button">
+      <View>
+        <Text
+          accessible={false}
+          focusable={false}
+          style={{
+            height: 50,
+            marginLeft: 50,
+            marginTop: 50,
+            backgroundColor: 'white',
+          }}>
+          Text number 1
+        </Text>
+        <Text
+          accessible={false}
+          focusable={false}
+          style={{
+            height: 50,
+            marginLeft: 50,
+            marginTop: 50,
+            backgroundColor: 'white',
+          }}>
+          Text number 2
+        </Text>
+      </View>
+    </TouchableNativeFeedback>
+  );
+}
+
 exports.examples = [
   {
     title: 'Button with default styling',
     render: function (): React.Node {
       return (
-        <Button
-          onPress={() => onButtonPress('submitted')}
-          testID="button_default_styling"
-          title="Submit Application"
-          accessibilityLabel="Press to submit your application!"
+        <TouchableExample
+          childText="This is the child Text"
+          childText2="This is the second child"
         />
       );
     },
   },
-  {
-    title: 'Button with color="red"',
-    description:
-      ('Note: On iOS, the color prop controls the color of the text. On ' +
-        'Android, the color adjusts the background color of the button.': string),
-    render: function (): React.Node {
-      return (
-        <RNTesterThemeContext.Consumer>
-          {theme => {
-            return (
-              <Button
-                onPress={() => onButtonPress('cancelled')}
-                testID="cancel_button"
-                color={theme.SystemRedColor}
-                title="Cancel Application"
-                accessibilityLabel="Press to cancel your application!"
-              />
-            );
-          }}
-        </RNTesterThemeContext.Consumer>
-      );
-    },
-  },
-  {
-    title: 'Two Buttons with Flexbox layout',
-    description:
-      ('Two buttons wrapped inside view with justifyContent: spaceBetween,' +
-        'This layout strategy lets the title define the width of the button': string),
-    render: function (): React.Node {
-      return (
-        <RNTesterThemeContext.Consumer>
-          {theme => {
-            return (
-              <View style={styles.container}>
-                <Button
-                  onPress={() => onButtonPress('cancelled')}
-                  testID="two_cancel_button"
-                  color={theme.SystemRedColor}
-                  title="Cancel"
-                  accessibilityLabel="Press to cancel your application!"
-                />
-                <Button
-                  onPress={() => onButtonPress('submitted')}
-                  testID="two_submit_button"
-                  color={theme.SystemGreenColor}
-                  title="Submit"
-                  accessibilityLabel="Press to submit your application!"
-                />
-              </View>
-            );
-          }}
-        </RNTesterThemeContext.Consumer>
-      );
-    },
-  },
-  {
-    title: 'Three Buttons with Flexbox layout',
-    render: function (): React.Node {
-      return (
-        <RNTesterThemeContext.Consumer>
-          {theme => {
-            return (
-              <View style={styles.container}>
-                <Button
-                  onPress={() => onButtonPress('cancelled')}
-                  testID="three_cancel_button"
-                  color={theme.SystemRedColor}
-                  title="Cancel"
-                  accessibilityLabel="Press to cancel your application!"
-                />
-                <Button
-                  onPress={() => onButtonPress('saved')}
-                  testID="three_save_button"
-                  color={theme.LinkColor}
-                  title="Save For Later"
-                  accessibilityLabel="Press to save your application!"
-                />
-                <Button
-                  onPress={() => onButtonPress('submitted')}
-                  testID="three_submit_button"
-                  color={theme.SystemGreenColor}
-                  title="Submit"
-                  accessibilityLabel="Press to submit your application!"
-                />
-              </View>
-            );
-          }}
-        </RNTesterThemeContext.Consumer>
-      );
-    },
-  },
-  {
-    title: 'Button with disabled={true}',
-    description:
-      'By passing disabled={true} all interactions for the button are disabled.',
-    render: function (): React.Node {
-      return (
-        <RNTesterThemeContext.Consumer>
-          {theme => {
-            return (
-              <Button
-                disabled
-                onPress={() => onButtonPress('submitted')}
-                color={theme.LinkColor}
-                testID="disabled_button"
-                title="Submit Application"
-                accessibilityLabel="Press to submit your application!"
-              />
-            );
-          }}
-        </RNTesterThemeContext.Consumer>
-      );
-    },
-  },
-  {
-    title: 'Button with accessibilityLabel="label"',
-    description: ('Note: This prop changes the text that a screen ' +
-      'reader announces (there are no visual differences).': string),
-    render: function (): React.Node {
-      return (
-        <RNTesterThemeContext.Consumer>
-          {theme => {
-            return (
-              <Button
-                onPress={() => onButtonPress('submitted')}
-                testID="accessibilityLabel_button"
-                color={theme.LinkColor}
-                title="Submit Application"
-                accessibilityLabel="Press to submit your application!"
-              />
-            );
-          }}
-        </RNTesterThemeContext.Consumer>
-      );
-    },
-  },
-  {
-    title: 'Button with accessibilityState={{disabled: true}}',
-    description:
-      ('Note: This prop will announce on TalkBack that the button is disabled. ' +
-        'The "disabled" prop has higher precedence on the state of the component': string),
-    render: function (): React.Node {
-      return (
-        <RNTesterThemeContext.Consumer>
-          {theme => {
-            return (
-              <Button
-                accessibilityState={{disabled: true}}
-                onPress={() => onButtonPress('submitted')}
-                testID="accessibilityState_button"
-                color={theme.LinkColor}
-                title="Submit Application"
-                accessibilityLabel="Press to submit your application!"
-              />
-            );
-          }}
-        </RNTesterThemeContext.Consumer>
-      );
-    },
-  },
 ];
 
 const styles = StyleSheet.create({

From 4ffd15dca676cf5f3869fbf753658f67fdfc43a9 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 11 Apr 2022 18:20:02 +0800
Subject: [PATCH 02/52] draft

---
 .../Touchable/TouchableNativeFeedback.js      | 12 ++++---
 .../js/examples/Button/ButtonExample.js       | 35 ++++++-------------
 2 files changed, 19 insertions(+), 28 deletions(-)

diff --git a/Libraries/Components/Touchable/TouchableNativeFeedback.js b/Libraries/Components/Touchable/TouchableNativeFeedback.js
index 032c429f3db1ed..6a4345b7b3d2b4 100644
--- a/Libraries/Components/Touchable/TouchableNativeFeedback.js
+++ b/Libraries/Components/Touchable/TouchableNativeFeedback.js
@@ -238,14 +238,18 @@ class TouchableNativeFeedback extends React.Component<Props, State> {
   render(): React.Node {
     const element = React.Children.only(this.props.children);
     const children = [element.props.children];
-    let title = '';
+    let titles = [];
     // if !props.accessibilityLabel
     React.Children.map(children, child => {
+      console.log('TouchableNativeFeedback child', child);
+      console.log('TouchableNativeFeedback child?.props', child?.props);
+      /*
       if (!child?.props?.accessible && !child?.props?.focusable) {
-        title += ', ' + child.props.children;
+        titles.push(child.props.children);
       }
+      */
     });
-    console.log('TouchableNativeFeedback title', title);
+    // const title = titles.join(', ');
     if (__DEV__) {
       if (element.type === View) {
         children.push(
@@ -280,7 +284,7 @@ class TouchableNativeFeedback extends React.Component<Props, State> {
         accessible: this.props.accessible !== false,
         accessibilityHint: this.props.accessibilityHint,
         accessibilityLanguage: this.props.accessibilityLanguage,
-        accessibilityLabel: title,
+        // accessibilityLabel: title,
         accessibilityRole: this.props.accessibilityRole,
         accessibilityState: accessibilityState,
         accessibilityActions: this.props.accessibilityActions,
diff --git a/packages/rn-tester/js/examples/Button/ButtonExample.js b/packages/rn-tester/js/examples/Button/ButtonExample.js
index e7be8a10f0b655..6e0305a0f15080 100644
--- a/packages/rn-tester/js/examples/Button/ButtonExample.js
+++ b/packages/rn-tester/js/examples/Button/ButtonExample.js
@@ -41,30 +41,17 @@ function TouchableExample(props) {
       style={{height: 200, width: 400, backgroundColor: 'red'}}
       importantForAccessibility="yes"
       accessibilityRole="button">
-      <View>
-        <Text
-          accessible={false}
-          focusable={false}
-          style={{
-            height: 50,
-            marginLeft: 50,
-            marginTop: 50,
-            backgroundColor: 'white',
-          }}>
-          Text number 1
-        </Text>
-        <Text
-          accessible={false}
-          focusable={false}
-          style={{
-            height: 50,
-            marginLeft: 50,
-            marginTop: 50,
-            backgroundColor: 'white',
-          }}>
-          Text number 2
-        </Text>
-      </View>
+      <Text
+        accessible={false}
+        focusable={false}
+        style={{
+          height: 50,
+          marginLeft: 50,
+          marginTop: 50,
+          backgroundColor: 'white',
+        }}>
+        Text number 1
+      </Text>
     </TouchableNativeFeedback>
   );
 }

From 28f86a3d74217bcb960cfd4f32cd307bbb3d1550 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 14 Apr 2022 11:07:52 +0800
Subject: [PATCH 03/52] draft

---
 .../react/uimanager/BaseViewManager.java      | 29 +++++++++++++
 .../js/examples/Button/ButtonExample.js       | 42 +++++++++++--------
 2 files changed, 54 insertions(+), 17 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java
index 3cf7db7be1b55f..630cff416afd05 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java
@@ -10,12 +10,15 @@
 import android.graphics.Color;
 import android.os.Build;
 import android.text.TextUtils;
+import android.util.Log;
 import android.view.View;
+import android.view.ViewGroup;
 import android.view.ViewParent;
 import android.view.accessibility.AccessibilityEvent;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.core.view.ViewCompat;
+import androidx.core.view.accessibility.AccessibilityNodeInfoCompat;
 import com.facebook.common.logging.FLog;
 import com.facebook.react.R;
 import com.facebook.react.bridge.Dynamic;
@@ -225,13 +228,39 @@ public void setViewState(@NonNull T view, @Nullable ReadableMap accessibilitySta
 
   private void updateViewContentDescription(@NonNull T view) {
     final String accessibilityLabel = (String) view.getTag(R.id.accessibility_label);
+    Log.w("TESTING::BaseViewManager", "accessibilityLabel: " + (accessibilityLabel));
     final ReadableMap accessibilityState = (ReadableMap) view.getTag(R.id.accessibility_state);
     final String accessibilityHint = (String) view.getTag(R.id.accessibility_hint);
     final List<String> contentDescription = new ArrayList<>();
     final ReadableMap accessibilityValue = (ReadableMap) view.getTag(R.id.accessibility_value);
     if (accessibilityLabel != null) {
       contentDescription.add(accessibilityLabel);
+      // logic to get child content
+      final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
+      if (view instanceof ViewGroup) {
+        final StringBuilder concatChildDescription = new StringBuilder();
+        final ViewGroup viewGroup = (ViewGroup) view;
+
+        for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+          final View child = viewGroup.getChildAt(i);
+
+          final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
+          ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
+
+          if (AccessibilityEvaluationUtil.isSpeakingNode(childNodeInfo, child)
+              && !AccessibilityEvaluationUtil.isAccessibilityFocusable(childNodeInfo, child)) {
+            CharSequence childNodeDescription = getTalkbackDescription(child);
+            if (!TextUtils.isEmpty(childNodeDescription)) {
+              concatChildDescription.append(childNodeDescription + delimiter);
+            }
+          }
+          childNodeInfo.recycle();
+        }
+
+        return removeFinalDelimiter(concatChildDescription);
+      }
     }
+
     if (accessibilityState != null) {
       final ReadableMapKeySetIterator i = accessibilityState.keySetIterator();
       while (i.hasNextKey()) {
diff --git a/packages/rn-tester/js/examples/Button/ButtonExample.js b/packages/rn-tester/js/examples/Button/ButtonExample.js
index 6e0305a0f15080..1c164800006f81 100644
--- a/packages/rn-tester/js/examples/Button/ButtonExample.js
+++ b/packages/rn-tester/js/examples/Button/ButtonExample.js
@@ -36,23 +36,31 @@ exports.description = 'Simple React Native button component.';
 
 function TouchableExample(props) {
   return (
-    <TouchableNativeFeedback
-      accessible={true}
-      style={{height: 200, width: 400, backgroundColor: 'red'}}
-      importantForAccessibility="yes"
-      accessibilityRole="button">
-      <Text
-        accessible={false}
-        focusable={false}
-        style={{
-          height: 50,
-          marginLeft: 50,
-          marginTop: 50,
-          backgroundColor: 'white',
-        }}>
-        Text number 1
-      </Text>
-    </TouchableNativeFeedback>
+    <>
+      <View
+        style={{backgroundColor: 'red', height: 100, width: 100}}
+        accessibilityLabel="accessibilityLabel for View"
+        focusable={true}
+        accessible={true}></View>
+      <TouchableNativeFeedback
+        accessibilityLabel="testing"
+        accessible={true}
+        style={{height: 200, width: 400, backgroundColor: 'red'}}
+        importantForAccessibility="yes"
+        accessibilityRole="button">
+        <Text
+          accessible={false}
+          focusable={false}
+          style={{
+            height: 50,
+            marginLeft: 50,
+            marginTop: 50,
+            backgroundColor: 'white',
+          }}>
+          Text number 1
+        </Text>
+      </TouchableNativeFeedback>
+    </>
   );
 }
 

From 69473de5adef4115e1897ed4e76e3b511f660b61 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 00:41:16 +0800
Subject: [PATCH 04/52] draft solution to announce content

---
 .../react/uimanager/BaseViewManager.java      |  29 --
 .../uimanager/ReactAccessibilityDelegate.java | 309 ++++++++++++++++++
 2 files changed, 309 insertions(+), 29 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java
index 630cff416afd05..3cf7db7be1b55f 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java
@@ -10,15 +10,12 @@
 import android.graphics.Color;
 import android.os.Build;
 import android.text.TextUtils;
-import android.util.Log;
 import android.view.View;
-import android.view.ViewGroup;
 import android.view.ViewParent;
 import android.view.accessibility.AccessibilityEvent;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.core.view.ViewCompat;
-import androidx.core.view.accessibility.AccessibilityNodeInfoCompat;
 import com.facebook.common.logging.FLog;
 import com.facebook.react.R;
 import com.facebook.react.bridge.Dynamic;
@@ -228,39 +225,13 @@ public void setViewState(@NonNull T view, @Nullable ReadableMap accessibilitySta
 
   private void updateViewContentDescription(@NonNull T view) {
     final String accessibilityLabel = (String) view.getTag(R.id.accessibility_label);
-    Log.w("TESTING::BaseViewManager", "accessibilityLabel: " + (accessibilityLabel));
     final ReadableMap accessibilityState = (ReadableMap) view.getTag(R.id.accessibility_state);
     final String accessibilityHint = (String) view.getTag(R.id.accessibility_hint);
     final List<String> contentDescription = new ArrayList<>();
     final ReadableMap accessibilityValue = (ReadableMap) view.getTag(R.id.accessibility_value);
     if (accessibilityLabel != null) {
       contentDescription.add(accessibilityLabel);
-      // logic to get child content
-      final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
-      if (view instanceof ViewGroup) {
-        final StringBuilder concatChildDescription = new StringBuilder();
-        final ViewGroup viewGroup = (ViewGroup) view;
-
-        for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
-          final View child = viewGroup.getChildAt(i);
-
-          final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
-          ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
-
-          if (AccessibilityEvaluationUtil.isSpeakingNode(childNodeInfo, child)
-              && !AccessibilityEvaluationUtil.isAccessibilityFocusable(childNodeInfo, child)) {
-            CharSequence childNodeDescription = getTalkbackDescription(child);
-            if (!TextUtils.isEmpty(childNodeDescription)) {
-              concatChildDescription.append(childNodeDescription + delimiter);
-            }
-          }
-          childNodeInfo.recycle();
-        }
-
-        return removeFinalDelimiter(concatChildDescription);
-      }
     }
-
     if (accessibilityState != null) {
       final ReadableMapKeySetIterator i = accessibilityState.keySetIterator();
       while (i.hasNextKey()) {
diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 7d38333ccf3074..df93e4af28f0f5 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -16,10 +16,15 @@
 import android.text.Layout;
 import android.text.Spannable;
 import android.text.Spanned;
+import android.text.TextUtils;
 import android.text.style.AbsoluteSizeSpan;
 import android.text.style.ClickableSpan;
+import android.util.Log;
 import android.view.View;
+import android.view.ViewGroup;
 import android.view.accessibility.AccessibilityEvent;
+import android.view.accessibility.AccessibilityNodeInfo;
+import android.widget.EditText;
 import android.widget.TextView;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
@@ -41,6 +46,7 @@
 import com.facebook.react.bridge.ReadableType;
 import com.facebook.react.bridge.UIManager;
 import com.facebook.react.bridge.WritableMap;
+import com.facebook.react.uimanager.ReactAccessibilityDelegate.AccessibilityRole;
 import com.facebook.react.uimanager.events.Event;
 import com.facebook.react.uimanager.events.EventDispatcher;
 import com.facebook.react.uimanager.util.ReactFindViewUtil;
@@ -59,6 +65,8 @@ public class ReactAccessibilityDelegate extends ExploreByTouchHelper {
   private static int sCounter = 0x3f000000;
   private static final int TIMEOUT_SEND_ACCESSIBILITY_EVENT = 200;
   private static final int SEND_EVENT = 1;
+  private static final String delimiter = ", ";
+  private static final int delimiterLength = delimiter.length();
 
   public static final HashMap<String, Integer> sActionIdMap = new HashMap<>();
 
@@ -122,6 +130,7 @@ public enum AccessibilityRole {
     TABLIST,
     TIMER,
     LIST,
+    GRID,
     TOOLBAR;
 
     public static String getValue(AccessibilityRole role) {
@@ -192,6 +201,46 @@ public static AccessibilityRole fromValue(@Nullable String value) {
   private static final String STATE_SELECTED = "selected";
   private static final String STATE_CHECKED = "checked";
 
+  public static boolean hasNonActionableSpeakingDescendants(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+
+    if (node == null || view == null || !(view instanceof ViewGroup)) {
+      return false;
+    }
+
+    final ViewGroup viewGroup = (ViewGroup) view;
+    for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+      final View childView = viewGroup.getChildAt(i);
+
+      if (childView == null) {
+        continue;
+      }
+
+      final AccessibilityNodeInfoCompat childNode = AccessibilityNodeInfoCompat.obtain();
+      try {
+        ViewCompat.onInitializeAccessibilityNodeInfo(childView, childNode);
+
+        if (!childNode.isVisibleToUser()) {
+          continue;
+        }
+
+        if (isAccessibilityFocusable(childNode, childView)) {
+          continue;
+        }
+
+        if (isSpeakingNode(childNode, childView)) {
+          return true;
+        }
+      } finally {
+        if (childNode != null) {
+          childNode.recycle();
+        }
+      }
+    }
+
+    return false;
+  }
+
   public ReactAccessibilityDelegate(
       final View view, boolean originalFocus, int originalImportantForAccessibility) {
     super(view);
@@ -217,9 +266,269 @@ public void handleMessage(Message msg) {
 
   @Nullable View mAccessibilityLabelledBy;
 
+  public static boolean isSpeakingNode(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+    if (node == null || view == null) {
+      return false;
+    }
+
+    final int important = ViewCompat.getImportantForAccessibility(view);
+    if (important == ViewCompat.IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS
+        || (important == ViewCompat.IMPORTANT_FOR_ACCESSIBILITY_NO && node.getChildCount() <= 0)) {
+      return false;
+    }
+
+    return node.isCheckable() || hasText(node) || hasNonActionableSpeakingDescendants(node, view);
+  }
+
+  public static boolean hasText(@Nullable AccessibilityNodeInfoCompat node) {
+    return node != null
+        && node.getCollectionInfo() == null
+        && (!TextUtils.isEmpty(node.getText()) || !TextUtils.isEmpty(node.getContentDescription()));
+  }
+
+  public static boolean isAccessibilityFocusable(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+    if (node == null || view == null) {
+      return false;
+    }
+
+    // Never focus invisible nodes.
+    if (!node.isVisibleToUser()) {
+      return false;
+    }
+
+    // Always focus "actionable" nodes.
+    if (isActionableForAccessibility(node)) {
+      return true;
+    }
+
+    // only focus top-level list items with non-actionable speaking children.
+    return isTopLevelScrollItem(node, view) && isSpeakingNode(node, view);
+  }
+
+  public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeInfoCompat node) {
+    if (node == null) {
+      return false;
+    }
+
+    if (node.isClickable() || node.isLongClickable() || node.isFocusable()) {
+      return true;
+    }
+
+    final List actionList = node.getActionList();
+    return actionList.contains(AccessibilityNodeInfoCompat.ACTION_CLICK)
+        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_LONG_CLICK)
+        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_FOCUS);
+  }
+
+  public static AccessibilityRole getRole(View view) {
+    if (view == null) {
+      return AccessibilityRole.NONE;
+    }
+    AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
+    ViewCompat.onInitializeAccessibilityNodeInfo(view, nodeInfo);
+    AccessibilityRole role = getRole(nodeInfo);
+    nodeInfo.recycle();
+    return role;
+  }
+
+  public static AccessibilityRole getRole(AccessibilityNodeInfo nodeInfo) {
+    return getRole(new AccessibilityNodeInfoCompat(nodeInfo));
+  }
+
+  public static AccessibilityRole getRole(AccessibilityNodeInfoCompat nodeInfo) {
+    AccessibilityRole role = AccessibilityRole.fromValue((String) nodeInfo.getClassName());
+    if (role.equals(AccessibilityRole.IMAGEBUTTON) || role.equals(AccessibilityRole.IMAGE)) {
+      return nodeInfo.isClickable() ? AccessibilityRole.IMAGEBUTTON : AccessibilityRole.IMAGE;
+    }
+
+    if (role.equals(AccessibilityRole.NONE)) {
+      AccessibilityNodeInfoCompat.CollectionInfoCompat collection = nodeInfo.getCollectionInfo();
+      if (collection != null) {
+        // RecyclerView will be classified as a list or grid.
+        if (collection.getRowCount() > 1 && collection.getColumnCount() > 1) {
+          return AccessibilityRole.GRID;
+        } else {
+          return AccessibilityRole.LIST;
+        }
+      }
+    }
+
+    return role;
+  }
+
+  @Nullable
+  public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
+    if (view == null) {
+      return null;
+    }
+
+    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
+
+    // For some unknown reason, Android seems to occasionally throw a NPE from
+    // onInitializeAccessibilityNodeInfo.
+    try {
+      ViewCompat.onInitializeAccessibilityNodeInfo(view, nodeInfo);
+    } catch (NullPointerException e) {
+      if (nodeInfo != null) {
+        nodeInfo.recycle();
+      }
+      return null;
+    }
+
+    return nodeInfo;
+  }
+
+  /*
+  public static String getRoleDescription(View view) {
+    AccessibilityNodeInfoCompat nodeInfo = createNodeInfoFromView(view);
+    String roleDescription = getRoleDescription(nodeInfo);
+    nodeInfo.recycle();
+
+    if (roleDescription == null || roleDescription == "") {
+      AccessibilityRole role = getRole(view);
+      roleDescription = role.getRoleString();
+    }
+
+    return roleDescription;
+  }
+  */
+
+  @Nullable
+  public static CharSequence getTalkbackDescription(View view) {
+    final AccessibilityNodeInfoCompat node = createNodeInfoFromView(view);
+    if (node == null) {
+      return null;
+    }
+    try {
+      final CharSequence contentDescription = node.getContentDescription();
+      final CharSequence nodeText = node.getText();
+      Log.w("TESTING::ReactAccessibilityDelegate", "nodeText: " + (nodeText));
+
+      final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
+      final boolean isEditText = view instanceof EditText;
+
+      StringBuilder talkbackSegments = new StringBuilder();
+      // AccessibilityRole role = getRole(view);
+      // String roleString = getRoleDescription(view);
+      // boolean disabled = isActionableForAccessibility(node) && !node.isEnabled();
+
+      // EditText's prioritize their own text content over a contentDescription so skip this
+      if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
+
+        // first prepend any status modifiers
+        // addStateSegments(talkbackSegments, node, role);
+
+        // next add content description
+        talkbackSegments.append(contentDescription + delimiter);
+
+        // then role
+        /*
+        if (roleString.length() > 0) {
+          talkbackSegments.append(roleString + delimiter);
+        }
+        */
+
+        return removeFinalDelimiter(talkbackSegments);
+      }
+
+      // EditText
+      if (hasNodeText) {
+        // skipped status checks above for EditText
+
+        // description
+        talkbackSegments.append(nodeText + delimiter);
+
+        return removeFinalDelimiter(talkbackSegments);
+      }
+
+      // If there are child views and no contentDescription the text of all non-focusable children,
+      // comma separated, becomes the description.
+      if (view instanceof ViewGroup) {
+        final StringBuilder concatChildDescription = new StringBuilder();
+        final ViewGroup viewGroup = (ViewGroup) view;
+
+        for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+          final View child = viewGroup.getChildAt(i);
+
+          final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
+          ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
+
+          if (isSpeakingNode(childNodeInfo, child)
+              && !isAccessibilityFocusable(childNodeInfo, child)) {
+            CharSequence childNodeDescription = getTalkbackDescription(child);
+            if (!TextUtils.isEmpty(childNodeDescription)) {
+              concatChildDescription.append(childNodeDescription + delimiter);
+            }
+          }
+          childNodeInfo.recycle();
+        }
+
+        return removeFinalDelimiter(concatChildDescription);
+      }
+
+      return null;
+    } finally {
+      node.recycle();
+    }
+  }
+
+  private static String removeFinalDelimiter(StringBuilder builder) {
+    int end = builder.length();
+    if (end > 0) {
+      builder.delete(end - delimiterLength, end);
+    }
+    return builder.toString();
+  }
+
+  public static boolean isTopLevelScrollItem(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+    if (node == null || view == null) {
+      return false;
+    }
+
+    final View parent = (View) ViewCompat.getParentForAccessibility(view);
+    if (parent == null) {
+      return false;
+    }
+
+    if (node.isScrollable()) {
+      return true;
+    }
+
+    final List actionList = node.getActionList();
+    if (actionList.contains(AccessibilityNodeInfoCompat.ACTION_SCROLL_FORWARD)
+        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_SCROLL_BACKWARD)) {
+      return true;
+    }
+
+    AccessibilityRole parentRole = AccessibilityRole.BUTTON;
+    return parentRole == AccessibilityRole.LIST || parentRole == AccessibilityRole.GRID;
+  }
+
   @Override
   public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCompat info) {
     super.onInitializeAccessibilityNodeInfo(host, info);
+    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
+    if (host instanceof ViewGroup) {
+      final StringBuilder concatChildDescription = new StringBuilder();
+      final ViewGroup viewGroup = (ViewGroup) host;
+      for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+        final View child = viewGroup.getChildAt(i);
+        final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
+        ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
+        if (isSpeakingNode(childNodeInfo, child)
+            && !isAccessibilityFocusable(childNodeInfo, child)) {
+          CharSequence childNodeDescription = getTalkbackDescription(child);
+          if (!TextUtils.isEmpty(childNodeDescription)) {
+            concatChildDescription.append(childNodeDescription);
+          }
+        }
+        childNodeInfo.recycle();
+      }
+      info.setContentDescription(concatChildDescription);
+    }
     final AccessibilityRole accessibilityRole =
         (AccessibilityRole) host.getTag(R.id.accessibility_role);
     if (accessibilityRole != null) {

From bdabc4c825d6464e86a600054a28b75eed2cf895 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 00:46:22 +0800
Subject: [PATCH 05/52] remove previous draft solution

---
 Libraries/Components/Button.js                     |  5 ++---
 .../Touchable/TouchableNativeFeedback.js           | 14 +-------------
 2 files changed, 3 insertions(+), 16 deletions(-)

diff --git a/Libraries/Components/Button.js b/Libraries/Components/Button.js
index 1170e8766000f4..89ec384fe2c1b7 100644
--- a/Libraries/Components/Button.js
+++ b/Libraries/Components/Button.js
@@ -317,15 +317,14 @@ class Button extends React.Component<ButtonProps> {
 
     return (
       <Touchable
-        /*
         accessible={accessible}
         accessibilityActions={accessibilityActions}
         onAccessibilityAction={onAccessibilityAction}
+        accessibilityLabel={accessibilityLabel}
         accessibilityHint={accessibilityHint}
         accessibilityLanguage={accessibilityLanguage}
-        accessibilityState={accessibilityState}
-        */
         accessibilityRole="button"
+        accessibilityState={accessibilityState}
         hasTVPreferredFocus={hasTVPreferredFocus}
         nextFocusDown={nextFocusDown}
         nextFocusForward={nextFocusForward}
diff --git a/Libraries/Components/Touchable/TouchableNativeFeedback.js b/Libraries/Components/Touchable/TouchableNativeFeedback.js
index 6a4345b7b3d2b4..910a51c5e30994 100644
--- a/Libraries/Components/Touchable/TouchableNativeFeedback.js
+++ b/Libraries/Components/Touchable/TouchableNativeFeedback.js
@@ -238,18 +238,6 @@ class TouchableNativeFeedback extends React.Component<Props, State> {
   render(): React.Node {
     const element = React.Children.only(this.props.children);
     const children = [element.props.children];
-    let titles = [];
-    // if !props.accessibilityLabel
-    React.Children.map(children, child => {
-      console.log('TouchableNativeFeedback child', child);
-      console.log('TouchableNativeFeedback child?.props', child?.props);
-      /*
-      if (!child?.props?.accessible && !child?.props?.focusable) {
-        titles.push(child.props.children);
-      }
-      */
-    });
-    // const title = titles.join(', ');
     if (__DEV__) {
       if (element.type === View) {
         children.push(
@@ -284,7 +272,7 @@ class TouchableNativeFeedback extends React.Component<Props, State> {
         accessible: this.props.accessible !== false,
         accessibilityHint: this.props.accessibilityHint,
         accessibilityLanguage: this.props.accessibilityLanguage,
-        // accessibilityLabel: title,
+        accessibilityLabel: this.props.accessibilityLabel,
         accessibilityRole: this.props.accessibilityRole,
         accessibilityState: accessibilityState,
         accessibilityActions: this.props.accessibilityActions,

From ae66001a0992285a09fd72bbc3252203451ad196 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 10:17:06 +0800
Subject: [PATCH 06/52] draft commit

---
 .../uimanager/ReactAccessibilityDelegate.java | 44 ++++++++++++-------
 .../js/examples/Button/ButtonExample.js       | 35 +++++----------
 2 files changed, 38 insertions(+), 41 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index df93e4af28f0f5..547f883348d296 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -19,7 +19,6 @@
 import android.text.TextUtils;
 import android.text.style.AbsoluteSizeSpan;
 import android.text.style.ClickableSpan;
-import android.util.Log;
 import android.view.View;
 import android.view.ViewGroup;
 import android.view.accessibility.AccessibilityEvent;
@@ -266,6 +265,14 @@ public void handleMessage(Message msg) {
 
   @Nullable View mAccessibilityLabelledBy;
 
+  /**
+   * Returns whether the supplied {@link View} and {@link AccessibilityNodeInfoCompat} would produce
+   * spoken feedback if it were accessibility focused. NOTE: not all speaking nodes are focusable.
+   *
+   * @param view The {@link View} to evaluate
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if it meets the criterion for producing spoken feedback
+   */
   public static boolean isSpeakingNode(
       @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
     if (node == null || view == null) {
@@ -287,6 +294,18 @@ public static boolean hasText(@Nullable AccessibilityNodeInfoCompat node) {
         && (!TextUtils.isEmpty(node.getText()) || !TextUtils.isEmpty(node.getContentDescription()));
   }
 
+  /**
+   * Determines if the provided {@link View} and {@link AccessibilityNodeInfoCompat} meet the
+   * criteria for gaining accessibility focus.
+   *
+   * <p>Note: this is evaluating general focusability by accessibility services, and does not mean
+   * this view will be guaranteed to be focused by specific services such as Talkback. For Talkback
+   * focusability, see {@link #isTalkbackFocusable(View)}
+   *
+   * @param view The {@link View} to evaluate
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if it is possible to gain accessibility focus
+   */
   public static boolean isAccessibilityFocusable(
       @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
     if (node == null || view == null) {
@@ -307,6 +326,14 @@ public static boolean isAccessibilityFocusable(
     return isTopLevelScrollItem(node, view) && isSpeakingNode(node, view);
   }
 
+  /**
+   * Returns whether a node is actionable. That is, the node supports one of {@link
+   * AccessibilityNodeInfoCompat#isClickable()}, {@link AccessibilityNodeInfoCompat#isFocusable()},
+   * or {@link AccessibilityNodeInfoCompat#isLongClickable()}.
+   *
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if node is actionable.
+   */
   public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeInfoCompat node) {
     if (node == null) {
       return false;
@@ -404,32 +431,17 @@ public static CharSequence getTalkbackDescription(View view) {
     try {
       final CharSequence contentDescription = node.getContentDescription();
       final CharSequence nodeText = node.getText();
-      Log.w("TESTING::ReactAccessibilityDelegate", "nodeText: " + (nodeText));
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
 
       StringBuilder talkbackSegments = new StringBuilder();
-      // AccessibilityRole role = getRole(view);
-      // String roleString = getRoleDescription(view);
-      // boolean disabled = isActionableForAccessibility(node) && !node.isEnabled();
 
       // EditText's prioritize their own text content over a contentDescription so skip this
       if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
-
-        // first prepend any status modifiers
-        // addStateSegments(talkbackSegments, node, role);
-
         // next add content description
         talkbackSegments.append(contentDescription + delimiter);
 
-        // then role
-        /*
-        if (roleString.length() > 0) {
-          talkbackSegments.append(roleString + delimiter);
-        }
-        */
-
         return removeFinalDelimiter(talkbackSegments);
       }
 
diff --git a/packages/rn-tester/js/examples/Button/ButtonExample.js b/packages/rn-tester/js/examples/Button/ButtonExample.js
index 1c164800006f81..f57976ce287a42 100644
--- a/packages/rn-tester/js/examples/Button/ButtonExample.js
+++ b/packages/rn-tester/js/examples/Button/ButtonExample.js
@@ -36,31 +36,16 @@ exports.description = 'Simple React Native button component.';
 
 function TouchableExample(props) {
   return (
-    <>
-      <View
-        style={{backgroundColor: 'red', height: 100, width: 100}}
-        accessibilityLabel="accessibilityLabel for View"
-        focusable={true}
-        accessible={true}></View>
-      <TouchableNativeFeedback
-        accessibilityLabel="testing"
-        accessible={true}
-        style={{height: 200, width: 400, backgroundColor: 'red'}}
-        importantForAccessibility="yes"
-        accessibilityRole="button">
-        <Text
-          accessible={false}
-          focusable={false}
-          style={{
-            height: 50,
-            marginLeft: 50,
-            marginTop: 50,
-            backgroundColor: 'white',
-          }}>
-          Text number 1
-        </Text>
-      </TouchableNativeFeedback>
-    </>
+    <View
+      accessible={true}
+      focusable={true}
+      style={{backgroundColor: 'red'}}
+      accessibilityRole="button">
+      <Text accessible={false}>Text number 1</Text>
+      <Text accessible={false}>
+        Text number 2<Text accessible={false}>Text number 3</Text>
+      </Text>
+    </View>
   );
 }
 

From 7c287a44d027edc7b97ba101b54e438c42b27d3e Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 17:15:12 +0800
Subject: [PATCH 07/52] remove getRole logic

---
 .../uimanager/ReactAccessibilityDelegate.java | 59 +------------------
 1 file changed, 1 insertion(+), 58 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 547f883348d296..4a9ec00d38f6f2 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -22,7 +22,6 @@
 import android.view.View;
 import android.view.ViewGroup;
 import android.view.accessibility.AccessibilityEvent;
-import android.view.accessibility.AccessibilityNodeInfo;
 import android.widget.EditText;
 import android.widget.TextView;
 import androidx.annotation.NonNull;
@@ -318,12 +317,7 @@ public static boolean isAccessibilityFocusable(
     }
 
     // Always focus "actionable" nodes.
-    if (isActionableForAccessibility(node)) {
-      return true;
-    }
-
-    // only focus top-level list items with non-actionable speaking children.
-    return isTopLevelScrollItem(node, view) && isSpeakingNode(node, view);
+    return isActionableForAccessibility(node);
   }
 
   /**
@@ -349,42 +343,6 @@ public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeIn
         || actionList.contains(AccessibilityNodeInfoCompat.ACTION_FOCUS);
   }
 
-  public static AccessibilityRole getRole(View view) {
-    if (view == null) {
-      return AccessibilityRole.NONE;
-    }
-    AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
-    ViewCompat.onInitializeAccessibilityNodeInfo(view, nodeInfo);
-    AccessibilityRole role = getRole(nodeInfo);
-    nodeInfo.recycle();
-    return role;
-  }
-
-  public static AccessibilityRole getRole(AccessibilityNodeInfo nodeInfo) {
-    return getRole(new AccessibilityNodeInfoCompat(nodeInfo));
-  }
-
-  public static AccessibilityRole getRole(AccessibilityNodeInfoCompat nodeInfo) {
-    AccessibilityRole role = AccessibilityRole.fromValue((String) nodeInfo.getClassName());
-    if (role.equals(AccessibilityRole.IMAGEBUTTON) || role.equals(AccessibilityRole.IMAGE)) {
-      return nodeInfo.isClickable() ? AccessibilityRole.IMAGEBUTTON : AccessibilityRole.IMAGE;
-    }
-
-    if (role.equals(AccessibilityRole.NONE)) {
-      AccessibilityNodeInfoCompat.CollectionInfoCompat collection = nodeInfo.getCollectionInfo();
-      if (collection != null) {
-        // RecyclerView will be classified as a list or grid.
-        if (collection.getRowCount() > 1 && collection.getColumnCount() > 1) {
-          return AccessibilityRole.GRID;
-        } else {
-          return AccessibilityRole.LIST;
-        }
-      }
-    }
-
-    return role;
-  }
-
   @Nullable
   public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
     if (view == null) {
@@ -407,21 +365,6 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
     return nodeInfo;
   }
 
-  /*
-  public static String getRoleDescription(View view) {
-    AccessibilityNodeInfoCompat nodeInfo = createNodeInfoFromView(view);
-    String roleDescription = getRoleDescription(nodeInfo);
-    nodeInfo.recycle();
-
-    if (roleDescription == null || roleDescription == "") {
-      AccessibilityRole role = getRole(view);
-      roleDescription = role.getRoleString();
-    }
-
-    return roleDescription;
-  }
-  */
-
   @Nullable
   public static CharSequence getTalkbackDescription(View view) {
     final AccessibilityNodeInfoCompat node = createNodeInfoFromView(view);

From 6d73cd4bbb9ba7b4fda0ee9192812eb62e26ab93 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 17:22:04 +0800
Subject: [PATCH 08/52] adding comments to methods

---
 .../uimanager/ReactAccessibilityDelegate.java | 61 +++++++++++--------
 1 file changed, 36 insertions(+), 25 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 4a9ec00d38f6f2..7a270160e4737d 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -199,6 +199,18 @@ public static AccessibilityRole fromValue(@Nullable String value) {
   private static final String STATE_SELECTED = "selected";
   private static final String STATE_CHECKED = "checked";
 
+  /**
+   * Determines if the supplied {@link View} and {@link AccessibilityNodeInfoCompat} has any
+   * children which are not independently accessibility focusable and also have a spoken
+   * description.
+   *
+   * <p>NOTE: Accessibility services will include these children's descriptions in the closest
+   * focusable ancestor.
+   *
+   * @param view The {@link View} to evaluate
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if it has any non-actionable speaking descendants within its subtree
+   */
   public static boolean hasNonActionableSpeakingDescendants(
       @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
 
@@ -343,6 +355,15 @@ public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeIn
         || actionList.contains(AccessibilityNodeInfoCompat.ACTION_FOCUS);
   }
 
+  /**
+   * Creates a {@link FlipperObject} of useful properties of AccessibilityNodeInfo, to be shown in
+   * the Flipper Layout Inspector accessibility extension. All properties are immutable since they
+   * are all derived from various {@link View} properties. This is a more complete list than
+   * getAccessibilityNodeInfoProperties returns.
+   *
+   * @param view The {@link View} to derive the AccessibilityNodeInfo properties from.
+   * @return {@link FlipperObject} containing the properties.
+   */
   @Nullable
   public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
     if (view == null) {
@@ -365,6 +386,21 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
     return nodeInfo;
   }
 
+  /**
+   * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
+   * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
+   * the {@code text} and {@code contentDescription} of any ancestor {@link View}.
+   *
+   * <p>This description is generally ported over from Google's TalkBack screen reader, and this
+   * should be kept up to date with their implementation (as much as necessary). Details can be seen
+   * in their source code here:
+   *
+   * <p>https://github.com/google/talkback/compositor/src/main/res/raw/compositor.json - search for
+   * "get_description_for_tree", "append_description_for_tree", "description_for_tree_nodes"
+   *
+   * @param view The {@link View} to evaluate.
+   * @return {@code String} representing what talkback will say when a {@link View} is focused.
+   */
   @Nullable
   public static CharSequence getTalkbackDescription(View view) {
     final AccessibilityNodeInfoCompat node = createNodeInfoFromView(view);
@@ -437,31 +473,6 @@ private static String removeFinalDelimiter(StringBuilder builder) {
     return builder.toString();
   }
 
-  public static boolean isTopLevelScrollItem(
-      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
-    if (node == null || view == null) {
-      return false;
-    }
-
-    final View parent = (View) ViewCompat.getParentForAccessibility(view);
-    if (parent == null) {
-      return false;
-    }
-
-    if (node.isScrollable()) {
-      return true;
-    }
-
-    final List actionList = node.getActionList();
-    if (actionList.contains(AccessibilityNodeInfoCompat.ACTION_SCROLL_FORWARD)
-        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_SCROLL_BACKWARD)) {
-      return true;
-    }
-
-    AccessibilityRole parentRole = AccessibilityRole.BUTTON;
-    return parentRole == AccessibilityRole.LIST || parentRole == AccessibilityRole.GRID;
-  }
-
   @Override
   public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCompat info) {
     super.onInitializeAccessibilityNodeInfo(host, info);

From 2a3a836b8dc4804606c784e12fd7fdb05f022ec2 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 17:24:19 +0800
Subject: [PATCH 09/52] reorganize methods alph order

---
 .../uimanager/ReactAccessibilityDelegate.java | 498 +++++++++---------
 1 file changed, 249 insertions(+), 249 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 7a270160e4737d..e7716dc17b18a3 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -199,58 +199,6 @@ public static AccessibilityRole fromValue(@Nullable String value) {
   private static final String STATE_SELECTED = "selected";
   private static final String STATE_CHECKED = "checked";
 
-  /**
-   * Determines if the supplied {@link View} and {@link AccessibilityNodeInfoCompat} has any
-   * children which are not independently accessibility focusable and also have a spoken
-   * description.
-   *
-   * <p>NOTE: Accessibility services will include these children's descriptions in the closest
-   * focusable ancestor.
-   *
-   * @param view The {@link View} to evaluate
-   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
-   * @return {@code true} if it has any non-actionable speaking descendants within its subtree
-   */
-  public static boolean hasNonActionableSpeakingDescendants(
-      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
-
-    if (node == null || view == null || !(view instanceof ViewGroup)) {
-      return false;
-    }
-
-    final ViewGroup viewGroup = (ViewGroup) view;
-    for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
-      final View childView = viewGroup.getChildAt(i);
-
-      if (childView == null) {
-        continue;
-      }
-
-      final AccessibilityNodeInfoCompat childNode = AccessibilityNodeInfoCompat.obtain();
-      try {
-        ViewCompat.onInitializeAccessibilityNodeInfo(childView, childNode);
-
-        if (!childNode.isVisibleToUser()) {
-          continue;
-        }
-
-        if (isAccessibilityFocusable(childNode, childView)) {
-          continue;
-        }
-
-        if (isSpeakingNode(childNode, childView)) {
-          return true;
-        }
-      } finally {
-        if (childNode != null) {
-          childNode.recycle();
-        }
-      }
-    }
-
-    return false;
-  }
-
   public ReactAccessibilityDelegate(
       final View view, boolean originalFocus, int originalImportantForAccessibility) {
     super(view);
@@ -276,203 +224,6 @@ public void handleMessage(Message msg) {
 
   @Nullable View mAccessibilityLabelledBy;
 
-  /**
-   * Returns whether the supplied {@link View} and {@link AccessibilityNodeInfoCompat} would produce
-   * spoken feedback if it were accessibility focused. NOTE: not all speaking nodes are focusable.
-   *
-   * @param view The {@link View} to evaluate
-   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
-   * @return {@code true} if it meets the criterion for producing spoken feedback
-   */
-  public static boolean isSpeakingNode(
-      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
-    if (node == null || view == null) {
-      return false;
-    }
-
-    final int important = ViewCompat.getImportantForAccessibility(view);
-    if (important == ViewCompat.IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS
-        || (important == ViewCompat.IMPORTANT_FOR_ACCESSIBILITY_NO && node.getChildCount() <= 0)) {
-      return false;
-    }
-
-    return node.isCheckable() || hasText(node) || hasNonActionableSpeakingDescendants(node, view);
-  }
-
-  public static boolean hasText(@Nullable AccessibilityNodeInfoCompat node) {
-    return node != null
-        && node.getCollectionInfo() == null
-        && (!TextUtils.isEmpty(node.getText()) || !TextUtils.isEmpty(node.getContentDescription()));
-  }
-
-  /**
-   * Determines if the provided {@link View} and {@link AccessibilityNodeInfoCompat} meet the
-   * criteria for gaining accessibility focus.
-   *
-   * <p>Note: this is evaluating general focusability by accessibility services, and does not mean
-   * this view will be guaranteed to be focused by specific services such as Talkback. For Talkback
-   * focusability, see {@link #isTalkbackFocusable(View)}
-   *
-   * @param view The {@link View} to evaluate
-   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
-   * @return {@code true} if it is possible to gain accessibility focus
-   */
-  public static boolean isAccessibilityFocusable(
-      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
-    if (node == null || view == null) {
-      return false;
-    }
-
-    // Never focus invisible nodes.
-    if (!node.isVisibleToUser()) {
-      return false;
-    }
-
-    // Always focus "actionable" nodes.
-    return isActionableForAccessibility(node);
-  }
-
-  /**
-   * Returns whether a node is actionable. That is, the node supports one of {@link
-   * AccessibilityNodeInfoCompat#isClickable()}, {@link AccessibilityNodeInfoCompat#isFocusable()},
-   * or {@link AccessibilityNodeInfoCompat#isLongClickable()}.
-   *
-   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
-   * @return {@code true} if node is actionable.
-   */
-  public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeInfoCompat node) {
-    if (node == null) {
-      return false;
-    }
-
-    if (node.isClickable() || node.isLongClickable() || node.isFocusable()) {
-      return true;
-    }
-
-    final List actionList = node.getActionList();
-    return actionList.contains(AccessibilityNodeInfoCompat.ACTION_CLICK)
-        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_LONG_CLICK)
-        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_FOCUS);
-  }
-
-  /**
-   * Creates a {@link FlipperObject} of useful properties of AccessibilityNodeInfo, to be shown in
-   * the Flipper Layout Inspector accessibility extension. All properties are immutable since they
-   * are all derived from various {@link View} properties. This is a more complete list than
-   * getAccessibilityNodeInfoProperties returns.
-   *
-   * @param view The {@link View} to derive the AccessibilityNodeInfo properties from.
-   * @return {@link FlipperObject} containing the properties.
-   */
-  @Nullable
-  public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
-    if (view == null) {
-      return null;
-    }
-
-    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
-
-    // For some unknown reason, Android seems to occasionally throw a NPE from
-    // onInitializeAccessibilityNodeInfo.
-    try {
-      ViewCompat.onInitializeAccessibilityNodeInfo(view, nodeInfo);
-    } catch (NullPointerException e) {
-      if (nodeInfo != null) {
-        nodeInfo.recycle();
-      }
-      return null;
-    }
-
-    return nodeInfo;
-  }
-
-  /**
-   * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
-   * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
-   * the {@code text} and {@code contentDescription} of any ancestor {@link View}.
-   *
-   * <p>This description is generally ported over from Google's TalkBack screen reader, and this
-   * should be kept up to date with their implementation (as much as necessary). Details can be seen
-   * in their source code here:
-   *
-   * <p>https://github.com/google/talkback/compositor/src/main/res/raw/compositor.json - search for
-   * "get_description_for_tree", "append_description_for_tree", "description_for_tree_nodes"
-   *
-   * @param view The {@link View} to evaluate.
-   * @return {@code String} representing what talkback will say when a {@link View} is focused.
-   */
-  @Nullable
-  public static CharSequence getTalkbackDescription(View view) {
-    final AccessibilityNodeInfoCompat node = createNodeInfoFromView(view);
-    if (node == null) {
-      return null;
-    }
-    try {
-      final CharSequence contentDescription = node.getContentDescription();
-      final CharSequence nodeText = node.getText();
-
-      final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
-      final boolean isEditText = view instanceof EditText;
-
-      StringBuilder talkbackSegments = new StringBuilder();
-
-      // EditText's prioritize their own text content over a contentDescription so skip this
-      if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
-        // next add content description
-        talkbackSegments.append(contentDescription + delimiter);
-
-        return removeFinalDelimiter(talkbackSegments);
-      }
-
-      // EditText
-      if (hasNodeText) {
-        // skipped status checks above for EditText
-
-        // description
-        talkbackSegments.append(nodeText + delimiter);
-
-        return removeFinalDelimiter(talkbackSegments);
-      }
-
-      // If there are child views and no contentDescription the text of all non-focusable children,
-      // comma separated, becomes the description.
-      if (view instanceof ViewGroup) {
-        final StringBuilder concatChildDescription = new StringBuilder();
-        final ViewGroup viewGroup = (ViewGroup) view;
-
-        for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
-          final View child = viewGroup.getChildAt(i);
-
-          final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
-          ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
-
-          if (isSpeakingNode(childNodeInfo, child)
-              && !isAccessibilityFocusable(childNodeInfo, child)) {
-            CharSequence childNodeDescription = getTalkbackDescription(child);
-            if (!TextUtils.isEmpty(childNodeDescription)) {
-              concatChildDescription.append(childNodeDescription + delimiter);
-            }
-          }
-          childNodeInfo.recycle();
-        }
-
-        return removeFinalDelimiter(concatChildDescription);
-      }
-
-      return null;
-    } finally {
-      node.recycle();
-    }
-  }
-
-  private static String removeFinalDelimiter(StringBuilder builder) {
-    int end = builder.length();
-    if (end > 0) {
-      builder.delete(end - delimiterLength, end);
-    }
-    return builder.toString();
-  }
-
   @Override
   public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCompat info) {
     super.onInitializeAccessibilityNodeInfo(host, info);
@@ -976,4 +727,253 @@ private static class AccessibleLink {
 
     return null;
   }
+
+  /**
+   * Determines if the supplied {@link View} and {@link AccessibilityNodeInfoCompat} has any
+   * children which are not independently accessibility focusable and also have a spoken
+   * description.
+   *
+   * <p>NOTE: Accessibility services will include these children's descriptions in the closest
+   * focusable ancestor.
+   *
+   * @param view The {@link View} to evaluate
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if it has any non-actionable speaking descendants within its subtree
+   */
+  public static boolean hasNonActionableSpeakingDescendants(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+
+    if (node == null || view == null || !(view instanceof ViewGroup)) {
+      return false;
+    }
+
+    final ViewGroup viewGroup = (ViewGroup) view;
+    for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+      final View childView = viewGroup.getChildAt(i);
+
+      if (childView == null) {
+        continue;
+      }
+
+      final AccessibilityNodeInfoCompat childNode = AccessibilityNodeInfoCompat.obtain();
+      try {
+        ViewCompat.onInitializeAccessibilityNodeInfo(childView, childNode);
+
+        if (!childNode.isVisibleToUser()) {
+          continue;
+        }
+
+        if (isAccessibilityFocusable(childNode, childView)) {
+          continue;
+        }
+
+        if (isSpeakingNode(childNode, childView)) {
+          return true;
+        }
+      } finally {
+        if (childNode != null) {
+          childNode.recycle();
+        }
+      }
+    }
+
+    return false;
+  }
+
+  /**
+   * Returns whether the supplied {@link View} and {@link AccessibilityNodeInfoCompat} would produce
+   * spoken feedback if it were accessibility focused. NOTE: not all speaking nodes are focusable.
+   *
+   * @param view The {@link View} to evaluate
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if it meets the criterion for producing spoken feedback
+   */
+  public static boolean isSpeakingNode(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+    if (node == null || view == null) {
+      return false;
+    }
+
+    final int important = ViewCompat.getImportantForAccessibility(view);
+    if (important == ViewCompat.IMPORTANT_FOR_ACCESSIBILITY_NO_HIDE_DESCENDANTS
+        || (important == ViewCompat.IMPORTANT_FOR_ACCESSIBILITY_NO && node.getChildCount() <= 0)) {
+      return false;
+    }
+
+    return node.isCheckable() || hasText(node) || hasNonActionableSpeakingDescendants(node, view);
+  }
+
+  public static boolean hasText(@Nullable AccessibilityNodeInfoCompat node) {
+    return node != null
+        && node.getCollectionInfo() == null
+        && (!TextUtils.isEmpty(node.getText()) || !TextUtils.isEmpty(node.getContentDescription()));
+  }
+
+  /**
+   * Determines if the provided {@link View} and {@link AccessibilityNodeInfoCompat} meet the
+   * criteria for gaining accessibility focus.
+   *
+   * <p>Note: this is evaluating general focusability by accessibility services, and does not mean
+   * this view will be guaranteed to be focused by specific services such as Talkback. For Talkback
+   * focusability, see {@link #isTalkbackFocusable(View)}
+   *
+   * @param view The {@link View} to evaluate
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if it is possible to gain accessibility focus
+   */
+  public static boolean isAccessibilityFocusable(
+      @Nullable AccessibilityNodeInfoCompat node, @Nullable View view) {
+    if (node == null || view == null) {
+      return false;
+    }
+
+    // Never focus invisible nodes.
+    if (!node.isVisibleToUser()) {
+      return false;
+    }
+
+    // Always focus "actionable" nodes.
+    return isActionableForAccessibility(node);
+  }
+
+  /**
+   * Returns whether a node is actionable. That is, the node supports one of {@link
+   * AccessibilityNodeInfoCompat#isClickable()}, {@link AccessibilityNodeInfoCompat#isFocusable()},
+   * or {@link AccessibilityNodeInfoCompat#isLongClickable()}.
+   *
+   * @param node The {@link AccessibilityNodeInfoCompat} to evaluate
+   * @return {@code true} if node is actionable.
+   */
+  public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeInfoCompat node) {
+    if (node == null) {
+      return false;
+    }
+
+    if (node.isClickable() || node.isLongClickable() || node.isFocusable()) {
+      return true;
+    }
+
+    final List actionList = node.getActionList();
+    return actionList.contains(AccessibilityNodeInfoCompat.ACTION_CLICK)
+        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_LONG_CLICK)
+        || actionList.contains(AccessibilityNodeInfoCompat.ACTION_FOCUS);
+  }
+
+  /**
+   * Creates a {@link FlipperObject} of useful properties of AccessibilityNodeInfo, to be shown in
+   * the Flipper Layout Inspector accessibility extension. All properties are immutable since they
+   * are all derived from various {@link View} properties. This is a more complete list than
+   * getAccessibilityNodeInfoProperties returns.
+   *
+   * @param view The {@link View} to derive the AccessibilityNodeInfo properties from.
+   * @return {@link FlipperObject} containing the properties.
+   */
+  @Nullable
+  public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
+    if (view == null) {
+      return null;
+    }
+
+    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
+
+    // For some unknown reason, Android seems to occasionally throw a NPE from
+    // onInitializeAccessibilityNodeInfo.
+    try {
+      ViewCompat.onInitializeAccessibilityNodeInfo(view, nodeInfo);
+    } catch (NullPointerException e) {
+      if (nodeInfo != null) {
+        nodeInfo.recycle();
+      }
+      return null;
+    }
+
+    return nodeInfo;
+  }
+
+  /**
+   * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
+   * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
+   * the {@code text} and {@code contentDescription} of any ancestor {@link View}.
+   *
+   * <p>This description is generally ported over from Google's TalkBack screen reader, and this
+   * should be kept up to date with their implementation (as much as necessary). Details can be seen
+   * in their source code here:
+   *
+   * <p>https://github.com/google/talkback/compositor/src/main/res/raw/compositor.json - search for
+   * "get_description_for_tree", "append_description_for_tree", "description_for_tree_nodes"
+   *
+   * @param view The {@link View} to evaluate.
+   * @return {@code String} representing what talkback will say when a {@link View} is focused.
+   */
+  @Nullable
+  public static CharSequence getTalkbackDescription(View view) {
+    final AccessibilityNodeInfoCompat node = createNodeInfoFromView(view);
+    if (node == null) {
+      return null;
+    }
+    try {
+      final CharSequence contentDescription = node.getContentDescription();
+      final CharSequence nodeText = node.getText();
+
+      final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
+      final boolean isEditText = view instanceof EditText;
+
+      StringBuilder talkbackSegments = new StringBuilder();
+
+      // EditText's prioritize their own text content over a contentDescription so skip this
+      if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
+        // next add content description
+        talkbackSegments.append(contentDescription + delimiter);
+
+        return removeFinalDelimiter(talkbackSegments);
+      }
+
+      // EditText
+      if (hasNodeText) {
+        // skipped status checks above for EditText
+
+        // description
+        talkbackSegments.append(nodeText + delimiter);
+
+        return removeFinalDelimiter(talkbackSegments);
+      }
+
+      // If there are child views and no contentDescription the text of all non-focusable children,
+      // comma separated, becomes the description.
+      if (view instanceof ViewGroup) {
+        final StringBuilder concatChildDescription = new StringBuilder();
+        final ViewGroup viewGroup = (ViewGroup) view;
+
+        for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+          final View child = viewGroup.getChildAt(i);
+
+          final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
+          ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
+
+          if (isSpeakingNode(childNodeInfo, child)
+              && !isAccessibilityFocusable(childNodeInfo, child)) {
+            CharSequence childNodeDescription = getTalkbackDescription(child);
+            if (!TextUtils.isEmpty(childNodeDescription)) {
+              concatChildDescription.append(childNodeDescription + delimiter);
+            }
+          }
+          childNodeInfo.recycle();
+        }
+
+        return removeFinalDelimiter(concatChildDescription);
+      }
+
+      return null;
+    } finally {
+      node.recycle();
+    }
+  }
+
+  private static String removeFinalDelimiter(StringBuilder builder) {
+    int end = builder.length();
+    if (end > 0) {
+      builder.delete(end - delimiterLength, end);
+    }
+    return builder.toString();
+  }
 }

From 982fb205b86259203ae40ac6fd6320bd00c9f147 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 22 Apr 2022 18:56:58 +0800
Subject: [PATCH 10/52] revert changes to ButtonExample

---
 .../js/examples/Button/ButtonExample.js       | 193 +++++++++++++++---
 1 file changed, 166 insertions(+), 27 deletions(-)

diff --git a/packages/rn-tester/js/examples/Button/ButtonExample.js b/packages/rn-tester/js/examples/Button/ButtonExample.js
index f57976ce287a42..dd97ad5a78f414 100644
--- a/packages/rn-tester/js/examples/Button/ButtonExample.js
+++ b/packages/rn-tester/js/examples/Button/ButtonExample.js
@@ -12,15 +12,7 @@
 
 const React = require('react');
 
-const {
-  Alert,
-  Button,
-  View,
-  StyleSheet,
-  Text,
-  Pressable,
-  TouchableNativeFeedback,
-} = require('react-native');
+const {Alert, Button, View, StyleSheet} = require('react-native');
 const {RNTesterThemeContext} = require('../../components/RNTesterTheme');
 
 function onButtonPress(buttonName) {
@@ -34,33 +26,180 @@ exports.title = 'Button';
 exports.documentationURL = 'https://reactnative.dev/docs/button';
 exports.description = 'Simple React Native button component.';
 
-function TouchableExample(props) {
-  return (
-    <View
-      accessible={true}
-      focusable={true}
-      style={{backgroundColor: 'red'}}
-      accessibilityRole="button">
-      <Text accessible={false}>Text number 1</Text>
-      <Text accessible={false}>
-        Text number 2<Text accessible={false}>Text number 3</Text>
-      </Text>
-    </View>
-  );
-}
-
 exports.examples = [
   {
     title: 'Button with default styling',
     render: function (): React.Node {
       return (
-        <TouchableExample
-          childText="This is the child Text"
-          childText2="This is the second child"
+        <Button
+          onPress={() => onButtonPress('submitted')}
+          testID="button_default_styling"
+          title="Submit Application"
+          accessibilityLabel="Press to submit your application!"
         />
       );
     },
   },
+  {
+    title: 'Button with color="red"',
+    description:
+      ('Note: On iOS, the color prop controls the color of the text. On ' +
+        'Android, the color adjusts the background color of the button.': string),
+    render: function (): React.Node {
+      return (
+        <RNTesterThemeContext.Consumer>
+          {theme => {
+            return (
+              <Button
+                onPress={() => onButtonPress('cancelled')}
+                testID="cancel_button"
+                color={theme.SystemRedColor}
+                title="Cancel Application"
+                accessibilityLabel="Press to cancel your application!"
+              />
+            );
+          }}
+        </RNTesterThemeContext.Consumer>
+      );
+    },
+  },
+  {
+    title: 'Two Buttons with Flexbox layout',
+    description:
+      ('Two buttons wrapped inside view with justifyContent: spaceBetween,' +
+        'This layout strategy lets the title define the width of the button': string),
+    render: function (): React.Node {
+      return (
+        <RNTesterThemeContext.Consumer>
+          {theme => {
+            return (
+              <View style={styles.container}>
+                <Button
+                  onPress={() => onButtonPress('cancelled')}
+                  testID="two_cancel_button"
+                  color={theme.SystemRedColor}
+                  title="Cancel"
+                  accessibilityLabel="Press to cancel your application!"
+                />
+                <Button
+                  onPress={() => onButtonPress('submitted')}
+                  testID="two_submit_button"
+                  color={theme.SystemGreenColor}
+                  title="Submit"
+                  accessibilityLabel="Press to submit your application!"
+                />
+              </View>
+            );
+          }}
+        </RNTesterThemeContext.Consumer>
+      );
+    },
+  },
+  {
+    title: 'Three Buttons with Flexbox layout',
+    render: function (): React.Node {
+      return (
+        <RNTesterThemeContext.Consumer>
+          {theme => {
+            return (
+              <View style={styles.container}>
+                <Button
+                  onPress={() => onButtonPress('cancelled')}
+                  testID="three_cancel_button"
+                  color={theme.SystemRedColor}
+                  title="Cancel"
+                  accessibilityLabel="Press to cancel your application!"
+                />
+                <Button
+                  onPress={() => onButtonPress('saved')}
+                  testID="three_save_button"
+                  color={theme.LinkColor}
+                  title="Save For Later"
+                  accessibilityLabel="Press to save your application!"
+                />
+                <Button
+                  onPress={() => onButtonPress('submitted')}
+                  testID="three_submit_button"
+                  color={theme.SystemGreenColor}
+                  title="Submit"
+                  accessibilityLabel="Press to submit your application!"
+                />
+              </View>
+            );
+          }}
+        </RNTesterThemeContext.Consumer>
+      );
+    },
+  },
+  {
+    title: 'Button with disabled={true}',
+    description:
+      'By passing disabled={true} all interactions for the button are disabled.',
+    render: function (): React.Node {
+      return (
+        <RNTesterThemeContext.Consumer>
+          {theme => {
+            return (
+              <Button
+                disabled
+                onPress={() => onButtonPress('submitted')}
+                color={theme.LinkColor}
+                testID="disabled_button"
+                title="Submit Application"
+                accessibilityLabel="Press to submit your application!"
+              />
+            );
+          }}
+        </RNTesterThemeContext.Consumer>
+      );
+    },
+  },
+  {
+    title: 'Button with accessibilityLabel="label"',
+    description: ('Note: This prop changes the text that a screen ' +
+      'reader announces (there are no visual differences).': string),
+    render: function (): React.Node {
+      return (
+        <RNTesterThemeContext.Consumer>
+          {theme => {
+            return (
+              <Button
+                onPress={() => onButtonPress('submitted')}
+                testID="accessibilityLabel_button"
+                color={theme.LinkColor}
+                title="Submit Application"
+                accessibilityLabel="Press to submit your application!"
+              />
+            );
+          }}
+        </RNTesterThemeContext.Consumer>
+      );
+    },
+  },
+  {
+    title: 'Button with accessibilityState={{disabled: true}}',
+    description:
+      ('Note: This prop will announce on TalkBack that the button is disabled. ' +
+        'The "disabled" prop has higher precedence on the state of the component': string),
+    render: function (): React.Node {
+      return (
+        <RNTesterThemeContext.Consumer>
+          {theme => {
+            return (
+              <Button
+                accessibilityState={{disabled: true}}
+                onPress={() => onButtonPress('submitted')}
+                testID="accessibilityState_button"
+                color={theme.LinkColor}
+                title="Submit Application"
+                accessibilityLabel="Press to submit your application!"
+              />
+            );
+          }}
+        </RNTesterThemeContext.Consumer>
+      );
+    },
+  },
 ];
 
 const styles = StyleSheet.create({

From e0e162921c183e4203fd64c35b2f1a29c6e1ef4f Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 25 Apr 2022 18:51:30 +0800
Subject: [PATCH 11/52] Conditions that should trigger the custom
 contentDescription

https://github.com/facebook/react-native/pull/33690#issuecomment-1106779940

The custom contentDescription should be triggered if:
- an element has no contentDescription **AND** text
- **AND** has other content to announce (role, state, etc.)
---
 .../uimanager/ReactAccessibilityDelegate.java | 47 +++++++++++--------
 1 file changed, 28 insertions(+), 19 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index e7716dc17b18a3..2ce0b5fd446273 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -227,25 +227,6 @@ public void handleMessage(Message msg) {
   @Override
   public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCompat info) {
     super.onInitializeAccessibilityNodeInfo(host, info);
-    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
-    if (host instanceof ViewGroup) {
-      final StringBuilder concatChildDescription = new StringBuilder();
-      final ViewGroup viewGroup = (ViewGroup) host;
-      for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
-        final View child = viewGroup.getChildAt(i);
-        final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
-        ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
-        if (isSpeakingNode(childNodeInfo, child)
-            && !isAccessibilityFocusable(childNodeInfo, child)) {
-          CharSequence childNodeDescription = getTalkbackDescription(child);
-          if (!TextUtils.isEmpty(childNodeDescription)) {
-            concatChildDescription.append(childNodeDescription);
-          }
-        }
-        childNodeInfo.recycle();
-      }
-      info.setContentDescription(concatChildDescription);
-    }
     final AccessibilityRole accessibilityRole =
         (AccessibilityRole) host.getTag(R.id.accessibility_role);
     if (accessibilityRole != null) {
@@ -321,6 +302,34 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     if (testId != null) {
       info.setViewIdResourceName(testId);
     }
+    String accessibilityLabel = (String) host.getTag(R.id.accessibility_label);
+    boolean missingTextOrDescription =
+        info.getText() == null && info.getContentDescription() == null;
+    boolean hasContentToAnnounce =
+        accessibilityActions != null
+            || accessibilityState != null
+            || accessibilityLabelledBy != null
+            || accessibilityRole != null
+            || accessibilityLabel != null;
+    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
+    if (missingTextOrDescription && hasContentToAnnounce && host instanceof ViewGroup) {
+      final StringBuilder concatChildDescription = new StringBuilder();
+      final ViewGroup viewGroup = (ViewGroup) host;
+      for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
+        final View child = viewGroup.getChildAt(i);
+        final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
+        ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
+        if (isSpeakingNode(childNodeInfo, child)
+            && !isAccessibilityFocusable(childNodeInfo, child)) {
+          CharSequence childNodeDescription = getTalkbackDescription(child);
+          if (!TextUtils.isEmpty(childNodeDescription)) {
+            concatChildDescription.append(childNodeDescription);
+          }
+        }
+        childNodeInfo.recycle();
+      }
+      info.setContentDescription(concatChildDescription);
+    }
   }
 
   @Override

From a22c435d0c60182bdac6d2aa75b6eb5e3f730915 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 25 Apr 2022 19:23:23 +0800
Subject: [PATCH 12/52] remove check on accessibilityLabel

the check on the contentDescription is enough also for the accessibilityLabel
https://github.com/fabriziobertoglio1987/react-native/blob/e0e162921c183e4203fd64c35b2f1a29c6e1ef4f/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java#L233
---
 .../facebook/react/uimanager/ReactAccessibilityDelegate.java  | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 2ce0b5fd446273..bb40c0ec6ef10b 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -302,15 +302,13 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     if (testId != null) {
       info.setViewIdResourceName(testId);
     }
-    String accessibilityLabel = (String) host.getTag(R.id.accessibility_label);
     boolean missingTextOrDescription =
         info.getText() == null && info.getContentDescription() == null;
     boolean hasContentToAnnounce =
         accessibilityActions != null
             || accessibilityState != null
             || accessibilityLabelledBy != null
-            || accessibilityRole != null
-            || accessibilityLabel != null;
+            || accessibilityRole != null;
     final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
     if (missingTextOrDescription && hasContentToAnnounce && host instanceof ViewGroup) {
       final StringBuilder concatChildDescription = new StringBuilder();

From 07bac604544afd9e0f02cd7c3e0bf759a6f07953 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 26 Apr 2022 23:35:25 +0800
Subject: [PATCH 13/52] remove duplicate logic and add condition

---
 .../uimanager/ReactAccessibilityDelegate.java | 31 ++++++-------------
 1 file changed, 9 insertions(+), 22 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index bb40c0ec6ef10b..4ab0fc1ed3cbf3 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -309,24 +309,8 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
             || accessibilityState != null
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
-    final AccessibilityNodeInfoCompat nodeInfo = AccessibilityNodeInfoCompat.obtain();
-    if (missingTextOrDescription && hasContentToAnnounce && host instanceof ViewGroup) {
-      final StringBuilder concatChildDescription = new StringBuilder();
-      final ViewGroup viewGroup = (ViewGroup) host;
-      for (int i = 0, count = viewGroup.getChildCount(); i < count; i++) {
-        final View child = viewGroup.getChildAt(i);
-        final AccessibilityNodeInfoCompat childNodeInfo = AccessibilityNodeInfoCompat.obtain();
-        ViewCompat.onInitializeAccessibilityNodeInfo(child, childNodeInfo);
-        if (isSpeakingNode(childNodeInfo, child)
-            && !isAccessibilityFocusable(childNodeInfo, child)) {
-          CharSequence childNodeDescription = getTalkbackDescription(child);
-          if (!TextUtils.isEmpty(childNodeDescription)) {
-            concatChildDescription.append(childNodeDescription);
-          }
-        }
-        childNodeInfo.recycle();
-      }
-      info.setContentDescription(concatChildDescription);
+    if (missingTextOrDescription && hasContentToAnnounce) {
+      info.setContentDescription(getTalkbackDescription(host, info));
     }
   }
 
@@ -913,8 +897,9 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
    * @return {@code String} representing what talkback will say when a {@link View} is focused.
    */
   @Nullable
-  public static CharSequence getTalkbackDescription(View view) {
-    final AccessibilityNodeInfoCompat node = createNodeInfoFromView(view);
+  public static CharSequence getTalkbackDescription(
+      View view, @Nullable AccessibilityNodeInfoCompat info) {
+    final AccessibilityNodeInfoCompat node = info == null ? createNodeInfoFromView(view) : info;
     if (node == null) {
       return null;
     }
@@ -959,7 +944,7 @@ public static CharSequence getTalkbackDescription(View view) {
 
           if (isSpeakingNode(childNodeInfo, child)
               && !isAccessibilityFocusable(childNodeInfo, child)) {
-            CharSequence childNodeDescription = getTalkbackDescription(child);
+            CharSequence childNodeDescription = getTalkbackDescription(child, null);
             if (!TextUtils.isEmpty(childNodeDescription)) {
               concatChildDescription.append(childNodeDescription + delimiter);
             }
@@ -972,7 +957,9 @@ public static CharSequence getTalkbackDescription(View view) {
 
       return null;
     } finally {
-      node.recycle();
+      if (info == null) {
+        node.recycle();
+      }
     }
   }
 

From 821e62397198876350e598d43139eeb211bf5b7f Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 26 Apr 2022 23:44:32 +0800
Subject: [PATCH 14/52] avoid setContentDescription if talkback returns null

---
 .../facebook/react/uimanager/ReactAccessibilityDelegate.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 4ab0fc1ed3cbf3..4ea1de8f89ff7d 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -310,7 +310,8 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
     if (missingTextOrDescription && hasContentToAnnounce) {
-      info.setContentDescription(getTalkbackDescription(host, info));
+      CharSequence description = getTalkbackDescription(host, info);
+      if (description != null) info.setContentDescription(description);
     }
   }
 
@@ -894,6 +895,7 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
    * "get_description_for_tree", "append_description_for_tree", "description_for_tree_nodes"
    *
    * @param view The {@link View} to evaluate.
+   * @param info The default {@link AccessibilityNodeInfoCompat}.
    * @return {@code String} representing what talkback will say when a {@link View} is focused.
    */
   @Nullable

From 35190c0e31977c2e86b49b0b067b2d1c7ea0af15 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 26 Apr 2022 23:52:32 +0800
Subject: [PATCH 15/52] adding GRID role to RADelegate

---
 .../facebook/react/uimanager/ReactAccessibilityDelegate.java    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 4ea1de8f89ff7d..8668e63f510129 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -159,6 +159,8 @@ public static String getValue(AccessibilityRole role) {
           return "android.widget.Switch";
         case LIST:
           return "android.widget.AbsListView";
+        case GRID:
+          return "android.widget.GridView";
         case NONE:
         case LINK:
         case SUMMARY:

From 986b10b58b907db109b3efbb420b42f9a2a2297c Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 27 Apr 2022 11:22:03 +0800
Subject: [PATCH 16/52] adding TouchableNativeFeedback with non-acc child
 example

---
 .../examples/Accessibility/AccessibilityExample.js  | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 1cabf06e9f1e0e..31252d700ada3d 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -21,6 +21,7 @@ const {
   Text,
   View,
   TouchableOpacity,
+  TouchableNativeFeedback,
   TouchableWithoutFeedback,
   Alert,
   StyleSheet,
@@ -212,6 +213,18 @@ class AccessibilityExample extends React.Component<{}> {
             />
           </View>
         </RNTesterBlock>
+
+        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts">
+          <TouchableNativeFeedback
+            accessible={true}
+            importantForAccessibility="yes"
+            accessibilityRole="button">
+            <View accessible={false}>
+              <Text accessible={false}>Text number 1</Text>
+              <Text accessible={false}>Text number 2</Text>
+            </View>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
       </View>
     );
   }

From 28ed277b4aca0ec72c9732464a0fbbb9046377ae Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 27 Apr 2022 11:59:21 +0800
Subject: [PATCH 17/52] adding TouchableNativeFeedback with non-acc child
 second example

---
 .../examples/Accessibility/AccessibilityExample.js | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 31252d700ada3d..4cb47022547c07 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -225,6 +225,20 @@ class AccessibilityExample extends React.Component<{}> {
             </View>
           </TouchableNativeFeedback>
         </RNTesterBlock>
+
+        <RNTesterBlock title="Button with non-accessible child Texts">
+          <TouchableNativeFeedback
+            accessible={true}
+            importantForAccessibility="yes"
+            accessibilityRole="button">
+            <View>
+              <Text accessible={false}>Text number 1</Text>
+              <Text accessible={false}>
+                Text number 2<Text accessible={false}>Text number 3</Text>
+              </Text>
+            </View>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
       </View>
     );
   }

From e1521cf94c783e50aa9dfa1d48ab549980bd3018 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 27 Apr 2022 14:12:26 +0800
Subject: [PATCH 18/52] adding @Nullable to description variable

---
 .../facebook/react/uimanager/ReactAccessibilityDelegate.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 8668e63f510129..4006815279db61 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -312,7 +312,7 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
     if (missingTextOrDescription && hasContentToAnnounce) {
-      CharSequence description = getTalkbackDescription(host, info);
+      @Nullable CharSequence description = getTalkbackDescription(host, info);
       if (description != null) info.setContentDescription(description);
     }
   }

From 90dd4a885dcd54da1016fbe6698f4144c0d6a8f8 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 27 Apr 2022 14:12:47 +0800
Subject: [PATCH 19/52] adding accessibilityAction to Text example

---
 .../Accessibility/AccessibilityExample.js      | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 4cb47022547c07..af58fe14278eeb 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -226,11 +226,25 @@ class AccessibilityExample extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="Button with non-accessible child Texts">
+        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts">
           <TouchableNativeFeedback
             accessible={true}
             importantForAccessibility="yes"
-            accessibilityRole="button">
+            accessibilityRole="button"
+            accessibilityActions={[
+              {name: 'activate', label: 'activate label'},
+              {name: 'copy', label: 'copy label'},
+            ]}
+            onAccessibilityAction={event => {
+              switch (event.nativeEvent.actionName) {
+                case 'activate':
+                  Alert.alert('Alert', 'Activate accessiblity action');
+                  break;
+                case 'copy':
+                  Alert.alert('Alert', 'copy action success');
+                  break;
+              }
+            }}>
             <View>
               <Text accessible={false}>Text number 1</Text>
               <Text accessible={false}>

From a05b912d9dab1a341e8d6ec234f27394cc7f0f2d Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 27 Apr 2022 18:35:40 +0800
Subject: [PATCH 20/52] Returns a cached instance if such is available or a new
 one is create.

The returned instance is initialized from the given <code>info</code>.
---
 .../react/uimanager/ReactAccessibilityDelegate.java      | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 4006815279db61..e331b7472dc801 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -854,10 +854,7 @@ public static boolean isActionableForAccessibility(@Nullable AccessibilityNodeIn
   }
 
   /**
-   * Creates a {@link FlipperObject} of useful properties of AccessibilityNodeInfo, to be shown in
-   * the Flipper Layout Inspector accessibility extension. All properties are immutable since they
-   * are all derived from various {@link View} properties. This is a more complete list than
-   * getAccessibilityNodeInfoProperties returns.
+   * Returns a cached instance if such is available otherwise a new one.
    *
    * @param view The {@link View} to derive the AccessibilityNodeInfo properties from.
    * @return {@link FlipperObject} containing the properties.
@@ -903,7 +900,9 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
   @Nullable
   public static CharSequence getTalkbackDescription(
       View view, @Nullable AccessibilityNodeInfoCompat info) {
-    final AccessibilityNodeInfoCompat node = info == null ? createNodeInfoFromView(view) : info;
+    final AccessibilityNodeInfoCompat node =
+        info == null ? createNodeInfoFromView(view) : AccessibilityNodeInfoCompat.obtain(info);
+
     if (node == null) {
       return null;
     }

From 64bb48a709291695fad35f3170db64e6a69739a4 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 28 Apr 2022 16:16:21 +0800
Subject: [PATCH 21/52] remove not required logic

- node.recycle does not trigger runtime as we use
  obtain(AccessibilityNodeInfo info)
- setContentDescription(null) will use the default content description
  (Button, Text number 1, Text number 2) which is the existing behaviour
---
 .../react/uimanager/ReactAccessibilityDelegate.java        | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index e331b7472dc801..1ff85d9bb602c1 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -312,8 +312,7 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
     if (missingTextOrDescription && hasContentToAnnounce) {
-      @Nullable CharSequence description = getTalkbackDescription(host, info);
-      if (description != null) info.setContentDescription(description);
+      info.setContentDescription(getTalkbackDescription(host, info));
     }
   }
 
@@ -960,9 +959,7 @@ public static CharSequence getTalkbackDescription(
 
       return null;
     } finally {
-      if (info == null) {
-        node.recycle();
-      }
+      node.recycle();
     }
   }
 

From 0b620e0019fbe0452acde6021a5b5d2f7141b57f Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 24 May 2022 20:31:11 +0800
Subject: [PATCH 22/52] adding check on stateDescription in isSpeakingNode

https://github.com/facebook/react-native/pull/33690#discussion_r865137017
---
 .../uimanager/ReactAccessibilityDelegate.java | 43 ++++++++++++++++++-
 1 file changed, 42 insertions(+), 1 deletion(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 1ff85d9bb602c1..64af46d9a35948 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -773,6 +773,44 @@ public static boolean hasNonActionableSpeakingDescendants(
     return false;
   }
 
+  /**
+   * Returns whether the node has valid RangeInfo.
+   *
+   * @param node The node to check.
+   * @return Whether the node has valid RangeInfo.
+   */
+  public static boolean hasValidRangeInfo(@Nullable AccessibilityNodeInfoCompat node) {
+    if (node == null) {
+      return false;
+    }
+
+    @Nullable final RangeInfoCompat rangeInfo = node.getRangeInfo();
+    if (rangeInfo == null) {
+      return false;
+    }
+
+    final float maxProgress = rangeInfo.getMax();
+    final float minProgress = rangeInfo.getMin();
+    final float currentProgress = rangeInfo.getCurrent();
+    final float diffProgress = maxProgress - minProgress;
+    return (diffProgress > 0.0f)
+        && (currentProgress >= minProgress)
+        && (currentProgress <= maxProgress);
+  }
+
+  /**
+   * Returns whether the specified node has state description.
+   *
+   * @param node The node to check.
+   * @return {@code true} if the node has state description.
+   */
+  private static boolean hasStateDescription(@Nullable AccessibilityNodeInfoCompat node) {
+    return node != null
+        && (!TextUtils.isEmpty(node.getStateDescription())
+            || node.isCheckable()
+            || hasValidRangeInfo(node));
+  }
+
   /**
    * Returns whether the supplied {@link View} and {@link AccessibilityNodeInfoCompat} would produce
    * spoken feedback if it were accessibility focused. NOTE: not all speaking nodes are focusable.
@@ -793,7 +831,10 @@ public static boolean isSpeakingNode(
       return false;
     }
 
-    return node.isCheckable() || hasText(node) || hasNonActionableSpeakingDescendants(node, view);
+    return hasText(node)
+        || hasStateDescription(node)
+        || node.isCheckable()
+        || hasNonActionableSpeakingDescendants(node, view);
   }
 
   public static boolean hasText(@Nullable AccessibilityNodeInfoCompat node) {

From 45e30402030e370ba2f8a045fef2346e13de2859 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 26 May 2022 17:20:58 +0800
Subject: [PATCH 23/52] check for isScreenReaderFocusable and hasHint

adding check on hintText in hasText method
the "hint text" for EditText's also counts as content, as it's still announced when the EditText has no text or contentDescription.
See Talkbacks implementation here for reference:
https://github.com/google/talkback/blob/6c0b475b7f52469e309e51bfcc13de58f18176ff/utils/src/main/java/com/google/android/accessibility/utils/AccessibilityNodeInfoUtils.java#L1663
https://github.com/facebook/react-native/pull/33690#discussion_r865140866

adding check for isScreenReaderFocusable in isAccessibilityFocusable
See Talkback's implementation here for reference:
https://github.com/google/talkback/blob/6c0b475b7f52469e309e51bfcc13de58f18176ff/utils/src/main/java/com/google/android/accessibility/utils/AccessibilityNodeInfoUtils.java#L1690
https://github.com/facebook/react-native/pull/33690#discussion_r865138542
---
 .../react/uimanager/ReactAccessibilityDelegate.java         | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 64af46d9a35948..759d202a15d9db 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -840,7 +840,9 @@ public static boolean isSpeakingNode(
   public static boolean hasText(@Nullable AccessibilityNodeInfoCompat node) {
     return node != null
         && node.getCollectionInfo() == null
-        && (!TextUtils.isEmpty(node.getText()) || !TextUtils.isEmpty(node.getContentDescription()));
+        && (!TextUtils.isEmpty(node.getText())
+            || !TextUtils.isEmpty(node.getContentDescription())
+            || !TextUtils.isEmpty(node.getHintText()));
   }
 
   /**
@@ -867,7 +869,7 @@ public static boolean isAccessibilityFocusable(
     }
 
     // Always focus "actionable" nodes.
-    return isActionableForAccessibility(node);
+    return node.isScreenReaderFocusable() || isActionableForAccessibility(node);
   }
 
   /**

From 5488ababbf37b71b5ca9892a3c379d92507be5c1 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 26 May 2022 19:39:12 +0800
Subject: [PATCH 24/52] avoid adding/removing final delimiter

---
 .../react/uimanager/ReactAccessibilityDelegate.java    | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 759d202a15d9db..651670e8359f73 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -960,9 +960,8 @@ public static CharSequence getTalkbackDescription(
       // EditText's prioritize their own text content over a contentDescription so skip this
       if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
         // next add content description
-        talkbackSegments.append(contentDescription + delimiter);
-
-        return removeFinalDelimiter(talkbackSegments);
+        talkbackSegments.append(contentDescription);
+        return talkbackSegments;
       }
 
       // EditText
@@ -970,9 +969,8 @@ public static CharSequence getTalkbackDescription(
         // skipped status checks above for EditText
 
         // description
-        talkbackSegments.append(nodeText + delimiter);
-
-        return removeFinalDelimiter(talkbackSegments);
+        talkbackSegments.append(nodeText);
+        return talkbackSegments;
       }
 
       // If there are child views and no contentDescription the text of all non-focusable children,

From 751b83ef012efda82d92b67d62912942cb16ad4d Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 26 May 2022 20:19:01 +0800
Subject: [PATCH 25/52] test child with accessibilityValue, state, label

The regression is caused by the missing logic in
https://github.com/facebook/flipper/blob/e44cad5e9912b359378a43ec47c5a49bbca85fc3/android/src/main/java/com/facebook/flipper/plugins/inspector/descriptors/utils/AccessibilityUtil.java#L477-L491

Also needs to verify the following cases with EditText
https://github.com/facebook/flipper/blob/e44cad5e9912b359378a43ec47c5a49bbca85fc3/android/src/main/java/com/facebook/flipper/plugins/inspector/descriptors/utils/AccessibilityUtil.java#L500-L516
---
 .../Accessibility/AccessibilityExample.js      | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 3fe7b55081edef..e7739c2599e2b2 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -253,6 +253,24 @@ class AccessibilityExample extends React.Component<{}> {
             </View>
           </TouchableNativeFeedback>
         </RNTesterBlock>
+
+        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts, one of them have the accessibilityLabel (contentDescription)">
+          <TouchableNativeFeedback
+            accessible={true}
+            importantForAccessibility="yes"
+            accessibilityRole="button">
+            <View>
+              <Text accessible={false}>Text number 1</Text>
+              <Text
+                accessible={false}
+                accessibilityLabel="this is my label"
+                accessibilityState={{disabled: true}}
+                accessibilityValue={{text: 'this is the accessibilityValue'}}>
+                Text number 2<Text accessible={false}>Text number 3</Text>
+              </Text>
+            </View>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
       </View>
     );
   }

From e3edefb75a7b4fddca48502ebf75af7c01756125 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 26 May 2022 20:50:15 +0800
Subject: [PATCH 26/52] add addStateSegments to custom announcement

The previous developed functionality would remove the following
announcements from the child nodes:
- state
- disabled
- accessibilityRole (for the child)

The fields were not retrieved/added to the new Talkback announcement
Fixes regression tested in example
https://github.com/fabriziobertoglio1987/react-native/commit/751b83ef012efda82d92b67d62912942cb16ad4d

the current announcement have been added, the other will follow up in
the next commits:
- state
---
 .../uimanager/ReactAccessibilityDelegate.java | 57 ++++++++++++++++++-
 1 file changed, 56 insertions(+), 1 deletion(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 651670e8359f73..9cb1b9d38cf027 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -923,6 +923,46 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
     return nodeInfo;
   }
 
+  private static boolean supportsAction(AccessibilityNodeInfoCompat node, int action) {
+    if (node != null) {
+      final int supportedActions = node.getActions();
+
+      if ((supportedActions & action) == action) {
+        return true;
+      }
+    }
+
+    return false;
+  }
+
+  /**
+   * Adds the state segments of Talkback's response to a given list. This should be kept up to date
+   * as much as necessary. Details can be seen in the source code here :
+   *
+   * <p>https://github.com/google/talkback/compositor/src/main/res/raw/compositor.json - search for
+   * "description_for_tree_status", "get_switch_state"
+   *
+   * <p>https://github.com/google/talkback/compositor/src/main/res/values/strings.xml
+   */
+  private static void addStateSegments(
+      StringBuilder talkbackSegments, AccessibilityNodeInfoCompat node) {
+    // selected status is always prepended
+    if (node.isSelected()) {
+      talkbackSegments.append("selected" + delimiter);
+    }
+
+    // documented in reactnative accessibilityState
+    // https://reactnative.dev/docs/accessibility#accessibilitystate
+    // next check collapse/expand/checked status
+    if (supportsAction(node, AccessibilityNodeInfoCompat.ACTION_EXPAND)) {
+      talkbackSegments.append("collapsed" + delimiter);
+    }
+
+    if (supportsAction(node, AccessibilityNodeInfoCompat.ACTION_COLLAPSE)) {
+      talkbackSegments.append("expanded" + delimiter);
+    }
+  }
+
   /**
    * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
    * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
@@ -954,14 +994,29 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
+      boolean disabled = isActionableForAccessibility(node) && !node.isEnabled();
 
       StringBuilder talkbackSegments = new StringBuilder();
 
       // EditText's prioritize their own text content over a contentDescription so skip this
       if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
+
+        // first prepend any status modifiers
+        addStateSegments(talkbackSegments, node);
+
         // next add content description
         talkbackSegments.append(contentDescription);
-        return talkbackSegments;
+        // then role
+        //
+        //
+        //
+
+        // lastly disabled is appended if applicable
+        if (disabled) {
+          talkbackSegments.append("disabled" + delimiter);
+        }
+
+        return removeFinalDelimiter(talkbackSegments);
       }
 
       // EditText

From a0fb221d27250009bb181d052250ad8bf6114cdf Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 26 May 2022 20:55:09 +0800
Subject: [PATCH 27/52] adding accRole to example child component

---
 .../js/examples/Accessibility/AccessibilityExample.js          | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index e7739c2599e2b2..253263ac023710 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -264,8 +264,9 @@ class AccessibilityExample extends React.Component<{}> {
               <Text
                 accessible={false}
                 accessibilityLabel="this is my label"
+                accessibilityRole="text"
                 accessibilityState={{disabled: true}}
-                accessibilityValue={{text: 'this is the accessibilityValue'}}>
+                accessibilityValue={{text: 'this is the accessibility value'}}>
                 Text number 2<Text accessible={false}>Text number 3</Text>
               </Text>
             </View>

From d2c8bfb82e0835fa255ce6c667e45793e6ade120 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 27 May 2022 17:11:07 +0800
Subject: [PATCH 28/52] roleDescription and disabled attribute in node child

---
 .../uimanager/ReactAccessibilityDelegate.java | 14 ++++++++-----
 .../Accessibility/AccessibilityExample.js     | 21 ++++++++++++-------
 2 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 9cb1b9d38cf027..bb65f1dce2b100 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -994,7 +994,10 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
-      boolean disabled = isActionableForAccessibility(node) && !node.isEnabled();
+      CharSequence roleDescription = node.getRoleDescription();
+      // The original flipper implementation would check isActionableForAccessibility
+      // The check was removed for this reason https://bit.ly/3wPnmPE
+      boolean disabled = !node.isEnabled();
 
       StringBuilder talkbackSegments = new StringBuilder();
 
@@ -1005,11 +1008,12 @@ public static CharSequence getTalkbackDescription(
         addStateSegments(talkbackSegments, node);
 
         // next add content description
-        talkbackSegments.append(contentDescription);
+        talkbackSegments.append(contentDescription + delimiter);
         // then role
-        //
-        //
-        //
+        if (roleDescription != null) {
+          String roleString = roleDescription.toString();
+          talkbackSegments.append(roleString + delimiter);
+        }
 
         // lastly disabled is appended if applicable
         if (disabled) {
diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 253263ac023710..4350c60375c975 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -255,22 +255,29 @@ class AccessibilityExample extends React.Component<{}> {
         </RNTesterBlock>
 
         <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts, one of them have the accessibilityLabel (contentDescription)">
-          <TouchableNativeFeedback
+          <View
+            style={{height: 100, width: 100, backgroundColor: 'blue'}}
             accessible={true}
-            importantForAccessibility="yes"
             accessibilityRole="button">
             <View>
               <Text accessible={false}>Text number 1</Text>
-              <Text
+              <TouchableNativeFeedback
+                style={{
+                  height: 100,
+                  width: 100,
+                  backgroundColor: 'yellow',
+                }}
+                focusable={true}
+                onPress={() => console.warn('onPress child')}
                 accessible={false}
                 accessibilityLabel="this is my label"
-                accessibilityRole="text"
+                accessibilityRole="image"
                 accessibilityState={{disabled: true}}
                 accessibilityValue={{text: 'this is the accessibility value'}}>
-                Text number 2<Text accessible={false}>Text number 3</Text>
-              </Text>
+                <Text accessible={false}>Text number 3</Text>
+              </TouchableNativeFeedback>
             </View>
-          </TouchableNativeFeedback>
+          </View>
         </RNTesterBlock>
       </View>
     );

From 9d72a70c3b27d8fb198dd32a1cd8c031e458d089 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 27 May 2022 18:57:40 +0800
Subject: [PATCH 29/52] adding test case for child EditText node

---
 .../Accessibility/AccessibilityExample.js        | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 4350c60375c975..b1fa53b4a69902 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -279,6 +279,22 @@ class AccessibilityExample extends React.Component<{}> {
             </View>
           </View>
         </RNTesterBlock>
+
+        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts, one of them is an EditText">
+          <TouchableNativeFeedback
+            accessible={true}
+            importantForAccessibility="yes"
+            accessibilityRole="button">
+            <TextInput
+              accessible={false}
+              style={styles.default}
+              placeholder="this is the placeholder"
+              accessibilityRole="text"
+              secureTextEntry={true}>
+              <Text>Don't read this</Text>
+            </TextInput>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
       </View>
     );
   }

From 9d1bf23028ef056ae0aba4283e727b2b4e239f52 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 27 May 2022 19:05:58 +0800
Subject: [PATCH 30/52] EditText child component

---
 .../uimanager/ReactAccessibilityDelegate.java | 21 +++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index bb65f1dce2b100..29292c7e16ea2e 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -1026,10 +1026,27 @@ public static CharSequence getTalkbackDescription(
       // EditText
       if (hasNodeText) {
         // skipped status checks above for EditText
+        //
+        // password
+        if (node.isPassword()) {
+          talkbackSegments.append("password" + delimiter);
+        }
 
         // description
-        talkbackSegments.append(nodeText);
-        return talkbackSegments;
+        talkbackSegments.append(nodeText + delimiter);
+
+        // role
+        if (roleDescription != null) {
+          String roleString = roleDescription.toString();
+          talkbackSegments.append(roleString + delimiter);
+        }
+
+        // disabled
+        if (disabled) {
+          talkbackSegments.append("disabled" + delimiter);
+        }
+
+        return removeFinalDelimiter(talkbackSegments);
       }
 
       // If there are child views and no contentDescription the text of all non-focusable children,

From abed89ec86affa80f41a03eaf14e0cf41f1f1dbf Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 30 May 2022 17:51:45 +0800
Subject: [PATCH 31/52] minimum repr ex
 https://github.com/facebook/react-native/pull/33690/commits/9d72a70c3b27d8fb198dd32a1cd8c031e458d089

---
 .../js/examples/Accessibility/AccessibilityExample.js        | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index b1fa53b4a69902..a8737f7d383410 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -289,10 +289,7 @@ class AccessibilityExample extends React.Component<{}> {
               accessible={false}
               style={styles.default}
               placeholder="this is the placeholder"
-              accessibilityRole="text"
-              secureTextEntry={true}>
-              <Text>Don't read this</Text>
-            </TextInput>
+            />
           </TouchableNativeFeedback>
         </RNTesterBlock>
       </View>

From b90fff49bb4e8352bd353199c6ee88e0fbdc5a3c Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 1 Jun 2022 13:57:33 +0800
Subject: [PATCH 32/52] check that text/description are not empty

---
 .../react/uimanager/ReactAccessibilityDelegate.java        | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 29292c7e16ea2e..2e12bab184b17e 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -304,8 +304,11 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     if (testId != null) {
       info.setViewIdResourceName(testId);
     }
-    boolean missingTextOrDescription =
-        info.getText() == null && info.getContentDescription() == null;
+    // need to check that string length is not == 0
+    boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
+    boolean missingText = TextUtils.isEmpty(info.getText());
+    boolean missingTextOrDescription = missingContentDescription && missingText;
+    // add check on placeholder
     boolean hasContentToAnnounce =
         accessibilityActions != null
             || accessibilityState != null

From f1852f1e58165301342939eb10b09a9d1c148b03 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 1 Jun 2022 14:18:27 +0800
Subject: [PATCH 33/52] updating TextInput example with value different from
 placeholder

---
 .../rn-tester/js/examples/Accessibility/AccessibilityExample.js  | 1 +
 1 file changed, 1 insertion(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index a8737f7d383410..e4f966a10a67fc 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -286,6 +286,7 @@ class AccessibilityExample extends React.Component<{}> {
             importantForAccessibility="yes"
             accessibilityRole="button">
             <TextInput
+              value="this is the value"
               accessible={false}
               style={styles.default}
               placeholder="this is the placeholder"

From 2dc702efda8389b0d22ed379fb86fd37a5586048 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 1 Jun 2022 18:37:42 +0800
Subject: [PATCH 34/52] fix circleci tests

https://app.circleci.com/pipelines/github/facebook/react-native/13665/workflows/efdceb92-2020-4ca4-8317-f3889b0b7800/jobs/258498

```
Cannot create `TouchableNativeFeedback` element because property `style` is missing in `Props` [1] but exists in
props [2]. [prop-missing]

   packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js:264:16
   264|               <TouchableNativeFeedback
                       ^^^^^^^^^^^^^^^^^^^^^^^

References:
   Libraries/Components/Touchable/TouchableNativeFeedback.js:97:55
    97| class TouchableNativeFeedback extends React.Component<Props, State> {
                                                              ^^^^^ [1]
   packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js:264:15
                      v-----------------------
   264|               <TouchableNativeFeedback
   265|                 style={{
   266|                   height: 100,
   267|                   width: 100,
   268|                   backgroundColor: 'yellow',
   269|                 }}
   270|                 focusable={true}
   271|                 onPress={() => console.warn('onPress child')}
   272|                 accessible={false}
   273|                 accessibilityLabel="this is my label"
   274|                 accessibilityRole="image"
   275|                 accessibilityState={{disabled: true}}
   276|                 accessibilityValue={{text: 'this is the accessibility value'}}>
   277|                 <Text accessible={false}>Text number 3</Text>
   278|               </TouchableNativeFeedback>
                      -------------------------^ [2]

Found 1 error
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Failed to run flow.
Finished.
/home/circleci/react-native/scripts/run-ci-javascript-tests.js:47
    throw Error(exitCode);
    ^

Error: 1
    at Object.<anonymous> (/home/circleci/react-native/scripts/run-ci-javascript-tests.js:47:11)
    at Module._compile (node:internal/modules/cjs/loader:1103:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1157:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
    at node:internal/main/run_main_module:17:47

Exited with code exit status 1
```
---
 .../js/examples/Accessibility/AccessibilityExample.js  | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index e4f966a10a67fc..62c060c3a4eea8 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -255,18 +255,10 @@ class AccessibilityExample extends React.Component<{}> {
         </RNTesterBlock>
 
         <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts, one of them have the accessibilityLabel (contentDescription)">
-          <View
-            style={{height: 100, width: 100, backgroundColor: 'blue'}}
-            accessible={true}
-            accessibilityRole="button">
+          <View accessible={true} accessibilityRole="button">
             <View>
               <Text accessible={false}>Text number 1</Text>
               <TouchableNativeFeedback
-                style={{
-                  height: 100,
-                  width: 100,
-                  backgroundColor: 'yellow',
-                }}
                 focusable={true}
                 onPress={() => console.warn('onPress child')}
                 accessible={false}

From 8622c130bcf670396a45ac42e9e8a99bb4d29b9f Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 1 Jun 2022 20:30:44 +0800
Subject: [PATCH 35/52] moving getRoleDescription to another branch

https://github.com/facebook/react-native/commit/388911e4e7b27a627def6e276562018e7e400c4f
---
 .../react/uimanager/ReactAccessibilityDelegate.java  | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 2e12bab184b17e..044c1f37e9a248 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -997,7 +997,6 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
-      CharSequence roleDescription = node.getRoleDescription();
       // The original flipper implementation would check isActionableForAccessibility
       // The check was removed for this reason https://bit.ly/3wPnmPE
       boolean disabled = !node.isEnabled();
@@ -1012,11 +1011,6 @@ public static CharSequence getTalkbackDescription(
 
         // next add content description
         talkbackSegments.append(contentDescription + delimiter);
-        // then role
-        if (roleDescription != null) {
-          String roleString = roleDescription.toString();
-          talkbackSegments.append(roleString + delimiter);
-        }
 
         // lastly disabled is appended if applicable
         if (disabled) {
@@ -1038,12 +1032,6 @@ public static CharSequence getTalkbackDescription(
         // description
         talkbackSegments.append(nodeText + delimiter);
 
-        // role
-        if (roleDescription != null) {
-          String roleString = roleDescription.toString();
-          talkbackSegments.append(roleString + delimiter);
-        }
-
         // disabled
         if (disabled) {
           talkbackSegments.append("disabled" + delimiter);

From e4b31c3142dd820806a0cb842ace118a387b835c Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 1 Jun 2022 21:41:27 +0800
Subject: [PATCH 36/52] using hasText instead of adding checks

---
 .../react/uimanager/ReactAccessibilityDelegate.java        | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 044c1f37e9a248..ca4a871f90d1e3 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -304,17 +304,12 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     if (testId != null) {
       info.setViewIdResourceName(testId);
     }
-    // need to check that string length is not == 0
-    boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
-    boolean missingText = TextUtils.isEmpty(info.getText());
-    boolean missingTextOrDescription = missingContentDescription && missingText;
-    // add check on placeholder
     boolean hasContentToAnnounce =
         accessibilityActions != null
             || accessibilityState != null
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
-    if (missingTextOrDescription && hasContentToAnnounce) {
+    if (!hasText(info) && hasContentToAnnounce) {
       info.setContentDescription(getTalkbackDescription(host, info));
     }
   }

From 251df2763ca72d6bc319e2e2d069f58a64e96570 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Thu, 2 Jun 2022 15:02:29 +0800
Subject: [PATCH 37/52] Revert "moving getRoleDescription to another branch"

This reverts commit 8622c130bcf670396a45ac42e9e8a99bb4d29b9f.
---
 .../react/uimanager/ReactAccessibilityDelegate.java  | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index ca4a871f90d1e3..0b4cb23078e45f 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -992,6 +992,7 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
+      CharSequence roleDescription = node.getRoleDescription();
       // The original flipper implementation would check isActionableForAccessibility
       // The check was removed for this reason https://bit.ly/3wPnmPE
       boolean disabled = !node.isEnabled();
@@ -1006,6 +1007,11 @@ public static CharSequence getTalkbackDescription(
 
         // next add content description
         talkbackSegments.append(contentDescription + delimiter);
+        // then role
+        if (roleDescription != null) {
+          String roleString = roleDescription.toString();
+          talkbackSegments.append(roleString + delimiter);
+        }
 
         // lastly disabled is appended if applicable
         if (disabled) {
@@ -1027,6 +1033,12 @@ public static CharSequence getTalkbackDescription(
         // description
         talkbackSegments.append(nodeText + delimiter);
 
+        // role
+        if (roleDescription != null) {
+          String roleString = roleDescription.toString();
+          talkbackSegments.append(roleString + delimiter);
+        }
+
         // disabled
         if (disabled) {
           talkbackSegments.append("disabled" + delimiter);

From 8aa5b251c3c3e2b4e226c15daa5477374947a72d Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 3 Jun 2022 14:47:31 +0800
Subject: [PATCH 38/52] adding child accessibilityRole

---
 .../uimanager/ReactAccessibilityDelegate.java | 41 ++++++++++++++-----
 .../Accessibility/AccessibilityExample.js     |  6 ++-
 2 files changed, 35 insertions(+), 12 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 0b4cb23078e45f..3329c32484d856 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -191,6 +191,15 @@ public static AccessibilityRole fromValue(@Nullable String value) {
       }
       throw new IllegalArgumentException("Invalid accessibility role value: " + value);
     }
+
+    public static @Nullable String fromAnyValue(@Nullable String value) {
+      for (AccessibilityRole role : AccessibilityRole.values()) {
+        if (getValue(role).equalsIgnoreCase(value)) {
+          return role.toString();
+        }
+      }
+      return null;
+    }
   }
 
   private final HashMap<Integer, String> mAccessibilityActionsMap;
@@ -309,7 +318,10 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
             || accessibilityState != null
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
-    if (!hasText(info) && hasContentToAnnounce) {
+    boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
+    boolean missingText = TextUtils.isEmpty(info.getText());
+    boolean missingTextOrDescription = missingContentDescription && missingText;
+    if (missingContentDescription && hasContentToAnnounce) {
       info.setContentDescription(getTalkbackDescription(host, info));
     }
   }
@@ -961,6 +973,20 @@ private static void addStateSegments(
     }
   }
 
+  public static @Nullable String getRoleDescription(AccessibilityNodeInfoCompat node) {
+    CharSequence role = node.getRoleDescription();
+    String roleDescription;
+
+    if (role == null || role == "") {
+      roleDescription =
+          ReactAccessibilityDelegate.AccessibilityRole.fromAnyValue((String) node.getClassName());
+    } else {
+      roleDescription = role.toString();
+    }
+
+    return roleDescription;
+  }
+
   /**
    * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
    * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
@@ -992,7 +1018,7 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
-      CharSequence roleDescription = node.getRoleDescription();
+      String role = getRoleDescription(node);
       // The original flipper implementation would check isActionableForAccessibility
       // The check was removed for this reason https://bit.ly/3wPnmPE
       boolean disabled = !node.isEnabled();
@@ -1007,11 +1033,6 @@ public static CharSequence getTalkbackDescription(
 
         // next add content description
         talkbackSegments.append(contentDescription + delimiter);
-        // then role
-        if (roleDescription != null) {
-          String roleString = roleDescription.toString();
-          talkbackSegments.append(roleString + delimiter);
-        }
 
         // lastly disabled is appended if applicable
         if (disabled) {
@@ -1034,9 +1055,9 @@ public static CharSequence getTalkbackDescription(
         talkbackSegments.append(nodeText + delimiter);
 
         // role
-        if (roleDescription != null) {
-          String roleString = roleDescription.toString();
-          talkbackSegments.append(roleString + delimiter);
+        if (role != null) {
+          String roleDescription = role.toString();
+          talkbackSegments.append(roleDescription + delimiter);
         }
 
         // disabled
diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 62c060c3a4eea8..c543b3c0d26704 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -214,13 +214,15 @@ class AccessibilityExample extends React.Component<{}> {
           </View>
         </RNTesterBlock>
 
-        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts">
+        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts - basic example">
           <TouchableNativeFeedback
             accessible={true}
             importantForAccessibility="yes"
             accessibilityRole="button">
             <View accessible={false}>
-              <Text accessible={false}>Text number 1</Text>
+              <Text accessibilityRole="image" accessible={false}>
+                Text number 1 with a role
+              </Text>
               <Text accessible={false}>Text number 2</Text>
             </View>
           </TouchableNativeFeedback>

From b804bd036950763300a49b6a5a348a9394280d36 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 3 Jun 2022 14:51:33 +0800
Subject: [PATCH 39/52] removing accessibilityRole functionality as not
 required from specs

if the child has a role defined, it should probably also be accessible on its own,
  and therefore won't get pulled up into its parents announcement in the first place.
https://github.com/facebook/react-native/issues/30719#issuecomment-1143887578
---
 .../uimanager/ReactAccessibilityDelegate.java | 30 -------------------
 1 file changed, 30 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 3329c32484d856..12e18d2592207c 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -191,15 +191,6 @@ public static AccessibilityRole fromValue(@Nullable String value) {
       }
       throw new IllegalArgumentException("Invalid accessibility role value: " + value);
     }
-
-    public static @Nullable String fromAnyValue(@Nullable String value) {
-      for (AccessibilityRole role : AccessibilityRole.values()) {
-        if (getValue(role).equalsIgnoreCase(value)) {
-          return role.toString();
-        }
-      }
-      return null;
-    }
   }
 
   private final HashMap<Integer, String> mAccessibilityActionsMap;
@@ -973,20 +964,6 @@ private static void addStateSegments(
     }
   }
 
-  public static @Nullable String getRoleDescription(AccessibilityNodeInfoCompat node) {
-    CharSequence role = node.getRoleDescription();
-    String roleDescription;
-
-    if (role == null || role == "") {
-      roleDescription =
-          ReactAccessibilityDelegate.AccessibilityRole.fromAnyValue((String) node.getClassName());
-    } else {
-      roleDescription = role.toString();
-    }
-
-    return roleDescription;
-  }
-
   /**
    * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
    * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
@@ -1018,7 +995,6 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
-      String role = getRoleDescription(node);
       // The original flipper implementation would check isActionableForAccessibility
       // The check was removed for this reason https://bit.ly/3wPnmPE
       boolean disabled = !node.isEnabled();
@@ -1054,12 +1030,6 @@ public static CharSequence getTalkbackDescription(
         // description
         talkbackSegments.append(nodeText + delimiter);
 
-        // role
-        if (role != null) {
-          String roleDescription = role.toString();
-          talkbackSegments.append(roleDescription + delimiter);
-        }
-
         // disabled
         if (disabled) {
           talkbackSegments.append("disabled" + delimiter);

From f77a4a226edfb5eb1461bfc6b5d0fb15c54e62a9 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 3 Jun 2022 18:06:00 +0800
Subject: [PATCH 40/52] add check on missingTextOrDescription

---
 .../facebook/react/uimanager/ReactAccessibilityDelegate.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index 12e18d2592207c..ca143a708fd676 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -312,7 +312,7 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
     boolean missingText = TextUtils.isEmpty(info.getText());
     boolean missingTextOrDescription = missingContentDescription && missingText;
-    if (missingContentDescription && hasContentToAnnounce) {
+    if (missingTextOrDescription && hasContentToAnnounce) {
       info.setContentDescription(getTalkbackDescription(host, info));
     }
   }

From 44122ff095bef19fbabbcf5617fddad9a0f8df04 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Fri, 3 Jun 2022 19:27:15 +0800
Subject: [PATCH 41/52] Moving not required logic to a separate branch

based on my personal notes from the last meeting and the below comment
https://github.com/facebook/react-native/issues/30719#issuecomment-1143887578
---
 .../uimanager/ReactAccessibilityDelegate.java | 78 ++-----------------
 1 file changed, 7 insertions(+), 71 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index ca143a708fd676..a9f680d26c87f0 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -304,14 +304,14 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     if (testId != null) {
       info.setViewIdResourceName(testId);
     }
+    boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
+    boolean missingText = TextUtils.isEmpty(info.getText());
+    boolean missingTextOrDescription = missingContentDescription && missingText;
     boolean hasContentToAnnounce =
         accessibilityActions != null
             || accessibilityState != null
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
-    boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
-    boolean missingText = TextUtils.isEmpty(info.getText());
-    boolean missingTextOrDescription = missingContentDescription && missingText;
     if (missingTextOrDescription && hasContentToAnnounce) {
       info.setContentDescription(getTalkbackDescription(host, info));
     }
@@ -924,46 +924,6 @@ public static AccessibilityNodeInfoCompat createNodeInfoFromView(View view) {
     return nodeInfo;
   }
 
-  private static boolean supportsAction(AccessibilityNodeInfoCompat node, int action) {
-    if (node != null) {
-      final int supportedActions = node.getActions();
-
-      if ((supportedActions & action) == action) {
-        return true;
-      }
-    }
-
-    return false;
-  }
-
-  /**
-   * Adds the state segments of Talkback's response to a given list. This should be kept up to date
-   * as much as necessary. Details can be seen in the source code here :
-   *
-   * <p>https://github.com/google/talkback/compositor/src/main/res/raw/compositor.json - search for
-   * "description_for_tree_status", "get_switch_state"
-   *
-   * <p>https://github.com/google/talkback/compositor/src/main/res/values/strings.xml
-   */
-  private static void addStateSegments(
-      StringBuilder talkbackSegments, AccessibilityNodeInfoCompat node) {
-    // selected status is always prepended
-    if (node.isSelected()) {
-      talkbackSegments.append("selected" + delimiter);
-    }
-
-    // documented in reactnative accessibilityState
-    // https://reactnative.dev/docs/accessibility#accessibilitystate
-    // next check collapse/expand/checked status
-    if (supportsAction(node, AccessibilityNodeInfoCompat.ACTION_EXPAND)) {
-      talkbackSegments.append("collapsed" + delimiter);
-    }
-
-    if (supportsAction(node, AccessibilityNodeInfoCompat.ACTION_COLLAPSE)) {
-      talkbackSegments.append("expanded" + delimiter);
-    }
-  }
-
   /**
    * Creates the text that Google's TalkBack screen reader will read aloud for a given {@link View}.
    * This may be any combination of the {@link View}'s {@code text}, {@code contentDescription}, and
@@ -995,47 +955,23 @@ public static CharSequence getTalkbackDescription(
 
       final boolean hasNodeText = !TextUtils.isEmpty(nodeText);
       final boolean isEditText = view instanceof EditText;
-      // The original flipper implementation would check isActionableForAccessibility
-      // The check was removed for this reason https://bit.ly/3wPnmPE
-      boolean disabled = !node.isEnabled();
 
       StringBuilder talkbackSegments = new StringBuilder();
 
       // EditText's prioritize their own text content over a contentDescription so skip this
       if (!TextUtils.isEmpty(contentDescription) && (!isEditText || !hasNodeText)) {
-
-        // first prepend any status modifiers
-        addStateSegments(talkbackSegments, node);
-
         // next add content description
-        talkbackSegments.append(contentDescription + delimiter);
-
-        // lastly disabled is appended if applicable
-        if (disabled) {
-          talkbackSegments.append("disabled" + delimiter);
-        }
-
-        return removeFinalDelimiter(talkbackSegments);
+        talkbackSegments.append(contentDescription);
+        return talkbackSegments;
       }
 
       // EditText
       if (hasNodeText) {
         // skipped status checks above for EditText
-        //
-        // password
-        if (node.isPassword()) {
-          talkbackSegments.append("password" + delimiter);
-        }
 
         // description
-        talkbackSegments.append(nodeText + delimiter);
-
-        // disabled
-        if (disabled) {
-          talkbackSegments.append("disabled" + delimiter);
-        }
-
-        return removeFinalDelimiter(talkbackSegments);
+        talkbackSegments.append(nodeText);
+        return talkbackSegments;
       }
 
       // If there are child views and no contentDescription the text of all non-focusable children,

From cb0e1b6dc714fb57bb04eff620962cd009937a1f Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 6 Jun 2022 12:36:55 +0800
Subject: [PATCH 42/52] updating test cases descriptions

---
 .../Accessibility/AccessibilityExample.js     | 22 +++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index c543b3c0d26704..95a2c01eb5ac3f 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -213,8 +213,16 @@ class AccessibilityExample extends React.Component<{}> {
             />
           </View>
         </RNTesterBlock>
+      </View>
+    );
+  }
+}
 
-        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts - basic example">
+class AutomaticContentGrouping extends React.Component<{}> {
+  render(): React.Node {
+    return (
+      <View>
+        <RNTesterBlock title="Parent and children have different role">
           <TouchableNativeFeedback
             accessible={true}
             importantForAccessibility="yes"
@@ -228,7 +236,7 @@ class AccessibilityExample extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts">
+        <RNTesterBlock title="Parent has accessibilityAction activate and copy">
           <TouchableNativeFeedback
             accessible={true}
             importantForAccessibility="yes"
@@ -256,7 +264,7 @@ class AccessibilityExample extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts, one of them have the accessibilityLabel (contentDescription)">
+        <RNTesterBlock title="One of the child has accessibilityLabel, role, state, and accValue">
           <View accessible={true} accessibilityRole="button">
             <View>
               <Text accessible={false}>Text number 1</Text>
@@ -274,7 +282,7 @@ class AccessibilityExample extends React.Component<{}> {
           </View>
         </RNTesterBlock>
 
-        <RNTesterBlock title="TouchableNativeFeedback with non-accessible child Texts, one of them is an EditText">
+        <RNTesterBlock title="With child TextInput">
           <TouchableNativeFeedback
             accessible={true}
             importantForAccessibility="yes"
@@ -1258,6 +1266,12 @@ exports.examples = [
       return <AccessibilityExample />;
     },
   },
+  {
+    title: 'Automatic Content Grouping',
+    render(): React.Element<typeof AccessibilityExample> {
+      return <AutomaticContentGrouping />;
+    },
+  },
   {
     title: 'New accessibility roles and states',
     render(): React.Element<typeof AccessibilityRoleAndStateExample> {

From 0c79fc80895b87dd623adb7586da3f63f59258a8 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 6 Jun 2022 16:13:11 +0800
Subject: [PATCH 43/52] adding test cases scenarios

Scenarios:
- 3 levels of nested components 10min
- the contentDescription is not empty and does not have nodeText
- contentDescription is not empty and is not an EditText
- contentDescription is empty, is an EditText and has a nodeText
---
 .../Accessibility/AccessibilityExample.js     | 57 +++++++++++++++++++
 1 file changed, 57 insertions(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 95a2c01eb5ac3f..fed6003079468f 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -295,6 +295,63 @@ class AutomaticContentGrouping extends React.Component<{}> {
             />
           </TouchableNativeFeedback>
         </RNTesterBlock>
+
+        <RNTesterBlock title="Three levels of nested Components">
+          <TouchableNativeFeedback
+            accessible={true}
+            importantForAccessibility="yes"
+            accessibilityRole="button">
+            <Text accessible={false}>
+              Text number 2
+              <Text accessible={false}>
+                Text number 3<Text accessible={false}>Text number 4</Text>
+              </Text>
+            </Text>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
+
+        <RNTesterBlock title="The child is not TextInput, the contentDescription is not empty and does not have nodeText">
+          <TouchableNativeFeedback
+            onPress={() => console.warn('onPress child')}
+            accessible={true}
+            accessibilityRole="button">
+            <View>
+              <Text
+                accessibilityLabel="this is the child Text accessibilityLabel"
+                accessible={false}
+              />
+            </View>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
+
+        <RNTesterBlock title="The child is not TextInput, the contentDescription is not empty and does not have nodeText">
+          <TouchableNativeFeedback
+            onPress={() => console.warn('onPress child')}
+            accessible={true}
+            accessibilityRole="button">
+            <View>
+              <Text
+                accessibilityLabel="this is the child Text accessibilityLabel"
+                accessible={false}
+              />
+            </View>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
+
+        <RNTesterBlock title="The child is TextInput, the contentDescription is empty and has a nodeText">
+          <TouchableNativeFeedback
+            onPress={() => console.warn('onPress child')}
+            accessible={true}
+            accessibilityRole="button">
+            <View>
+              <Text accessible={false}>This is text number 1</Text>
+              <TextInput
+                accessible={false}
+                accessibilityLabel="this is the child Text accessibilityLabel"
+              />
+            </View>
+          </TouchableNativeFeedback>
+        </RNTesterBlock>
       </View>
     );
   }

From 46826d8cef43a217c8b1eb9cc2efce61891f8d85 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 6 Jun 2022 16:25:34 +0800
Subject: [PATCH 44/52] removing test case for EditText

---
 .../uimanager/ReactAccessibilityDelegate.java     |  4 ++--
 .../Accessibility/AccessibilityExample.js         | 15 ---------------
 2 files changed, 2 insertions(+), 17 deletions(-)

diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
index a9f680d26c87f0..f9b4cae504fe69 100644
--- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
+++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java
@@ -306,13 +306,13 @@ public void onInitializeAccessibilityNodeInfo(View host, AccessibilityNodeInfoCo
     }
     boolean missingContentDescription = TextUtils.isEmpty(info.getContentDescription());
     boolean missingText = TextUtils.isEmpty(info.getText());
-    boolean missingTextOrDescription = missingContentDescription && missingText;
+    boolean missingTextAndDescription = missingContentDescription && missingText;
     boolean hasContentToAnnounce =
         accessibilityActions != null
             || accessibilityState != null
             || accessibilityLabelledBy != null
             || accessibilityRole != null;
-    if (missingTextOrDescription && hasContentToAnnounce) {
+    if (missingTextAndDescription && hasContentToAnnounce) {
       info.setContentDescription(getTalkbackDescription(host, info));
     }
   }
diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index fed6003079468f..069ddeb824036c 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -337,21 +337,6 @@ class AutomaticContentGrouping extends React.Component<{}> {
             </View>
           </TouchableNativeFeedback>
         </RNTesterBlock>
-
-        <RNTesterBlock title="The child is TextInput, the contentDescription is empty and has a nodeText">
-          <TouchableNativeFeedback
-            onPress={() => console.warn('onPress child')}
-            accessible={true}
-            accessibilityRole="button">
-            <View>
-              <Text accessible={false}>This is text number 1</Text>
-              <TextInput
-                accessible={false}
-                accessibilityLabel="this is the child Text accessibilityLabel"
-              />
-            </View>
-          </TouchableNativeFeedback>
-        </RNTesterBlock>
       </View>
     );
   }

From ad6732aa3576786d37478a729b112031dd94b682 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Mon, 6 Jun 2022 16:46:29 +0800
Subject: [PATCH 45/52] fix flow circleci error

---
 .../rn-tester/js/examples/Accessibility/AccessibilityExample.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 069ddeb824036c..da544cda513750 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -1310,7 +1310,7 @@ exports.examples = [
   },
   {
     title: 'Automatic Content Grouping',
-    render(): React.Element<typeof AccessibilityExample> {
+    render(): React.Element<typeof AutomaticContentGrouping> {
       return <AutomaticContentGrouping />;
     },
   },

From d5f4c9c694e41ac7ef3961a0a7c6754db92fd465 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 7 Jun 2022 09:21:41 +0800
Subject: [PATCH 46/52] example - getStateDescription returns allways null

---
 .../Accessibility/AccessibilityExample.js     | 50 +++++++++++++++++++
 1 file changed, 50 insertions(+)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index da544cda513750..46d536bdc92e72 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -27,6 +27,7 @@ const {
   StyleSheet,
   Slider,
   Platform,
+  Switch,
 } = require('react-native');
 import type {EventSubscription} from 'react-native/Libraries/vendor/emitter/EventEmitter';
 
@@ -337,6 +338,55 @@ class AutomaticContentGrouping extends React.Component<{}> {
             </View>
           </TouchableNativeFeedback>
         </RNTesterBlock>
+
+        <RNTesterBlock title="One of the child has accessibilityState (hasStateDescription triggers the announcement)">
+          <View accessible={true} accessibilityRole="button">
+            <View>
+              <Text accessible={false}>Text number 1</Text>
+              <Image
+                source={{
+                  uri: 'https://play.google.com/intl/en_us/badges/images/generic/en_badge_web_generic.png',
+                }}
+                style={{height: 50, width: 50}}
+              />
+            </View>
+          </View>
+        </RNTesterBlock>
+
+        <RNTesterBlock title="One of the child has accessibilityState (hasStateDescription triggers the announcement)">
+          <View
+            accessible={true}
+            accessibilityHint="this is parent hint"
+            accessibilityState={{
+              checked: true,
+              selected: true,
+              disabled: true,
+              busy: true,
+              expanded: true,
+            }}>
+            <View
+              accessibilityState={{
+                checked: true,
+                selected: true,
+                disabled: true,
+                busy: true,
+                expanded: true,
+              }}>
+              <Text
+                accessibilityState={{
+                  checked: true,
+                  selected: true,
+                  disabled: true,
+                  busy: true,
+                  expanded: true,
+                }}
+                accessibilityState={{checked: true}}
+                accessible={false}>
+                Text number 1
+              </Text>
+            </View>
+          </View>
+        </RNTesterBlock>
       </View>
     );
   }

From c17f00ea321221d379a4b9e4fefbca991f50afc1 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 8 Jun 2022 13:17:45 +0800
Subject: [PATCH 47/52] One of the child has accessibilityState
 (hasStateDescription triggers the announcement)

from https://github.com/facebook/react-native/pull/33690#discussion_r890259702

I noticed that getStateDescription returns null even when adding accessibilityState. Calling setStateDescription will update the state description, which is not set via the react accessibilityState or other props.

Seems that the method [setViewState](https://github.com/fabriziobertoglio1987/react-native/blob/ad6732aa3576786d37478a729b112031dd94b682/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java#L254) does not call setStateDescription.

I will further investigate tomorrow.

<details><summary>View#setStateDescription</summary>
<p>

```java
    /**
     * Sets the {@link View}'s state description.
     * <p>
     * A state description briefly describes the states of the view and is primarily used
     * for accessibility support to determine how the states of a view should be presented to
     * the user. It is a supplement to the boolean states (for example, checked/unchecked) and
     * it is used for customized state description (for example, "wifi, connected, three bars").
     * State description changes frequently while content description should change less often.
     * State description should be localized. For android widgets which have default state
     * descriptions, app developers can call this method to override the state descriptions.
     * Setting state description to null restores the default behavior.
     *
     * @param stateDescription The state description.
     * @see #getStateDescription()
     */
    @RemotableViewMethod
    public void setStateDescription(@Nullable CharSequence stateDescription) {
        if (mStateDescription == null) {
            if (stateDescription == null) {
                return;
            }
        } else if (mStateDescription.equals(stateDescription)) {
            return;
        }
        mStateDescription = stateDescription;
        if (!TextUtils.isEmpty(stateDescription)
                && getImportantForAccessibility() == IMPORTANT_FOR_ACCESSIBILITY_AUTO) {
            setImportantForAccessibility(IMPORTANT_FOR_ACCESSIBILITY_YES);
        }
        if (AccessibilityManager.getInstance(mContext).isEnabled()) {
            AccessibilityEvent event = AccessibilityEvent.obtain();
            event.setEventType(AccessibilityEvent.TYPE_WINDOW_CONTENT_CHANGED);
            event.setContentChangeTypes(AccessibilityEvent.CONTENT_CHANGE_TYPE_STATE_DESCRIPTION);
            sendAccessibilityEventUnchecked(event);
        }
    }
```

</p>
</details>

This is the implementation of TalkBack [hasStateDescription](https://github.com/google/talkback/blob/6c0b475b7f52469e309e51bfcc13de58f18176ff/utils/src/main/java/com/google/android/accessibility/utils/AccessibilityNodeInfoUtils.java#L1672-L1677).

The following fields of the view are set with the accessibilityState:

- [checkable](https://github.com/fabriziobertoglio1987/react-native/blob/ad6732aa3576786d37478a729b112031dd94b682/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java#L411) => BOOLEAN_PROPERTY_CHECKABLE
- [enabled](https://github.com/fabriziobertoglio1987/react-native/blob/ad6732aa3576786d37478a729b112031dd94b682/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAccessibilityDelegate.java#L408) => BOOLEAN_PROPERTY_ENABLED

<details><summary>setCheckable</summary>
<p>

https://github.com/aosp-mirror/platform_frameworks_base/blob/19e53cfdc8a5c6ef45c0adf2dd239576ddce5822/core/java/android/view/accessibility/AccessibilityNodeInfo.java#L2008

```java
  /**
   * Sets whether this node is checkable.
   * <p>
   *   <strong>Note:</strong> Cannot be called from an
   *   {@link android.accessibilityservice.AccessibilityService}.
   *   This class is made immutable before being delivered to an AccessibilityService.
   * </p>
   *
   * @param checkable True if the node is checkable.
   *
   * @throws IllegalStateException If called from an AccessibilityService.
   */
  public void setCheckable(boolean checkable) {
      setBooleanProperty(BOOLEAN_PROPERTY_CHECKABLE, checkable);
  }
```

</p>
</details>

<details><summary>setEnabled</summary>
<p>

https://github.com/aosp-mirror/platform_frameworks_base/blob/19e53cfdc8a5c6ef45c0adf2dd239576ddce5822/core/java/android/view/accessibility/AccessibilityNodeInfo.java#L2227

```java
    /**
     * Sets whether this node is enabled.
     * <p>
     *   <strong>Note:</strong> Cannot be called from an
     *   {@link android.accessibilityservice.AccessibilityService}.
     *   This class is made immutable before being delivered to an AccessibilityService.
     * </p>
     *
     * @param enabled True if the node is enabled.
     *
     * @throws IllegalStateException If called from an AccessibilityService.
     */
    public void setEnabled(boolean enabled) {
        setBooleanProperty(BOOLEAN_PROPERTY_ENABLED, enabled);
    }
```

</p>
</details>

The implementation of `hasStateDescription` is still valid
---
 .../Accessibility/AccessibilityExample.js     | 56 ++++---------------
 1 file changed, 12 insertions(+), 44 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 46d536bdc92e72..db6b5ddf73b2c9 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -341,50 +341,18 @@ class AutomaticContentGrouping extends React.Component<{}> {
 
         <RNTesterBlock title="One of the child has accessibilityState (hasStateDescription triggers the announcement)">
           <View accessible={true} accessibilityRole="button">
-            <View>
-              <Text accessible={false}>Text number 1</Text>
-              <Image
-                source={{
-                  uri: 'https://play.google.com/intl/en_us/badges/images/generic/en_badge_web_generic.png',
-                }}
-                style={{height: 50, width: 50}}
-              />
-            </View>
-          </View>
-        </RNTesterBlock>
-
-        <RNTesterBlock title="One of the child has accessibilityState (hasStateDescription triggers the announcement)">
-          <View
-            accessible={true}
-            accessibilityHint="this is parent hint"
-            accessibilityState={{
-              checked: true,
-              selected: true,
-              disabled: true,
-              busy: true,
-              expanded: true,
-            }}>
-            <View
-              accessibilityState={{
-                checked: true,
-                selected: true,
-                disabled: true,
-                busy: true,
-                expanded: true,
-              }}>
-              <Text
-                accessibilityState={{
-                  checked: true,
-                  selected: true,
-                  disabled: true,
-                  busy: true,
-                  expanded: true,
-                }}
-                accessibilityState={{checked: true}}
-                accessible={false}>
-                Text number 1
-              </Text>
-            </View>
+            <Text
+              accessible={false}
+              accessibilityLabel="label for text number 1">
+              Text number 1
+            </Text>
+            <Text
+              style={{backgroundColor: 'red', height: 40, width: 40}}
+              accessible={false}
+              accessibilityState={{checked: true}}
+              accessibilityLabel="label for last child"
+              accessibilityRole="image"
+            />
           </View>
         </RNTesterBlock>
       </View>

From 087e64212ecdd69f5dfedb9405b9fd9c62bdfb08 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 8 Jun 2022 14:59:41 +0800
Subject: [PATCH 48/52] One of the child has accessibilityHint (hasText trigers
 the announcement)

---
 .../Accessibility/AccessibilityExample.js     | 25 +++++++++++++------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index db6b5ddf73b2c9..128224928f00dd 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -69,6 +69,11 @@ const styles = StyleSheet.create({
     flexDirection: 'column',
     justifyContent: 'space-between',
   },
+  smallRedSquare: {
+    backgroundColor: 'red',
+    height: 40,
+    width: 40,
+  },
 });
 
 class AccessibilityExample extends React.Component<{}> {
@@ -341,20 +346,26 @@ class AutomaticContentGrouping extends React.Component<{}> {
 
         <RNTesterBlock title="One of the child has accessibilityState (hasStateDescription triggers the announcement)">
           <View accessible={true} accessibilityRole="button">
+            <Text accessible={false}>Text number 1</Text>
             <Text
-              accessible={false}
-              accessibilityLabel="label for text number 1">
-              Text number 1
-            </Text>
-            <Text
-              style={{backgroundColor: 'red', height: 40, width: 40}}
+              style={styles.smallRedSquare}
               accessible={false}
               accessibilityState={{checked: true}}
-              accessibilityLabel="label for last child"
+              accessibilityLabel="this child Text does not have text, but has state and should be announced by TalkBack"
               accessibilityRole="image"
             />
           </View>
         </RNTesterBlock>
+
+        <RNTesterBlock title="One of the child has accessibilityHint (hasText triggers the announcement)">
+          <View accessible={true} accessibilityRole="button">
+            <Text
+              style={styles.smallRedSquare}
+              accessible={false}
+              accessibilityHint="this child Text does not have text, but has hint and should be announced by TalkBack"
+            />
+          </View>
+        </RNTesterBlock>
       </View>
     );
   }

From 51f23f058fdc068a2f49f177d679b5995ff7f20c Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 8 Jun 2022 21:46:29 +0800
Subject: [PATCH 49/52] remove import of Switch in AccessibilityExample

---
 .../rn-tester/js/examples/Accessibility/AccessibilityExample.js  | 1 -
 1 file changed, 1 deletion(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 128224928f00dd..2cadc81384bbd8 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -27,7 +27,6 @@ const {
   StyleSheet,
   Slider,
   Platform,
-  Switch,
 } = require('react-native');
 import type {EventSubscription} from 'react-native/Libraries/vendor/emitter/EventEmitter';
 

From 8b84521de8b7d6fd2383a42fa8cf776407da54a8 Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Wed, 8 Jun 2022 22:04:33 +0800
Subject: [PATCH 50/52] fix AccessibilityExample actions

---
 .../Accessibility/AccessibilityExample.js      | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 2cadc81384bbd8..e89b382b95d9fa 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -241,25 +241,29 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="Parent has accessibilityAction activate and copy">
+        <RNTesterBlock title="Parent has accessibilityAction cut, copy and paste">
           <TouchableNativeFeedback
             accessible={true}
             importantForAccessibility="yes"
-            accessibilityRole="button"
             accessibilityActions={[
-              {name: 'activate', label: 'activate label'},
+              {name: 'cut', label: 'cut label'},
               {name: 'copy', label: 'copy label'},
+              {name: 'paste', label: 'paste label'},
             ]}
             onAccessibilityAction={event => {
               switch (event.nativeEvent.actionName) {
-                case 'activate':
-                  Alert.alert('Alert', 'Activate accessiblity action');
+                case 'cut':
+                  Alert.alert('Alert', 'cut action success');
                   break;
                 case 'copy':
                   Alert.alert('Alert', 'copy action success');
                   break;
+                case 'paste':
+                  Alert.alert('Alert', 'paste action success');
+                  break;
               }
-            }}>
+            }}
+            accessibilityRole="button">
             <View>
               <Text accessible={false}>Text number 1</Text>
               <Text accessible={false}>
@@ -322,6 +326,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
             accessibilityRole="button">
             <View>
               <Text
+                style={styles.smallRedSquare}
                 accessibilityLabel="this is the child Text accessibilityLabel"
                 accessible={false}
               />
@@ -336,6 +341,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
             accessibilityRole="button">
             <View>
               <Text
+                style={styles.smallRedSquare}
                 accessibilityLabel="this is the child Text accessibilityLabel"
                 accessible={false}
               />

From ddfb34575d3155e5a7cd2ea13e62bed902fc92dc Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 30 Aug 2022 19:35:27 +0800
Subject: [PATCH 51/52] updating examples in AccessibilityExample

This example feels a bit odd to me. The state itself is not concatenated into the announcement, which makes sense, as it would lead to potentially announcing conflicting state if two child elements had separate states like "checked" and "unchecked", but because of this I'm not sure what value this example is providing. If there was no state here and just the label it would work the exact same way.
I think we can probably just remove this one and maybe replace it with one where the parent element has a state and making sure that state is reflected even when the children are combined into it.
https://github.com/facebook/react-native/pull/33690#discussion_r939454427
https://github.com/facebook/react-native/pull/33690#discussion_r958336725

I don't think importantForAccessibility="yes" is necessary for any of these examples, so lets remove it on all of them just to make it clear that this isn't a prequalification for this feature working.
https://github.com/facebook/react-native/pull/33690#discussion_r939457620

Remove duplicate example (this is the same as the previous one)
https://github.com/facebook/react-native/pull/33690#discussion_r939454068
---
 .../Accessibility/AccessibilityExample.js     | 64 +++++++------------
 1 file changed, 23 insertions(+), 41 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index 1a7b4ca950b36d..d0b2a18d2fb8c8 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -255,10 +255,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
     return (
       <View>
         <RNTesterBlock title="Parent and children have different role">
-          <TouchableNativeFeedback
-            accessible={true}
-            importantForAccessibility="yes"
-            accessibilityRole="button">
+          <TouchableNativeFeedback accessible={true} accessibilityRole="button">
             <View accessible={false}>
               <Text accessibilityRole="image" accessible={false}>
                 Text number 1 with a role
@@ -271,7 +268,6 @@ class AutomaticContentGrouping extends React.Component<{}> {
         <RNTesterBlock title="Parent has accessibilityAction cut, copy and paste">
           <TouchableNativeFeedback
             accessible={true}
-            importantForAccessibility="yes"
             accessibilityActions={[
               {name: 'cut', label: 'cut label'},
               {name: 'copy', label: 'copy label'},
@@ -300,6 +296,26 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
+        <RNTesterBlock title="Talkback only pulls the child's contentDescription or text but does not include the child's accessibilityState or accessibilityRole. TalkBack avoids announcements of conflicting states or roles (for example, 'button' and 'slider').">
+          <View
+            accessible={true}
+            accessibilityRole="button"
+            accessibilityState={{checked: true}}>
+            <Text
+              accessible={false}
+              accessibilityState={{checked: true, disabled: false}}>
+              Text number 1
+            </Text>
+            <Text
+              style={styles.smallRedSquare}
+              accessible={false}
+              accessibilityState={{checked: false, disabled: true}}
+              accessibilityLabel="This child Text does not have text, but has an accessibilityLabel and accessibilityState. The child accessibility state disabled is not announced."
+              accessibilityRole="image"
+            />
+          </View>
+        </RNTesterBlock>
+
         <RNTesterBlock title="One of the child has accessibilityLabel, role, state, and accValue">
           <View accessible={true} accessibilityRole="button">
             <View>
@@ -319,10 +335,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
         </RNTesterBlock>
 
         <RNTesterBlock title="With child TextInput">
-          <TouchableNativeFeedback
-            accessible={true}
-            importantForAccessibility="yes"
-            accessibilityRole="button">
+          <TouchableNativeFeedback accessible={true} accessibilityRole="button">
             <TextInput
               value="this is the value"
               accessible={false}
@@ -333,10 +346,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
         </RNTesterBlock>
 
         <RNTesterBlock title="Three levels of nested Components">
-          <TouchableNativeFeedback
-            accessible={true}
-            importantForAccessibility="yes"
-            accessibilityRole="button">
+          <TouchableNativeFeedback accessible={true} accessibilityRole="button">
             <Text accessible={false}>
               Text number 2
               <Text accessible={false}>
@@ -361,34 +371,6 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="The child is not TextInput, the contentDescription is not empty and does not have nodeText">
-          <TouchableNativeFeedback
-            onPress={() => console.warn('onPress child')}
-            accessible={true}
-            accessibilityRole="button">
-            <View>
-              <Text
-                style={styles.smallRedSquare}
-                accessibilityLabel="this is the child Text accessibilityLabel"
-                accessible={false}
-              />
-            </View>
-          </TouchableNativeFeedback>
-        </RNTesterBlock>
-
-        <RNTesterBlock title="One of the child has accessibilityState (hasStateDescription triggers the announcement)">
-          <View accessible={true} accessibilityRole="button">
-            <Text accessible={false}>Text number 1</Text>
-            <Text
-              style={styles.smallRedSquare}
-              accessible={false}
-              accessibilityState={{checked: true}}
-              accessibilityLabel="this child Text does not have text, but has state and should be announced by TalkBack"
-              accessibilityRole="image"
-            />
-          </View>
-        </RNTesterBlock>
-
         <RNTesterBlock title="One of the child has accessibilityHint (hasText triggers the announcement)">
           <View accessible={true} accessibilityRole="button">
             <Text

From 11c4b00a399914cd783c5b89ab1b1c815c6556af Mon Sep 17 00:00:00 2001
From: fabriziobertoglio1987 <fabrizio.bertoglio@gmail.com>
Date: Tue, 30 Aug 2022 19:51:17 +0800
Subject: [PATCH 52/52] improving examples titles

---
 .../examples/Accessibility/AccessibilityExample.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
index d0b2a18d2fb8c8..bbe0143ac5a1b7 100644
--- a/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
+++ b/packages/rn-tester/js/examples/Accessibility/AccessibilityExample.js
@@ -254,7 +254,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
   render(): React.Node {
     return (
       <View>
-        <RNTesterBlock title="Parent and children have different role">
+        <RNTesterBlock title="The parent and the children have a different role">
           <TouchableNativeFeedback accessible={true} accessibilityRole="button">
             <View accessible={false}>
               <Text accessibilityRole="image" accessible={false}>
@@ -265,7 +265,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="Parent has accessibilityAction cut, copy and paste">
+        <RNTesterBlock title="The parent has the accessibilityActions cut, copy and paste">
           <TouchableNativeFeedback
             accessible={true}
             accessibilityActions={[
@@ -316,7 +316,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </View>
         </RNTesterBlock>
 
-        <RNTesterBlock title="One of the child has accessibilityLabel, role, state, and accValue">
+        <RNTesterBlock title="One of the children has accessibilityLabel, role, state, and accessibilityValue.">
           <View accessible={true} accessibilityRole="button">
             <View>
               <Text accessible={false}>Text number 1</Text>
@@ -334,7 +334,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </View>
         </RNTesterBlock>
 
-        <RNTesterBlock title="With child TextInput">
+        <RNTesterBlock title="The parent has a TextInput child component.">
           <TouchableNativeFeedback accessible={true} accessibilityRole="button">
             <TextInput
               value="this is the value"
@@ -345,7 +345,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="Three levels of nested Components">
+        <RNTesterBlock title="The parents include three levels of nested Components.">
           <TouchableNativeFeedback accessible={true} accessibilityRole="button">
             <Text accessible={false}>
               Text number 2
@@ -356,7 +356,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="The child is not TextInput, the contentDescription is not empty and does not have nodeText">
+        <RNTesterBlock title="The child is not TextInput. The contentDescription is not empty and does not have node text.">
           <TouchableNativeFeedback
             onPress={() => console.warn('onPress child')}
             accessible={true}
@@ -371,7 +371,7 @@ class AutomaticContentGrouping extends React.Component<{}> {
           </TouchableNativeFeedback>
         </RNTesterBlock>
 
-        <RNTesterBlock title="One of the child has accessibilityHint (hasText triggers the announcement)">
+        <RNTesterBlock title="One of the child has accessibilityHint (hasText triggers the announcement).">
           <View accessible={true} accessibilityRole="button">
             <Text
               style={styles.smallRedSquare}