From 765a577669c65a4cb51e2ca7b6a6f15b19cada90 Mon Sep 17 00:00:00 2001
From: Urban Rotnik <urban.rotnik@gmail.com>
Date: Wed, 29 Jan 2020 13:22:35 +0100
Subject: [PATCH] Ellipsis with css for Sidebar

---
 app/assets/javascripts/application.js.erb      | 9 ---------
 app/views/repositories/_sidebar.html.erb       | 7 +++----
 app/views/shared/sidebar/_experiments.html.erb | 5 ++---
 app/views/shared/sidebar/_my_modules.html.erb  | 4 ++--
 app/views/shared/sidebar/_projects.html.erb    | 3 ++-
 5 files changed, 9 insertions(+), 19 deletions(-)

diff --git a/app/assets/javascripts/application.js.erb b/app/assets/javascripts/application.js.erb
index 313487c33..c07991020 100644
--- a/app/assets/javascripts/application.js.erb
+++ b/app/assets/javascripts/application.js.erb
@@ -191,15 +191,6 @@ var HelperModule = (function(){
 
   var helpers = {};
 
-  helpers.treeLinkTruncation = function() {
-    $('.tree-link a').each( function(){
-      truncateLongString( $(this), <%= Constants::NAME_TRUNCATION_LENGTH %>);
-    });
-    $(".tree-link span").each( function(){
-      truncateLongString( $(this), <%= Constants::NAME_TRUNCATION_LENGTH %>);
-    });
-  }
-
   helpers.hideFlashMsg =  function() {
     var flash = $('.alert');
     if (flash.length > 0) {
diff --git a/app/views/repositories/_sidebar.html.erb b/app/views/repositories/_sidebar.html.erb
index 43f03485b..1e065e59b 100644
--- a/app/views/repositories/_sidebar.html.erb
+++ b/app/views/repositories/_sidebar.html.erb
@@ -4,7 +4,7 @@
       <ul>
         <% if can_create_repositories?(current_team) %>
           <li>
-            <span class="tree-link link-wrap no-indent">
+            <span class="tree-link no-indent">
               <a class="sidebar-button create-new-repository" data-remote="true"
                  href="<%= create_modal_team_repositories_path(current_team) %>">
                 <span class="fas fa-plus"></span>
@@ -15,11 +15,10 @@
         <% end %>
         <% repositories.each do |repository| %>
           <li class="<%= 'active parent_li' if current_page?(repository_path(repository)) %>" >
-            <span class="tree-link line-wrap no-indent">
+            <span class="tree-link no-indent" title="<%= repository.name %>">
               <%= link_to repository.name,
                           repository_path(repository),
-                          class: current_page?(repository_path(repository)) ? 'disabled' : '',
-                          title: repository.name,
+                          class: current_page?(repository_path(repository)) ? 'disabled line-wrap' : 'line-wrap',
                           data: { 'no-turbolink' => 'true', type: 'repository', id: repository.id } %>
               <%= inventory_shared_status_icon(repository, current_team) %>
             </span>
diff --git a/app/views/shared/sidebar/_experiments.html.erb b/app/views/shared/sidebar/_experiments.html.erb
index 83113d638..78a9ccc45 100644
--- a/app/views/shared/sidebar/_experiments.html.erb
+++ b/app/views/shared/sidebar/_experiments.html.erb
@@ -3,12 +3,11 @@
     <% project.active_experiments.each do |experiment| %>
       <% cache [action_name, current_user, experiment] do %>
         <li data-parent="candidate">
-          <span class="tree-link line-wrap first-indent">
+          <span class="tree-link line-wrap first-indent" title="<%= experiment.name %>">
             <i class="no-arrow"></i>
             <%= link_to experiment.name,
                         experiment_action_to_link_to(experiment),
-                        title: experiment.name,
-                        class: 'overview_exp_label',
+                        class: 'overview_exp_label line-wrap',
                         data: { type: 'experiment', id: experiment.id }
                         %>
           </span>
diff --git a/app/views/shared/sidebar/_my_modules.html.erb b/app/views/shared/sidebar/_my_modules.html.erb
index 068a6c800..a3a144d72 100644
--- a/app/views/shared/sidebar/_my_modules.html.erb
+++ b/app/views/shared/sidebar/_my_modules.html.erb
@@ -2,10 +2,10 @@
   <ul>
     <% experiment.active_my_modules.each do |my_module| %>
       <li class="leaf" data-module-id="<%= my_module.id %>">
-        <span class="tree-link task-tree-link second-indent">
+        <span class="tree-link task-tree-link second-indent" title="<%= my_module.name %>">
           <%= link_to my_module.name,
                       module_action_to_link_to(my_module),
-                      title: my_module.name,
+                      class: 'line-wrap',
                       data: { type: 'my_module', id: my_module.id } %>
         </span>
       </li>
diff --git a/app/views/shared/sidebar/_projects.html.erb b/app/views/shared/sidebar/_projects.html.erb
index cb1515d97..f923b508c 100644
--- a/app/views/shared/sidebar/_projects.html.erb
+++ b/app/views/shared/sidebar/_projects.html.erb
@@ -2,11 +2,12 @@
   <% @projects_tree.each do |project| %>
     <% cache [action_name, current_user, project] do %>
       <li data-parent="candidate">
-        <span class="tree-link line-wrap no-indent">
+        <span class="tree-link no-indent" title="<%= project.name %>">
           <i class="no-arrow"></i>
             <%= link_to project.name,
                         project_action_to_link_to(project),
                         title: project.name,
+                        class: 'line-wrap',
                         data: { type: 'project', id: project.id } %>
         </span>
         <%= render partial: 'shared/sidebar/experiments', locals: { project: project } %>