Merge branch '23557-remove-extra-line-for-empty-issue-description' into 'master'

This MR removes an odd line for empty description

Closes #23557 #23695

See merge request !7045
Signed-off-by: Rémy Coutable <remy@rymai.me>

CHANGELOG.md

@@ -8,6 +8,7 @@ Please view this file on the master branch, on stable branches it's out of date.
   - Fix issue boards user link when in subdirectory. !7018
   - Refactor and add new environment functionality to CI yaml reference. !7026
   - Fix typo in project settings that prevents users from enabling container registry. !7037
+  - Remove extra line for empty issue description. !7045
 
 ## 8.13.0 (2016-10-22)
 

app/assets/stylesheets/framework/common.scss

@@ -372,3 +372,5 @@ table {
   margin-right: -$gl-padding;
   border-top: 1px solid $border-color;
 }
+
+.hide-bottom-border { border-bottom: none !important; }

app/views/projects/issues/show.html.haml

@@ -53,7 +53,7 @@
 
 
 .issue-details.issuable-details
-  .detail-page-description.content-block
+  .detail-page-description.content-block{ class: ('hide-bottom-border' unless @issue.description.present? ) }
     %h2.title
       = markdown_field(@issue, :title)
     - if @issue.description.present?

app/views/projects/milestones/show.html.haml

@@ -31,7 +31,7 @@
         = link_to namespace_project_milestone_path(@project.namespace, @project, @milestone), data: { confirm: 'Are you sure?' }, method: :delete, class: "btn btn-grouped btn-danger" do
           Delete
 
-  .detail-page-description.milestone-detail
+  .detail-page-description.milestone-detail{ class: ('hide-bottom-border' unless @milestone.description.present? ) }
     %h2.title
       = markdown_field(@milestone, :title)
     %div