Revert "refactor source viewer (#1705)"

This reverts commit ce9f0892fc3d15638c4eaa4054ed06f3d7e5fc19.

4 files changed, 20 insertions, 23 deletions

diff --git a/server/sonar-web/src/main/less/components/issues.less b/server/sonar-web/src/main/less/components/issues.less
index 8202664e4cc..b56858fe641 100644
--- a/server/sonar-web/src/main/less/components/issues.less
+++ b/server/sonar-web/src/main/less/components/issues.less
@@ -50,8 +50,7 @@
   border-color: @issueBorderColor !important;
 }
 
-.issue + .issue,
-.issue-container + .issue-container {
+.issue + .issue {
   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 9a89d87959a..18a2cfa5d1c 100644
--- a/server/sonar-web/src/main/less/components/source.less
+++ b/server/sonar-web/src/main/less/components/source.less
@@ -143,14 +143,6 @@
   user-select: none;
 }
 
-.source-meta:focus {
-  outline: none;
-}
-
-.source-meta[role="button"] {
-  cursor: pointer;
-}
-
 .source-meta + .source-meta {
   border-left: 1px solid @barBackgroundColor;
 }
@@ -162,6 +154,10 @@
   color: @secondFontColor;
   text-align: right;
 
+  &[data-line-number] {
+    cursor: pointer;
+  }
+
   &:before {
     content: attr(data-line-number);
   }
@@ -211,6 +207,10 @@
 .source-line-scm {
   padding: 0 5px;
   background-color: @barBackgroundColor;
+
+  &[data-line-number] {
+    cursor: pointer;
+  }
 }
 
 .source-line-scm-inner {
@@ -229,21 +229,29 @@
   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;
 }
 
 
diff --git a/server/sonar-web/src/main/less/pages/issues.less b/server/sonar-web/src/main/less/pages/issues.less
index 98bf375c2ea..fd1ddd7431e 100644
--- a/server/sonar-web/src/main/less/pages/issues.less
+++ b/server/sonar-web/src/main/less/pages/issues.less
@@ -45,15 +45,11 @@
   padding: 0 10px;
 }
 
-.issues-workspace-list-item + .issues-workspace-list-item {
-  margin-top: 5px;
-}
-
-.issues-workspace-list-component + .issues-workspace-list-item {
+.issues-workspace-list-component + .issue {
   margin-top: 10px;
 }
 
-.issues-workspace-list-item + .issues-workspace-list-component {
+.issue + .issues-workspace-list-component {
   margin-top: 25px;
 }
 
diff --git a/server/sonar-web/src/main/less/sonar-colorizer.less b/server/sonar-web/src/main/less/sonar-colorizer.less
index 760aec28453..325bcad8177 100644
--- a/server/sonar-web/src/main/less/sonar-colorizer.less
+++ b/server/sonar-web/src/main/less/sonar-colorizer.less
@@ -70,11 +70,5 @@
   cursor: pointer;
 }
 .highlighted {
-  background-color: #b3d4ff;
-  animation: highlightedFadeIn 0.3s forwards;
-}
-
-@keyframes highlightedFadeIn {
-  from { background-color: transparent; }
-  to { background-color: #b3d4ff; }
+  background-color: #B3D4FF;
 }