From 1e1785cc1699f33e4c1f98cde71fc19d1be10d6a Mon Sep 17 00:00:00 2001
From: Robert Speicher <rspeicher@gmail.com>
Date: Sat, 25 Apr 2015 16:03:18 -0400
Subject: [PATCH] Place the logo's tooltip dynamically

When collapsed it's on the right like everything else, when expanded
it's on the bottom.
---
 app/assets/javascripts/application.js.coffee | 8 ++++++++
 app/views/layouts/_head_panel.html.haml      | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/app/assets/javascripts/application.js.coffee b/app/assets/javascripts/application.js.coffee
index eb6ecaa4835ca..bb9da1470180b 100644
--- a/app/assets/javascripts/application.js.coffee
+++ b/app/assets/javascripts/application.js.coffee
@@ -134,6 +134,14 @@ $ ->
   # Initialize tooltips
   $('body').tooltip({
     selector: '.has_tooltip, [data-toggle="tooltip"], .page-sidebar-collapsed .nav-sidebar a'
+    placement: (_, el) ->
+      $el = $(el)
+      if $el.attr('id') == 'js-shortcuts-home'
+        # Place the logo tooltip on the right when collapsed, bottom when expanded
+        $el.parents('header').hasClass('header-collapsed') and 'right' or 'bottom'
+      else
+        # Otherwise use the data-placement attribute like normal
+        $el.data('placement')
   })
 
   # Form submitter
diff --git a/app/views/layouts/_head_panel.html.haml b/app/views/layouts/_head_panel.html.haml
index 83387ad902641..8d4c7b39b19b0 100644
--- a/app/views/layouts/_head_panel.html.haml
+++ b/app/views/layouts/_head_panel.html.haml
@@ -2,7 +2,7 @@
   .navbar-inner
     .container
       %div.app_logo
-        = link_to root_path, class: 'home', title: 'Dashboard', data: {toggle: 'tooltip', placement: 'bottom'} do
+        = link_to root_path, class: 'home', title: 'Dashboard', id: 'js-shortcuts-home', data: {toggle: 'tooltip', placement: 'bottom'} do
           = brand_header_logo
           %h3 GitLab
       %h1.title= title
-- 
GitLab