Improve rendering of source code

* remove top and bottom borders from all the line numbers
* remove top and bottom borders from groups of violations
* remove the margin before source code. It can be mixed up with real whitespaces or tabulations in source code

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/resource/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/resource/index.html.erb
index 3da6bd30dc8d90c768e7888b7ae1e9cdeef4666c..fd76f70250014c6ff9799ecf2e884ebc6612dcbb 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/resource/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/resource/index.html.erb
@@ -79,7 +79,7 @@
     <%   end
        end
     %>
-    <td class="lid <%= ' section' if line.violations? -%>" id="L<%= index+1 -%>"><a name="L<%= index+1 -%>" href="#L<%= index+1 -%>"><%= index + 1 -%></a></td>
+    <td class="lid" id="L<%= index+1 -%>"><a name="L<%= index+1 -%>" href="#L<%= index+1 -%>"><%= index + 1 -%></a></td>
 
     <% if @display_coverage  %>
       <% if line.highlighted? %>
@@ -105,7 +105,7 @@
       <% if @scm_available %>
       <td class="scm"></td>
       <% end %>
-      <td class="lid section"></td>
+      <td class="lid"></td>
       <td class="violations">
         <% line.violations.each_with_index do |violation, index| %>
           <%= render :partial => 'violation', :locals => {:violation => violation} -%>

diff --git a/sonar-server/src/main/webapp/stylesheets/style.css b/sonar-server/src/main/webapp/stylesheets/style.css
index 53f223085c916ee89351de8b9463708d31927856..a48a8d4f8dea7ad1719c2097f1cc8b95b503adb3 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/style.css
+++ b/sonar-server/src/main/webapp/stylesheets/style.css
@@ -834,7 +834,6 @@ ul.operations li img {
   -o-user-select: none;
   user-select: none;
 }
-
 .sources2 td.lid a {
   text-decoration: none;
   color: #AAA;
@@ -950,7 +949,9 @@ span.rulename a:hover {
 .sources2 td.line pre {
   font-size: 12px;
   font-family: monospace;
-  margin-left: 1em;
+}
+.sources2 td.line:hover {
+  background-color: #ECECEC;
 }
 .sources2 td.ind {
   border-right: 1px solid #DDD;
@@ -989,9 +990,7 @@ span.rulename a:hover {
 .sources2 td.violations {
   background-color: #FFF;
   padding: 10px;
-  border-bottom: 1px solid #DDD;
   border-right: 1px solid #DDD;
-  border-top: 1px solid #DDD;
 }
 
 #source_title {