Some feedback on the issues page

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 07a9451d12a8e6f581750af599f876101dd60588..42447410947a9321231f4490cf48c7cb869c694d 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
@@ -144,6 +144,7 @@ technical_debt=Technical Debt
 template=Template
 title=Title
 to=To
+to.downcase=to
 treemap=Treemap
 true=True
 type=Type

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 07c7546c33d9672e0a3980dfe443cd7d90bdb3a5..d283763dd6975de726d21fca9fd2bade5e04fff2 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
@@ -52,10 +52,12 @@
           <li><%= message('assigned_to') -%> <strong><%= h @issue_results.user(issue.assignee).name -%></strong></li>
         <% end %>
       <% elsif current_user %>
-        <li><a href='#' onclick="return issueForm('assign', this)" class="link-action"><%= message('issue.assign.formlink') -%></a></li>
-        <% if issue.assignee != current_user.login %>
-          <li>[<a href="#" onclick="return assignIssueToMe(this)" class="link-action"><%= message('issue.assign.to_me') -%></a>]</li>
-        <% end %>
+        <li>
+          <a href='#' onclick="return issueForm('assign', this)" class="link-action"><%= message('issue.assign.formlink') -%></a>
+          <% if issue.assignee != current_user.login %>
+            [<a href="#" onclick="return assignIssueToMe(this)" class="link-action"><%= message('issue.assign.to_me') -%></a>]
+          <% end %>
+        </li>
       <% end %>
       <% if issue.actionPlanKey %>
         <% if current_user %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/issues/search.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/issues/search.html.erb
index da0af92c124fedf8014eee52d5c4161f668f3ec0..d0c89a2b112d93d08c033e0be6fb7147b8a55a9b 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/issues/search.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/issues/search.html.erb
@@ -48,7 +48,7 @@
       'all':           '<%= escape_javascript message('all') -%>',
       'edit':          '<%= escape_javascript message('edit') -%>',
       'delete':        '<%= escape_javascript message('delete') -%>',
-      'to':            '<%= escape_javascript message('to') -%>',
+      'to':            '<%= escape_javascript message('to.downcase') -%>',
       'project':       '<%= escape_javascript message('issue_filter.criteria.project') -%>',
       'severity':      '<%= escape_javascript message('issue_filter.criteria.severity') -%>',
       'severities': {

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/measures/search.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/measures/search.html.erb
index 709e1caf4d38b217f18b99f1f101b46148a3a894..8885e6806e7c8bb969de5d7177ffc7e06dc22a9a 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/measures/search.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/measures/search.html.erb
@@ -66,7 +66,7 @@
       'anytime':        '<%= escape_javascript message('anytime') -%>',
       'all':            '<%= escape_javascript message('all') -%>',
       'manage':         '<%= escape_javascript message('manage') -%>',
-      'to':             '<%= escape_javascript message('to') -%>',
+      'to':             '<%= escape_javascript message('to.downcase') -%>',
       'value':          '<%= escape_javascript message('value') -%>',
       'components':     '<%= escape_javascript message('measure_filter.criteria.components') -%>',
       'age':            '<%= escape_javascript message('measure_filter.criteria.age') -%>',

diff --git a/sonar-server/src/main/webapp/javascripts/navigator/issues.js b/sonar-server/src/main/webapp/javascripts/navigator/issues.js
index d4111c6447bf5993cb074a62c9610db03363ad28..7cc91ce5e04df61866f37b7b5c4c7a8647b19518 100644 (file)
--- a/sonar-server/src/main/webapp/javascripts/navigator/issues.js
+++ b/sonar-server/src/main/webapp/javascripts/navigator/issues.js
@@ -556,8 +556,16 @@ jQuery(function() {
 
 
     onRender: function() {
+      var format = function(state) {
+        if (!state.id) return state.text; // optgroup
+        return '<i class="icon-severity-' + state.id.toLowerCase() + '"></i>' + state.text;
+      }
+
       this.ui.select.select2({
-        minimumResultsForSearch: 100
+        minimumResultsForSearch: 100,
+        formatResult: format,
+        formatSelection: format,
+        escapeMarkup: function(m) { return m; }
       });
     },
 

diff --git a/sonar-server/src/main/webapp/stylesheets/navigator.css b/sonar-server/src/main/webapp/stylesheets/navigator.css
index fca7d52c52cdb711cb4f1bd4c45b347f58f51694..2847ce7adae5b4d1fa10aac4850008d2a40ca772 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/navigator.css
+++ b/sonar-server/src/main/webapp/stylesheets/navigator.css
@@ -108,12 +108,14 @@
   position: relative;
 }
 .navigator-fetching#tab-issue-rule:before {
+  z-index: 3;
   background-color: #EFEFEF;
 }
 .navigator-fetching.code-issue-actions {
   position: relative;
 }
 .navigator-fetching.code-issue-actions:before {
+  z-index: 3;
   background-color: #E4ECF3;
 }
 .navigator-header {
@@ -269,7 +271,7 @@
 .navigator-details .source {
   padding-right: 10px;
 }
-.navigator-details .source table {
+.navigator-details .source > table {
   border: 1px solid #DDD;
 }
 .navigator-details .scm .author {

diff --git a/sonar-server/src/main/webapp/stylesheets/navigator/base.css b/sonar-server/src/main/webapp/stylesheets/navigator/base.css
index 3e37f544611ffd8ef3e774b3a2716db2f8d3fd5d..c86ddfc501dee8a42f062812dcb8df38fbf270b2 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/navigator/base.css
+++ b/sonar-server/src/main/webapp/stylesheets/navigator/base.css
@@ -108,12 +108,14 @@
   position: relative;
 }
 .navigator-fetching#tab-issue-rule:before {
+  z-index: 3;
   background-color: #EFEFEF;
 }
 .navigator-fetching.code-issue-actions {
   position: relative;
 }
 .navigator-fetching.code-issue-actions:before {
+  z-index: 3;
   background-color: #E4ECF3;
 }
 .navigator-header {
@@ -269,7 +271,7 @@
 .navigator-details .source {
   padding-right: 10px;
 }
-.navigator-details .source table {
+.navigator-details .source > table {
   border: 1px solid #DDD;
 }
 .navigator-details .scm .author {

diff --git a/sonar-server/src/main/webapp/stylesheets/navigator/base.less b/sonar-server/src/main/webapp/stylesheets/navigator/base.less
index ea7de94df0dea38b5c15e55c29725700808b36ff..73623b8bdc995f7428eaa6d883a83bc98d1e3978 100644 (file)
--- a/sonar-server/src/main/webapp/stylesheets/navigator/base.less
+++ b/sonar-server/src/main/webapp/stylesheets/navigator/base.less
@@ -108,6 +108,7 @@
     position: relative;
 
     &:before {
+      z-index: 3;
       background-color: #EFEFEF;
     }
   }
@@ -116,6 +117,7 @@
     position: relative;
 
     &:before {
+      z-index: 3;
       background-color: #E4ECF3;
     }
   }
@@ -321,7 +323,7 @@
   .source {
     padding-right: 10px;
 
-    table {
+    & > table {
       border: 1px solid #DDD;
     }
   }