improve labels of diff views

diff --git a/plugins/sonar-core-gwt/src/main/java/org/sonar/plugins/core/violationsviewer/client/I18nConstants.java b/plugins/sonar-core-gwt/src/main/java/org/sonar/plugins/core/violationsviewer/client/I18nConstants.java
index 4d2fa97d408447ccd9b18279a33b91b73bdc166d..588852943e6b5ca745f772c3c57bfcf29a9f201a 100644 (file)
--- a/plugins/sonar-core-gwt/src/main/java/org/sonar/plugins/core/violationsviewer/client/I18nConstants.java
+++ b/plugins/sonar-core-gwt/src/main/java/org/sonar/plugins/core/violationsviewer/client/I18nConstants.java
@@ -31,7 +31,7 @@ public interface I18nConstants extends com.google.gwt.i18n.client.Constants {
   @DefaultStringValue("No filters")
   String noFilters();
 
-  @DefaultStringValue("Added...")
+  @DefaultStringValue("Time changes...")
   String addedPeriod();
 
   @DefaultStringValue("Expand:")

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/helpers/application_helper.rb b/sonar-server/src/main/webapp/WEB-INF/app/helpers/application_helper.rb
index a62478a85fcde7341dffd0c92109362595724f8f..c5f61b8c6c43ef704c8bff6d97c063f768b525df 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/helpers/application_helper.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/helpers/application_helper.rb
@@ -475,6 +475,7 @@ module ApplicationHelper
   # === Optional parameters
   # * color: true|false. Default is true.
   # * index: integer between 1 and 5. By default the index is defined by the dashboard variation select-box
+  # * style: light|normal. Default is normal (parenthesis + bold)
   #
   # === Examples
   # format_variation('ncloc')
@@ -505,7 +506,10 @@ module ApplicationHelper
             end
           end
         end
-        html="<span class='#{css_class}'>(#{formatted_val})</span>"
+        if options[:style]!='light'
+          formatted_val="<b>(#{formatted_val})</b>"
+        end
+        html="<span class='#{css_class}'>#{formatted_val}</span>"
       end
     end
     html

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/helpers/drilldown_helper.rb b/sonar-server/src/main/webapp/WEB-INF/app/helpers/drilldown_helper.rb
index 8cdfaf522e2167e23c9385614ea1f5b0ebd524d2..fcab616168a4cfe70056536bceff4ba852bbfc18 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/helpers/drilldown_helper.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/helpers/drilldown_helper.rb
@@ -26,7 +26,7 @@ module DrilldownHelper
 
     if mode
       if mode=='days'
-        label = "Added over %s previous days" % mode_param
+        label = "Added over %s days" % mode_param
       elsif mode=='version'
         label = "Added since version %s" % mode_param
       elsif mode=='previous_analysis'

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb
index ce47937e37c36436e322fc32c2a2ce7a14a1d348..8174a2a8e6d2c753187173fbade1c3ea85f04951 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb
@@ -21,7 +21,7 @@
     -
     <form method="GET" action="<%= url_for :only_path=>true, :overwrite_params => {:period => nil} -%>" style="display: inline">
     <select id="select-comparison" name="period" onchange="submit()" class="small">
-      <option value="">Variations...</option>
+      <option value="">Time changes...</option>
       <%= period_select_options(@snapshot, 1) -%>
       <%= period_select_options(@snapshot, 2) -%>
       <%= period_select_options(@snapshot, 3) -%>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/_rule_priority.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/_rule_priority.erb
index e9e8533335724cc9e8c258a795e2f6ba2fba9bb3..aff4f99f88ab9eb122d5a02fd6f6e4f1186e4462 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/_rule_priority.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/drilldown/_rule_priority.erb
@@ -2,10 +2,10 @@
        <td><%= image_tag "priority/#{priority_id}.png" %></td>
   <td><%= link_to label, {:controller => 'drilldown', :action => 'violations', :id => @project.id, :priority => Sonar::RulePriority.to_s(priority_id), :period => @period_index} %></td>
   <td style="padding-left: 10px;" align="right">
-         <%= @period_index ? format_variation(measure, :index => @period_index) : format_measure(measure) -%>
+         <%= @period_index ? format_variation(measure, :index => @period_index, :style => 'light') : format_measure(measure) -%>
   </td>
   <td align="left">
     <% value = measure_or_variation_value(measure) %>
-    <%= barchart(:width => 60, :percent => (value ? (100 * value / max).to_i : 0), :color => '#777') if max>0  %>
+    <%= barchart(:width => 60, :percent => (value ? (100 * value / max).to_i : 0), :color => (@period_index ? '#cc0000' : '#777')) if max>0  %>
   </td>
 </tr>
\ No newline at end of file

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 c813c23adaf18cc55c469c151bb9e37caad66a80..29fcff2eafa315a44059895f4f28b5edd2771ffc 100644 (file)
--- 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
@@ -8,7 +8,7 @@
 <% if @snapshot.project_snapshot.periods? %>
 - <form method="GET" action="<%= url_for :only_path=>true, :overwrite_params => {:period => nil} -%>" style="display: inline">
     <select id="select-comparison" name="period" onchange="submit()" class="small">
-      <option value="">All violations</option>
+      <option value="">Time changes...</option>
       <%= period_select_options(@snapshot, 1) -%>
       <%= period_select_options(@snapshot, 2) -%>
       <%= period_select_options(@snapshot, 3) -%>
@@ -103,10 +103,10 @@
                          <%= link_to(rule.name, {:overwrite_params => {:rule => rule.key, :sid => nil, :priority => Sonar::RulePriority.to_s(@priority_id)}}, :title => "#{rule.plugin_name}: #{rule.plugin_rule_key}") %>
                    </td>
                    <td class="right" nowrap="nowrap">
-                       <span><%= @period_index ? format_variation(rule_measure, :index => @period_index) : rule_measure.formatted_value -%></span>
+                       <span><%= @period_index ? format_variation(rule_measure, :index => @period_index, :style => 'light') : rule_measure.formatted_value -%></span>
                    </td>
                    <td class="left last">
-                         <%= barchart(:width => 70, :percent => (100 * value / max).to_i, :color => '#777')  if max>0 %>
+                         <%= barchart(:width => 70, :percent => (100 * value / max).to_i, :color => (@period_index ? '#cc0000' : '#777'))  if max>0 %>
                          <!--[if IE]> &nbsp; &nbsp; <![endif]-->
                       </td>
                </tr>
@@ -159,7 +159,7 @@
    <% end %>
    </td>
    <td class="right last" nowrap>
-       <%= @period_index ? format_variation(measure, :index => @period_index) : measure.formatted_value -%>
+       <%= @period_index ? format_variation(measure, :index => @period_index, :style => 'light') : measure.formatted_value -%>
        <!--[if IE]> &nbsp; &nbsp; <![endif]-->
    </td>
 </tr>

diff --git a/sonar-server/src/main/webapp/stylesheets/style.css b/sonar-server/src/main/webapp/stylesheets/style.css
index 2be561e7ec217c2dbd6ceb086fa2b0963841ed48..765e060731b5b0f6e6febd75c6df60242119e4d5 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/style.css
+++ b/sonar-server/src/main/webapp/stylesheets/style.css
@@ -888,15 +888,12 @@ ul.operations li a {
 
 /* ------------------- VARIATIONS ------------------- */
 .var {
-  font-weight: bold;
   color: #444 !important;
 }
 .varb {/* better */
-  font-weight: bold;
-  color: #078C00 !important;
+   color: #078C00 !important;
 }
 .varw {/* worst */
-  font-weight: bold;
   color: #cc0000 !important;
 }