Merge branch '66067-pages-domain-doesnt-set-target-blank' into 'master'

Makes custom Pages domain open as external link in new tab

Closes #66067

See merge request gitlab-org/gitlab-ce!32130

4 files changed, 15 insertions, 8 deletions

diff --git a/app/helpers/external_link_helper.rb b/app/helpers/external_link_helper.rb
new file mode 100644
index 00000000000..9dbad1f5032
--- /dev/null
+++ b/app/helpers/external_link_helper.rb
@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module ExternalLinkHelper
+  def external_link(body, url, options = {})
+    link_to url, { target: '_blank', rel: 'noopener noreferrer' }.merge(options) do
+      "#{body} #{icon('external-link')}".html_safe
+    end
+  end
+end
diff --git a/app/views/projects/pages/_access.html.haml b/app/views/projects/pages/_access.html.haml
index 539f223ca9b..7b6d46964a2 100644
--- a/app/views/projects/pages/_access.html.haml
+++ b/app/views/projects/pages/_access.html.haml
@@ -7,9 +7,11 @@
         %strong
           = _("Your pages are served under:")
 
-      %p= link_to @project.pages_url, @project.pages_url
+      %p
+        = external_link(@project.pages_url, @project.pages_url)
 
       - @project.pages_domains.each do |domain|
-        %p= link_to domain.url, domain.url
+        %p
+          = external_link(domain.url, domain.url)
     .card-footer.alert-primary
       = _("It may take up to 30 minutes before the site is available after the first deployment.")
diff --git a/app/views/projects/pages/_list.html.haml b/app/views/projects/pages/_list.html.haml
index 2427b4d7611..c4285e7f3d2 100644
--- a/app/views/projects/pages/_list.html.haml
+++ b/app/views/projects/pages/_list.html.haml
@@ -12,9 +12,7 @@
             .domain-status.ci-status-icon.has-tooltip{ class: "ci-status-icon-#{status}", title: tooltip }
               = sprite_icon("status_#{status}", size: 16 )
           .domain-name
-            = link_to domain.url do
-              = domain.url
-              = icon('external-link')
+            = external_link(domain.url, domain.url)
             - if domain.subject
               %div
                 %span.badge.badge-gray Certificate: #{domain.subject}
diff --git a/app/views/projects/pages_domains/show.html.haml b/app/views/projects/pages_domains/show.html.haml
index d0b54946f7e..33837e21c8d 100644
--- a/app/views/projects/pages_domains/show.html.haml
+++ b/app/views/projects/pages_domains/show.html.haml
@@ -21,9 +21,7 @@
       %td
         = _("Domain")
       %td
-        = link_to @domain.url do
-          = @domain.url
-          = icon('external-link')
+        = external_link(@domain.url, @domain.url)
     %tr
       %td
         = _("DNS")