Fix discussion reply holder padding

3 files changed, 6 insertions, 14 deletions

diff --git a/app/assets/stylesheets/framework/timeline.scss b/app/assets/stylesheets/framework/timeline.scss
index 3d5208c3db5..233aada9c38 100644
--- a/app/assets/stylesheets/framework/timeline.scss
+++ b/app/assets/stylesheets/framework/timeline.scss
@@ -36,6 +36,7 @@
   &:target,
   &.target {
     background: $line-target-blue;
+    margin-bottom: $gl-padding;
 
     &.system-note .note-body .note-text.system-note-commit-list::after {
       background: linear-gradient(rgba($line-target-blue, 0.1) -100px, $line-target-blue 100%);
diff --git a/app/assets/stylesheets/pages/note_form.scss b/app/assets/stylesheets/pages/note_form.scss
index 011fdf63fb7..05cc52e9e5e 100644
--- a/app/assets/stylesheets/pages/note_form.scss
+++ b/app/assets/stylesheets/pages/note_form.scss
@@ -178,7 +178,7 @@
 
 table {
   .discussion-form-container {
-    padding: $gl-padding-top $gl-padding $gl-padding;
+    padding: $gl-padding;
   }
 }
 
@@ -236,22 +236,17 @@ table {
   }
 }
 
-.discussion-body {
+.discussion-body,
+.commit-diff {
   .discussion-reply-holder {
     background-color: $white-light;
     border-radius: 0 0 3px 3px;
-    padding: 0 $gl-padding 10px;
+    padding: $gl-padding;
 
     &.is-replying {
       padding-bottom: $gl-padding;
     }
   }
-
-  .diff-file {
-    .discussion-reply-holder {
-      padding: 10px 16px;
-    }
-  }
 }
 
 .discussion-with-resolve-btn {
diff --git a/app/assets/stylesheets/pages/notes.scss b/app/assets/stylesheets/pages/notes.scss
index d40c34b2402..42ba7fcc20c 100644
--- a/app/assets/stylesheets/pages/notes.scss
+++ b/app/assets/stylesheets/pages/notes.scss
@@ -14,7 +14,7 @@ $note-form-margin-left: 72px;
 }
 
 @mixin outline-comment() {
-  margin: $gl-padding;
+  margin: $gl-padding $gl-padding 0;
   border: 1px solid $border-color;
   border-radius: $border-radius-default;
 }
@@ -526,10 +526,6 @@ $note-form-margin-left: 72px;
   .notes_content {
     background-color: $white-light;
   }
-
-  .discussion-reply-holder {
-    border-top: 1px solid $border-color;
-  }
 }
 
 .discussion-header,