refactor source viewer (#1705)

2 files changed, 10 insertions, 17 deletions

diff --git a/server/sonar-web/src/main/less/components/issues.less b/server/sonar-web/src/main/less/components/issues.less
index b56858fe641..8202664e4cc 100644
--- a/server/sonar-web/src/main/less/components/issues.less
+++ b/server/sonar-web/src/main/less/components/issues.less
@@ -50,7 +50,8 @@
   border-color: @issueBorderColor !important;
 }
 
-.issue + .issue {
+.issue + .issue,
+.issue-container + .issue-container {
   margin-top: 5px;
 }
 
diff --git a/server/sonar-web/src/main/less/components/source.less b/server/sonar-web/src/main/less/components/source.less
index 18a2cfa5d1c..9a89d87959a 100644
--- a/server/sonar-web/src/main/less/components/source.less
+++ b/server/sonar-web/src/main/less/components/source.less
@@ -143,6 +143,14 @@
   user-select: none;
 }
 
+.source-meta:focus {
+  outline: none;
+}
+
+.source-meta[role="button"] {
+  cursor: pointer;
+}
+
 .source-meta + .source-meta {
   border-left: 1px solid @barBackgroundColor;
 }
@@ -154,10 +162,6 @@
   color: @secondFontColor;
   text-align: right;
 
-  &[data-line-number] {
-    cursor: pointer;
-  }
-
   &:before {
     content: attr(data-line-number);
   }
@@ -207,10 +211,6 @@
 .source-line-scm {
   padding: 0 5px;
   background-color: @barBackgroundColor;
-
-  &[data-line-number] {
-    cursor: pointer;
-  }
 }
 
 .source-line-scm-inner {
@@ -229,29 +229,21 @@
   height: @source-line-height;
 }
 
-.source-line-with-issues {
-  cursor: pointer;
-}
-
 .source-line-covered {
   background-color: @green !important;
-  cursor: pointer;
 }
 
 .source-line-uncovered {
   background-color: @red !important;
-  cursor: pointer;
 }
 
 .source-line-partially-covered {
   background-color: @orange !important;
   background-image: repeating-linear-gradient(45deg, rgba(255, 255, 255, .5) 4px, transparent 4px, transparent 8px, rgba(255, 255, 255, .5) 8px, rgba(255, 255, 255, .5) 12px, transparent 12px, transparent 16px, rgba(255, 255, 255, .5) 16px, rgba(255, 255, 255, .5) 20px) !important;
-  cursor: pointer;
 }
 
 .source-line-duplicated {
   background-color: @duplicationColor !important;
-  cursor: pointer;
 }