SONAR-5490 Improve readability and payload of results in the Coding Rules page

Fix the issue with whitespace when there are no tags and no status

diff --git a/server/sonar-web/src/main/coffee/coding-rules/views/coding-rules-list-item-view.coffee b/server/sonar-web/src/main/coffee/coding-rules/views/coding-rules-list-item-view.coffee
index 92d7aa3aead33e5e7880b819520d10b2fe43ccac..0c77a630538495a6f1107a28f3fa4ade35375902 100644 (file)
--- a/server/sonar-web/src/main/coffee/coding-rules/views/coding-rules-list-item-view.coffee
+++ b/server/sonar-web/src/main/coffee/coding-rules/views/coding-rules-list-item-view.coffee
@@ -24,6 +24,8 @@ define [
 
 
     serializeData: ->
+      tags = _.union @model.get('sysTags'), @model.get('tags')
       _.extend super,
         manualRuleLabel: t 'coding_rules.manual_rule'
-        allTags: _.union @model.get('sysTags'), @model.get('tags')
+        allTags: tags
+        showDetails: (@model.get('status') != 'READY') || (_.isArray(tags) && tags.length > 0)

diff --git a/server/sonar-web/src/main/hbs/coding-rules/coding-rules-list-item.hbs b/server/sonar-web/src/main/hbs/coding-rules/coding-rules-list-item.hbs
index e8005ae79d2338a6b37f153378b871d2cf42b63d..afa53e8d33d7316fde5e864c92327f56f75c60b9 100644 (file)
--- a/server/sonar-web/src/main/hbs/coding-rules/coding-rules-list-item.hbs
+++ b/server/sonar-web/src/main/hbs/coding-rules/coding-rules-list-item.hbs
@@ -1,16 +1,18 @@
 <div class="line line-small">
   <span class="coding-rules-detail-status">{{default language manualRuleLabel}}</span>
 
-  <div class="line-right">
-    {{#if allTags}}
-      <span class="coding-rules-list-tags">
-        <i class="icon-tags"></i>
-        <span>{{join allTags ', '}}</span>
-      </span>
-      &nbsp;&nbsp;
-    {{/if}}
+  {{#if showDetails}}
+    <div class="line-right">
+      {{#if allTags}}
+        <span class="coding-rules-list-tags">
+          <i class="icon-tags"></i>
+          <span>{{join allTags ', '}}</span>
+        </span>
+        &nbsp;&nbsp;
+      {{/if}}
 
-    {{#notEq status 'READY'}}<span class="coding-rules-detail-not-ready">{{status}}</span>{{/notEq}}
-  </div>
+      {{#notEq status 'READY'}}<span class="coding-rules-detail-not-ready">{{status}}</span>{{/notEq}}
+    </div>
+  {{/if}}
 </div>
 <div class="line" title="{{name}}" name="{{key}}">{{name}}</div>