a11y] Let links discovered by dataDetectorType be accessible (#50914)

Summary:
Pull Request resolved: #50914

The URL spans generated by Linkify are not actually accessible because we do not update the delegate's virtual views.

I also had to change how AccessibilityLinks get generated, since it does not work well if it includes spans that are not `ReactClickableSpans`

Differential Revision: D73612119

Author: joevilches

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextViewAccessibilityDelegate.kt

@@ -270,11 +270,13 @@ internal class ReactTextViewAccessibilityDelegate : ReactAccessibilityDelegate {
     return null
   }
 
-  public class AccessibilityLinks(spans: Array<ClickableSpan?>, text: Spannable) {
+  public class AccessibilityLinks(text: Spannable) {
     private val links: List<AccessibleLink>
 
     init {
       val accessibleLinks = mutableListOf<AccessibleLink>()
+      val spans = text.getSpans(0, text.length, ClickableSpan::class.java)
+      spans.sortBy { text.getSpanStart(it) }
       for (i in spans.indices) {
         val span = spans[i]
         val start = text.getSpanStart(span)
@@ -288,14 +290,7 @@ internal class ReactTextViewAccessibilityDelegate : ReactAccessibilityDelegate {
         link.description = text.subSequence(start, end).toString()
         link.start = start
         link.end = end
-
-        // ID is the reverse of what is expected, since the ClickableSpans are returned in reverse
-        // order due to being added in reverse order. If we don't do this, focus will move to the
-        // last link first and move backwards.
-        //
-        // If this approach becomes unreliable, we should instead look at their start position and
-        // order them manually.
-        link.id = spans.size - 1 - i
+        link.id = i
         accessibleLinks.add(link)
       }
       links = accessibleLinks

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/views/text/ReactTextViewManager.java

@@ -10,6 +10,7 @@
 import android.content.Context;
 import android.os.Build;
 import android.text.Spannable;
+import android.text.style.ClickableSpan;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import com.facebook.common.logging.FLog;
@@ -26,7 +27,6 @@
 import com.facebook.react.uimanager.StateWrapper;
 import com.facebook.react.uimanager.ThemedReactContext;
 import com.facebook.react.uimanager.annotations.ReactProp;
-import com.facebook.react.views.text.internal.span.ReactClickableSpan;
 import com.facebook.react.views.text.internal.span.TextInlineImageSpan;
 import com.facebook.yoga.YogaMeasureMode;
 import java.util.HashMap;
@@ -106,12 +106,12 @@ public void updateExtraData(ReactTextView view, Object extraData) {
 
       // If this text view contains any clickable spans, set a view tag and reset the accessibility
       // delegate so that these can be picked up by the accessibility system.
-      ReactClickableSpan[] clickableSpans =
-          spannable.getSpans(0, update.getText().length(), ReactClickableSpan.class);
+      ClickableSpan[] clickableSpans =
+          spannable.getSpans(0, update.getText().length(), ClickableSpan.class);
       view.setTag(
           R.id.accessibility_links,
           clickableSpans.length > 0
-              ? new ReactTextViewAccessibilityDelegate.AccessibilityLinks(clickableSpans, spannable)
+              ? new ReactTextViewAccessibilityDelegate.AccessibilityLinks(spannable)
               : null);
       ReactTextViewAccessibilityDelegate.Companion.resetDelegate(
           view, view.isFocusable(), view.getImportantForAccessibility());