From 6fe71823302136b7dd4b199e194440a77e809070 Mon Sep 17 00:00:00 2001
From: Julien Lancelot <julien.lancelot@gmail.com>
Date: Thu, 22 Nov 2012 12:45:57 +0100
Subject: [PATCH] SONAR-3786 Remove the collapse/expand link and add spaced
 above the exception message

---
 .../plugins/core/web/tests_viewer.html.erb    | 22 +++++++++----------
 .../src/main/webapp/javascripts/resource.js   | 12 +++++-----
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb b/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb
index 531c1d8e8e7..af3909f8a61 100644
--- a/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb
+++ b/plugins/sonar-core-plugin/src/main/resources/org/sonar/plugins/core/web/tests_viewer.html.erb
@@ -34,7 +34,6 @@
   <table class="data width100 tests_viewer"  id="test_data">
   <thead>
     <tr>
-      <th class="thin">&nbsp;</th>
       <th class="thin">&nbsp;</th>
       <th class="thin right"><%= message('test_viewer.duration') -%></th>
       <th><%= message('test_viewer.unit_test_name') -%></th>
@@ -65,22 +64,23 @@
          failure = block.elements['failure']
          block_message = !failure.nil? ? failure : block_message
       %>
-      <td class="thin" nowrap>
-        <% if !block_message.nil? %>
-          <a id="expandLink_<%= index -%>" href="#" onclick="expandTests(<%= index -%>);"><%= message('test_viewer.expand') -%></a>
-          <a id="collapseLink_<%= index -%>" class="collapse" href="#" onclick="collapseTests(<%= index -%>);" style="display: none;"><%= message('test_viewer.collapse') -%></a>
-        <% else %>
-          &nbsp;
-        <% end %>
-      </td>
       <td class="thin" nowrap>
         <img id="test_status_<%= index -%>_<%= status -%>" src="<%= icon_url -%>" />
       </td>
       <td id="test_time_<%= index -%>" class="thin right" nowrap><%= block.attributes['time'] -%> ms</td>
       <td>
-        <span><%= block.attributes['name'] -%></span>
+        <span id="test_name_<%= index -%>">
+          <% name = block.attributes['name'] %>
+          <% if !block_message.nil? %>
+            <a id="test_expandLink_<%= index -%>" href="#" onclick="expandTests(<%= index -%>);"><%= name -%></a>
+            <a id="test_collapseLink_<%= index -%>" class="collapse" href="#" onclick="collapseTests(<%= index -%>);" style="display: none;"><%= name -%></a>
+          <% else %>
+            <%= name -%>
+          <% end %>
+        </span>
+
         <% if !block_message.nil? %>
-          <div id="message_<%= index -%>" class="message" style="display: none;">
+          <div id="test_message_<%= index -%>" class="message" style="display:none; margin-top:5px;">
             <% message = block_message.attributes['message'] %>
             <% if !message.nil? %>
               <span><%= h message -%></span>
diff --git a/sonar-server/src/main/webapp/javascripts/resource.js b/sonar-server/src/main/webapp/javascripts/resource.js
index 7b4327dfd47..18d32d2f519 100644
--- a/sonar-server/src/main/webapp/javascripts/resource.js
+++ b/sonar-server/src/main/webapp/javascripts/resource.js
@@ -187,13 +187,13 @@ function hVF(line) {
  Functions used in tests viewer
  */
 function expandTests(index){
-  $j(".tests_viewer #collapseLink_"+ index).show();
-  $j(".tests_viewer #expandLink_"+ index).hide();
-  $j(".tests_viewer #message_"+ index).show();
+  $j(".tests_viewer #test_collapseLink_"+ index).show();
+  $j(".tests_viewer #test_expandLink_"+ index).hide();
+  $j(".tests_viewer #test_message_"+ index).show();
 }
 
 function collapseTests(index){
-  $j(".tests_viewer #collapseLink_"+ index).hide();
-  $j(".tests_viewer #expandLink_"+ index).show();
-  $j(".tests_viewer #message_"+ index).hide();
+  $j(".tests_viewer #test_collapseLink_"+ index).hide();
+  $j(".tests_viewer #test_expandLink_"+ index).show();
+  $j(".tests_viewer #test_message_"+ index).hide();
 }
\ No newline at end of file
-- 
2.39.5