SONAR-4653 Provide new CSS styles for tables and titles in rule descriptions

diff --git a/plugins/sonar-core-plugin/src/main/resources/org/sonar/l10n/core.properties b/plugins/sonar-core-plugin/src/main/resources/org/sonar/l10n/core.properties
index ee093b5df8c90b0f5de4eb98d26437202460a197..74b4b87c2449d300fd2a5dc45f8cba15c3613635 100644 (file)
--- a/plugins/sonar-core-plugin/src/main/resources/org/sonar/l10n/core.properties
+++ b/plugins/sonar-core-plugin/src/main/resources/org/sonar/l10n/core.properties
@@ -1476,9 +1476,9 @@ rules_configuration.rule_deleted=Rule deleted
 rules_configuration.unknown_rule=Unknown rule
 rules_configuration.x_rules_have_been_activated={0} rules have been activated.
 rules_configuration.x_rules_have_been_deactivated={0} rules have been deactivated.
-rules_configuration.extend_description=Extend description
-rules_configuration.edit_note=Edit note
-rules_configuration.add_note=Add note
+rules_configuration.extend_description=Extend Description
+rules_configuration.edit_note=Edit Note
+rules_configuration.add_note=Add Note
 rules_configuration.confirm_delete_note=Do you really want to delete this note?
 rules_configuration.rule_inherited_from_profile_x=Rule inherited from profile "{0}"
 rules_configuration.rule_overriding_from_profile_x=Rule overrides parent definition from profile "{0}"

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/issue/_issue.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/issue/_issue.html.erb
index 6e1cb2eeeed9d728af81a9e0b1233a056aee013c..67097e935b4bcef9821e7ec0af7824dd7a8ce095 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/issue/_issue.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/issue/_issue.html.erb
@@ -37,7 +37,7 @@
     <% end %>
   </div>
 
-  <div class="issue-rule rule_detail" style="display: none"></div>
+  <div class="issue-rule rule-desc" style="display: none"></div>
   <div class="issue-changelog" id="issue-changelog" style="display: none"></div>
 
   <% unless issue.message.blank? %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules/_show_modal.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules/_show_modal.html.erb
index 71dcc6c944a495f559cff61eae5ef093504617f2..ea19f410b4e0488034f287f326e8b5c9af8404dc 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules/_show_modal.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules/_show_modal.html.erb
@@ -17,7 +17,7 @@
   </ul>
 </div>
 
-<div class="modal-body rule_detail">
+<div class="modal-body rule-desc">
   <% if @rule.description.strip.start_with?('<p>') %>
     <%= Internal.text.interpretMacros(@rule.description) %>
   <% else %>
@@ -42,6 +42,6 @@
 <script type="text/javascript">
   $j(document).ready(function () {
     // As links in rule description will be loaded after open-modal has been processed by jquery, we have to process manually rule-modal classes
-    $j('.rule_detail .rule-modal').modal();
+    $j('.rule-desc .rule-modal').modal();
   });
 </script>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules/show.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules/show.html.erb
index 2258fdd293eff3b05684487a1e46d0e348c1eddb..d437ae0a9ef2e757c83465b6d62d047ccb5a89bd 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules/show.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules/show.html.erb
@@ -15,7 +15,7 @@
 <%= h @rule.plugin_name -%> : <%= h @rule.plugin_rule_key -%>
 </div>
 
-<div class="rule_detail">
+<div class="rule-desc">
   <% if @rule.description.strip.start_with?('<p>') %>
     <%= Internal.text.interpretMacros(@rule.description) %>
   <% else %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule.html.erb
index a00f4f1f5b12f48f61adfcaeec54400a73eda4dc..7f005ead014c0ca8e32f149a74edccdc8196e86c 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule.html.erb
@@ -42,10 +42,10 @@
   end
   %></div>
 
-  <div id="desc_<%= rule.id -%>" class="rule_detail" style="<%= 'display:none' -%>">
+  <div id="desc_<%= rule.id -%>" class="rule-desc" style="<%= 'display:none' -%>">
     <a name="rule<%= rule.id -%>"></a>
 
-    <div class="spacer-top spacer-bottom" id="rule_note_<%= rule.id -%>">
+    <div id="rule_note_<%= rule.id -%>" class="marginbottom10">
       <%= render :partial => 'rule_note', :locals => {:rule => rule} %>
     </div>
 

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb
index 1b2a7f5646e5f0a4de940aeb5e482eb8c112ea52..16d0d6f21667df8467ac23962c17407b2a4da977 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_note.html.erb
@@ -8,7 +8,7 @@
 %>
 
 <div id="<%= note_detail_div_id -%>">
-  <div>
+  <div class="marginbottom10">
   <% if rule.description.strip.start_with?('<p>') %>
     <%= Internal.text.interpretMacros(rule.description) %>
   <% else %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb
index ba9dc32345515a81516203da0ebbe269b57387d1..2682494364c87c703d9294dcc5e8bb27ef002472 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb
@@ -24,11 +24,11 @@
   }
 
   function expand_rules(){
-     $j('.rule_detail').show();
+     $j('.rule-desc').show();
   }
 
   function collapse_rules(){
-    $j('.rule_detail').hide();
+    $j('.rule-desc').hide();
   }
 
   function submitSearch() {

diff --git a/sonar-server/src/main/webapp/stylesheets/style.css b/sonar-server/src/main/webapp/stylesheets/style.css
index 6750544429e420f3d32079070f2df396f5e742e7..047aafcd87c77026ddfffc8de73d245fe90d6a65 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/style.css
+++ b/sonar-server/src/main/webapp/stylesheets/style.css
@@ -1821,33 +1821,72 @@ ul.bullet li {
   color: #777;
 }
 
-.rule_detail ul li {
-  list-style: disc outside;
-  padding: 2px;
+
+/* API for Rule Descriptions */
+.rule-desc h2 {
+  margin-top: 16px;
+  font-size: 16px;
+  line-height: 1.5;
+  color: #2B547D;
 }
 
-.rule_detail ul, .rule_detail ol {
-  list-style: none outside;
-  padding-left: 30px;
+.rule-desc h3 {
+  margin-top: 16px;
+  font-size: 12px;
+  line-height: 1.5;
+  color: #2B547D;
+  font-weight: bold;
 }
 
-.rule_detail ol li {
-  list-style: decimal outside;
-  padding: 2px;
+.rule-desc p {
+  margin-top: 10px;
 }
 
-.rule_detail pre, .code .rule_detail pre {
+.rule-desc pre {
   margin: 10px 0;
-  padding: 7px;
-  font-family: "Courier New", Courier, monospace;;
+  padding: 10px;
   border: 1px dashed #aaa;
-  font-size: 93%;
+  font-size: 12px;
+  font-family: monospace;;
 }
 
-.rule_detail p {
-  margin: 8px 0;
+.rule-desc blockquote {
+  margin-top: 10px;
+  padding: 10px;
+}
+
+.rule-desc ul {
+  list-style-type: disc;
+  list-style-position: inside;
+  margin: 10px;
+}
+
+.rule-desc ol {
+  list-style-type: decimal;
+  list-style-position: inside;
+  margin: 10px;
 }
 
+.rule-table {
+  margin-top: 10px;
+  overflow-x: auto;
+  border-collapse: collapse;
+}
+.rule-table th {
+  background: none no-repeat scroll right center #EFEFEF;
+  border: 1px solid #DDD;
+  padding: 5px 10px;
+  font-weight: bold;
+}
+.rule-table td {
+  background: none no-repeat scroll right center transparent;
+  border: 1px solid #DDD;
+  padding: 5px 10px;
+}
+/* End of API for Rule Descriptions */
+
+
+
 .tip:hover {
   background: #FFF;
   position: relative;