Make AccessibilityOrder also include the host view (#51027)

jorge-cab · facebook-github-bot · commit ae2773f23196 · 2025-04-30T11:54:09.000-07:00
Summary: Pull Request resolved: #51027 Simple change to make the host of `experimental_accessibilityOrder` include the view that hosts the property in its order. Changelog: [Internal] Reviewed By: joevilches Differential Revision: D73808337 fbshipit-source-id: 441329a6ca0cd4b0aba08bddb15143215d337e01
diff --git a/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAxOrderHelper.kt b/packages/react-native/ReactAndroid/src/main/java/com/facebook/react/uimanager/ReactAxOrderHelper.kt
@@ -122,11 +122,7 @@ private object ReactAxOrderHelper {
       }
     }
 
-    if (root is ViewGroup) {
-      for (i in 0 until root.childCount) {
-        traverseAndDisableAxFromExcludedViews(root.getChildAt(i), root)
-      }
-    }
+    traverseAndDisableAxFromExcludedViews(root, root)
 
     return axOrderViews
   }

Original file line number	Diff line number	Diff line change
`@@ -122,11 +122,7 @@ private object ReactAxOrderHelper {`
`122`	`122`	`}`
`123`	`123`	`}`
`124`	`124`
`125`		`- if (root is ViewGroup) {`
`126`		`- for (i in 0 until root.childCount) {`
`127`		`- traverseAndDisableAxFromExcludedViews(root.getChildAt(i), root)`
`128`		`- }`
`129`		`- }`
	`125`	`+ traverseAndDisableAxFromExcludedViews(root, root)`
`130`	`126`
`131`	`127`	`return axOrderViews`
`132`	`128`	`}`