Merge pull request #1240 from City-of-Helsinki/bug/TP-2126_quick-link-target-blank

TP-2126 Added missing target=_blank for external quick links

Author: jussiviitamaki

public/themes/custom/palvelumanuaali/palvelumanuaali.theme

@@ -685,6 +685,7 @@ function palvelumanuaali_preprocess_paragraph__quick_link__default(&$variables,
   if (!empty($content['field_short_description']['#items'])) {
     $description = $content['field_short_description']['#items']->getValue()[0]['value'];
   }
+  $variables['quick_link']['external'] = $link['#url']->isExternal();
   $variables['quick_link']['attributes'] = [
     'title' => $title,
     'description' => $description,

public/themes/custom/palvelumanuaali/templates/paragraphs/paragraph--quick-link.html.twig

@@ -42,17 +42,19 @@
     paragraph.bundle|clean_class
   ]
 %}
-{% if content['field_link'].0['#url'].external %}
+{% if quick_link.external %}
   {% set icon_name = 'arrow-right-2-ext' %}
 {% else %}
   {% set icon_name = 'arrow-right-2' %}
 {% endif %}
-
 <a
     href="{{ quick_link.url }}"
     title="{{ quick_link.attributes.title }}"
     aria-label="{{ quick_link.attributes.title }}"
     aria-description="{{ quick_link.attributes.description }}"
+    {% if quick_link.external %}
+      target="_blank"
+    {% endif %}
     class="quick-link add--break-word add--hyphen card--default-hover">
   {{ content|without('field_link') }}
   {% include "@atoms/images/icons/_icon.twig" with {