Improve search results output. Fixes some markup issues

app/assets/stylesheets/sections/search.scss

+.search-results {
+  .search-result-row {
+    border-bottom: 1px solid #EEE;
+    padding-bottom: 10px;
+    margin-bottom: 10px;
+  }
+}

app/helpers/search_helper.rb

@@ -106,6 +106,6 @@ module SearchHelper
 
   # Sanitize html generated after parsing markdown from issue description or comment
   def search_md_sanitize(html)
-    sanitize(html, tags: %w(a p ul li pre code))
+    sanitize(html, tags: %w(a p ol ul li pre code))
   end
 end

app/views/search/_global_results.html.haml → app/views/search/_global_filter.html.haml

-.row
-  .col-sm-3
-    %ul.nav.nav-pills.nav-stacked.search-filter
-      %li{class: ("active" if @scope == 'projects')}
-        = link_to search_filter_path(scope: 'projects') do
-          Projects
-          .pull-right
-            = @search_results.projects_count
-      %li{class: ("active" if @scope == 'issues')}
-        = link_to search_filter_path(scope: 'issues') do
-          Issues
-          .pull-right
-            = @search_results.issues_count
-      %li{class: ("active" if @scope == 'merge_requests')}
-        = link_to search_filter_path(scope: 'merge_requests') do
-          Merge requests
-          .pull-right
-            = @search_results.merge_requests_count
-
-  .col-sm-9
-    .search_results
-      - if @search_results.empty?
-        = render partial: "search/results/empty", locals: { message: "We couldn't find any matching results" }
-
-      %ul.bordered-list.top-list
-        = render partial: "search/results/#{@scope.singularize}", collection: @objects
-      = paginate @objects, theme: 'gitlab'
+%ul.nav.nav-pills.nav-stacked.search-filter
+  %li{class: ("active" if @scope == 'projects')}
+    = link_to search_filter_path(scope: 'projects') do
+      Projects
+      .pull-right
+        = @search_results.projects_count
+  %li{class: ("active" if @scope == 'issues')}
+    = link_to search_filter_path(scope: 'issues') do
+      Issues
+      .pull-right
+        = @search_results.issues_count
+  %li{class: ("active" if @scope == 'merge_requests')}
+    = link_to search_filter_path(scope: 'merge_requests') do
+      Merge requests
+      .pull-right
+        = @search_results.merge_requests_count

app/views/search/_project_results.html.haml → app/views/search/_project_filter.html.haml

-.row
-  .col-sm-3
-    %ul.nav.nav-pills.nav-stacked.search-filter
-      %li{class: ("active" if @scope == 'blobs')}
-        = link_to search_filter_path(scope: 'blobs') do
-          %i.icon-code
-          Code
-          .pull-right
-            = @search_results.blobs_count
-      %li{class: ("active" if @scope == 'issues')}
-        = link_to search_filter_path(scope: 'issues') do
-          %i.icon-exclamation-sign
-          Issues
-          .pull-right
-            = @search_results.issues_count
-      %li{class: ("active" if @scope == 'merge_requests')}
-        = link_to search_filter_path(scope: 'merge_requests') do
-          %i.icon-code-fork
-          Merge requests
-          .pull-right
-            = @search_results.merge_requests_count
-      %li{class: ("active" if @scope == 'notes')}
-        = link_to search_filter_path(scope: 'notes') do
-          %i.icon-comments
-          Comments
-          .pull-right
-            = @search_results.notes_count
-
-  .col-sm-9
-    .search_results
-      - if @search_results.empty?
-        = render partial: "search/results/empty", locals: { message: "We couldn't find any matching results" }
-
-      %ul.bordered-list.top-list
-        = render partial: "search/results/#{@scope.singularize}", collection: @objects
-      = paginate @objects, theme: 'gitlab'
+%ul.nav.nav-pills.nav-stacked.search-filter
+  %li{class: ("active" if @scope == 'blobs')}
+    = link_to search_filter_path(scope: 'blobs') do
+      %i.icon-code
+      Code
+      .pull-right
+        = @search_results.blobs_count
+  %li{class: ("active" if @scope == 'issues')}
+    = link_to search_filter_path(scope: 'issues') do
+      %i.icon-exclamation-sign
+      Issues
+      .pull-right
+        = @search_results.issues_count
+  %li{class: ("active" if @scope == 'merge_requests')}
+    = link_to search_filter_path(scope: 'merge_requests') do
+      %i.icon-code-fork
+      Merge requests
+      .pull-right
+        = @search_results.merge_requests_count
+  %li{class: ("active" if @scope == 'notes')}
+    = link_to search_filter_path(scope: 'notes') do
+      %i.icon-comments
+      Comments
+      .pull-right
+        = @search_results.notes_count

app/views/search/_results.html.haml

@@ -7,10 +7,19 @@
 
 %hr
 
-- if @project
-  = render "project_results"
-- else
-  = render "global_results"
+.row
+  .col-sm-3
+    - if @project
+      = render "project_filter"
+    - else
+      = render "global_filter"
+  .col-sm-9
+    .search-results
+      - if @search_results.empty?
+        = render partial: "search/results/empty", locals: { message: "We couldn't find any matching results" }
+      - else
+        = render partial: "search/results/#{@scope.singularize}", collection: @objects
+        = paginate @objects, theme: 'gitlab'
 
 :javascript
-  $(".search_results .term").highlight("#{escape_javascript(params[:search])}");
+  $(".search-results .term").highlight("#{escape_javascript(params[:search])}");

app/views/search/results/_issue.html.haml

-%li
+.search-result-row
   %h4
     = link_to [issue.project, issue] do
       %span.term.str-truncated= issue.title

app/views/search/results/_merge_request.html.haml

-%li
+.search-result-row
   %h4
     = link_to [merge_request.target_project, merge_request] do
       %span.term.str-truncated= merge_request.title

app/views/search/results/_note.html.haml

 - project = note.project
-%li
-  %h5.note-search-caption
+.search-result-row
+  %h5.note-search-caption.str-truncated
     %i.icon-comment
     = link_to_member(project, note.author, avatar: false)
     commented on

app/views/search/results/_project.html.haml

-%li
+.search-result-row
   %h4
     = link_to project do
       %span.term= project.name_with_namespace