From 469bda166f41743c80bc3d615b8b83dff9f22ce5 Mon Sep 17 00:00:00 2001
From: Fabrice Bellingard <fabrice.bellingard@sonarsource.com>
Date: Fri, 28 Sep 2012 13:20:47 +0200
Subject: [PATCH] SONAR-968 Highlight lowest level in resource explorer when
 selected

---
 .../webapp/WEB-INF/app/views/drilldown/measures.html.erb   | 7 ++++---
 .../webapp/WEB-INF/app/views/drilldown/violations.html.erb | 7 ++++---
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/measures.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/measures.html.erb
index c3848b0c1ed..5e0f823f381 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/measures.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/measures.html.erb
@@ -55,20 +55,21 @@
     <td class="<%= 'spacer-left' unless first_column -%>" nowrap>
       <div class="scrollable" id="col_<%= index -%>">
         <table class="spaced">
-          <% column.measures.each do |measure|
+          <% column.measures.each_with_index do |measure, row_index|
             resource=column.resource(measure)
             selected = column.selected_snapshot && column.selected_snapshot.project_id==resource.id
             clazz = cycle("even", "odd", :name => "col_#{index}")
             clazz = clazz + ' selected' if selected
           %>
-            <tr class="<%= clazz -%>">
+            <tr class="<%= clazz -%>" id="row_<%= index -%>_<%= row_index -%>">
               <td nowrap>
                 <% unless resource.source_code? %>
                   <%= link_to(image_tag('zoom.png'), {:id => resource.id, :metric => @metric.id}, {:class => 'nolink'}) -%>
                 <% end %>
                 <%= qualifier_icon(resource) -%>&nbsp;
                 <% if resource.source_code? %>
-                  <a href="#" onclick="d(<%= resource.id -%>)" alt="<%= resource.name(true) -%>" title="<%= resource.name(true) -%>"><%= resource.name(false) -%></a>
+                  <a href="#" onclick="$$('#col_<%= index -%> tr').invoke('removeClassName', 'selected'); $('row_<%= index -%>_<%= row_index -%>').addClassName('selected'); d(<%= resource.id -%>)" 
+                     alt="<%= resource.name(true) -%>" title="<%= resource.name(true) -%>"><%= resource.name(false) -%></a>
                 <% else %>
                   <%= link_to(resource.name, {:only_path => true, :overwrite_params => {:rids => (selected ? rids-[resource.id] : rids+[resource.id])}}) -%>
                 <% end %>
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/violations.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/violations.html.erb
index b79d399a1d1..2a44c0e730c 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/violations.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/violations.html.erb
@@ -129,7 +129,7 @@
     <div class="scrollable" id="col_<%= index -%>">
       <table class="spaced">
         <%
-           column.measures.each do |measure|
+           column.measures.each_with_index do |measure, row_index|
              resource=column.resource(measure)
              clazz = cycle('even', 'odd', :name => "col_#{index}")
              selected = column.selected_snapshot && column.selected_snapshot.project_id==resource.id
@@ -138,14 +138,15 @@
                paths << [h(resource.name), @selected_rids-[resource.id]]
              end
         %>
-          <tr class="<%= clazz -%>">
+          <tr class="<%= clazz -%>" id="row_<%= index -%>_<%= row_index -%>">
             <td nowrap>
               <% unless resource.source_code? %>
                 <%= link_to(image_tag('zoom.png'), {:id => resource.id}, {:class => 'nolink'}) %>
               <% end %>
               <%= qualifier_icon(resource) -%>&nbsp;
               <% if resource.source_code? %>
-                <a href="#" onclick="d(<%= resource.id -%>, '<%= @period_index -%>', '<%= @rule ? @rule.key : @severity -%>');" alt="<%= resource.name(true) -%>" title="<%= resource.name(true) -%>"><%= resource.name(false) %></a>
+                <a href="#" onclick="$$('#col_<%= index -%> tr').invoke('removeClassName', 'selected'); $('row_<%= index -%>_<%= row_index -%>').addClassName('selected'); d(<%= resource.id -%>, '<%= @period_index -%>', '<%= @rule ? @rule.key : @severity -%>');" 
+                   alt="<%= resource.name(true) -%>" title="<%= resource.name(true) -%>"><%= resource.name(false) %></a>
               <% else %>
                 <%= link_to(h(resource.name), {:only_path => true, :overwrite_params => {:rids => (selected ? rids-[resource.id] : rids+[resource.id])}}) -%>
               <% end %>
-- 
2.39.5