From 81f2bdfccb6e72ac4b90e6ea24ce24c71e3074cb Mon Sep 17 00:00:00 2001
From: Jose Ivan Vargas <jvargas@gitlab.com>
Date: Fri, 10 Nov 2017 16:58:55 -0600
Subject: Removed tooltip from clone dropdown

---
 app/helpers/button_helper.rb | 46 +++++++++++++++++++-------------------------
 1 file changed, 20 insertions(+), 26 deletions(-)

(limited to 'app/helpers/button_helper.rb')

diff --git a/app/helpers/button_helper.rb b/app/helpers/button_helper.rb
index 8e8feeea1d8..fe5abf394a6 100644
--- a/app/helpers/button_helper.rb
+++ b/app/helpers/button_helper.rb
@@ -57,41 +57,35 @@ module ButtonHelper
   end
 
   def http_clone_button(project, placement = 'right', append_link: true)
-    klass = 'http-selector'
-    klass << ' has-tooltip' if current_user.try(:require_extra_setup_for_git_auth?)
-
     protocol = gitlab_config.protocol.upcase
 
-    tooltip_title =
-      if current_user.try(:require_password_creation_for_git?)
+    protocol_description =
+      if current_user.try(:require_password_creation?)
         _("Set a password on your account to pull or push via %{protocol}.") % { protocol: protocol }
       else
         _("Create a personal access token on your account to pull or push via %{protocol}.") % { protocol: protocol }
       end
 
-    content_tag (append_link ? :a : :span), protocol,
-      class: klass,
-      href: (project.http_url_to_repo if append_link),
-      data: {
-        html: true,
-        placement: placement,
-        container: 'body',
-        title: tooltip_title
-      }
+    protocol_element_output = content_tag(:strong, protocol, class: 'dropdown-menu-inner-title')
+
+    if current_user.try(:require_password_creation?) || current_user.try(:require_personal_access_token_creation_for_git_auth?)
+      protocol_element_output << content_tag(:span, protocol_description, class: 'dropdown-menu-inner-content')
+    end
+
+    content_tag (append_link ? :a : :span),
+      protocol_element_output,
+      class: 'http-selector',
+      href: (project.http_url_to_repo if append_link)
   end
 
-  def ssh_clone_button(project, placement = 'right', append_link: true)
-    klass = 'ssh-selector'
-    klass << ' has-tooltip' if current_user.try(:require_ssh_key?)
+  def ssh_clone_button(project, append_link: true)
+    ssh_description = _('Add an SSH key to your profile to pull or push via SSH.')
+    ssh_element_output = content_tag(:strong, 'SSH', class: 'dropdown-menu-inner-title')
+    ssh_element_output << content_tag(:span, ssh_description, class: 'dropdown-menu-inner-content') if current_user.try(:require_ssh_key?)
 
-    content_tag (append_link ? :a : :span), 'SSH',
-      class: klass,
-      href: (project.ssh_url_to_repo if append_link),
-      data: {
-        html: true,
-        placement: placement,
-        container: 'body',
-        title: _('Add an SSH key to your profile to pull or push via SSH.')
-      }
+    content_tag (append_link ? :a : :span),
+      ssh_element_output,
+      class: 'ssh-selector',
+      href: (project.ssh_url_to_repo if append_link)
   end
 end
-- 
cgit v1.2.3