apply the target attribute on semantic links

flutter · May 31, 2024 · d98eab2 · d98eab2
1 parent c1556cc
commit d98eab2
Showing 1 changed file with 29 additions and 13 deletions.
diff --git a/packages/url_launcher/url_launcher_web/lib/src/link.dart b/packages/url_launcher/url_launcher_web/lib/src/link.dart
@@ -512,12 +512,16 @@ class LinkViewController extends PlatformViewController {
     }
   }
 
+  late LinkTarget _target;
+  String get _htmlTargetAttribute => _getHtmlTargetAttribute(_target);
+
   /// Set the [LinkTarget] value for this link.
   void setTarget(LinkTarget target) {
-    _element.setAttribute('target', _getHtmlTarget(target));
+    _target = target;
+    _element.setAttribute('target', _htmlTargetAttribute);
   }
 
-  String _getHtmlTarget(LinkTarget target) {
+  String _getHtmlTargetAttribute(LinkTarget target) {
     switch (target) {
       case LinkTarget.defaultTarget:
       case LinkTarget.self:
@@ -537,18 +541,30 @@ class LinkViewController extends PlatformViewController {
   ///
   /// Returns null if [target] is not a semantics element for one of our Links.
   static int? _getViewIdFromSemanticLink(html.Element? target) {
-    // TODO: what if `target` IS the <a> semantic element?
-    if (target != null && _isWithinSemanticTree(target)) {
-      final html.Element? semanticLink = _getClosestSemanticLink(target);
-      if (semanticLink != null) {
-        // TODO: Find out the view ID of semantic link.
-        final String? semanticIdentifier = semanticLink.getAttribute('semantic-identifier');
-        if (semanticIdentifier != null) {
-          return _instancesBySemanticIdentifier[semanticIdentifier]?.viewId;
-        }
-      }
+    if (target == null) {
+      return null;
+    }
+    if (!_isWithinSemanticTree(target)) {
+      return null;
+    }
+
+    final html.Element? semanticLink = _getClosestSemanticLink(target);
+    if (semanticLink == null) {
+      return null;
     }
-    return null;
+
+    final String? semanticIdentifier = semanticLink.getAttribute('semantic-identifier');
+    if (semanticIdentifier == null) {
+      return null;
+    }
+
+    final LinkViewController? controller = _instancesBySemanticIdentifier[semanticIdentifier];
+    if (controller == null) {
+      return null;
+    }
+
+    semanticLink.setAttribute('target', controller._htmlTargetAttribute);
+    return controller.viewId;
   }
 
   @override