diff --git a/app/helpers/search_helper.rb b/app/helpers/search_helper.rb index fff850c1fbb359731fcdb20c926d0dce8f72a15b..51b6812cac3624eda2ff7dafa7dbd84cf7ae276f 100644 --- a/app/helpers/search_helper.rb +++ b/app/helpers/search_helper.rb @@ -103,4 +103,9 @@ module SearchHelper options = exist_opts.merge(options) search_path(options) end + + # 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)) + end end diff --git a/app/views/search/_global_results.html.haml b/app/views/search/_global_results.html.haml index 0225491e21589ca8320224205b480a8357ad9ced..cedb6b249bb4009627e334b27878cad902e4193a 100644 --- a/app/views/search/_global_results.html.haml +++ b/app/views/search/_global_results.html.haml @@ -22,6 +22,6 @@ - if @search_results.empty? = render partial: "search/results/empty", locals: { message: "We couldn't find any matching results" } - %ul.bordered-list + %ul.bordered-list.top-list = render partial: "search/results/#{@scope.singularize}", collection: @objects = paginate @objects, theme: 'gitlab' diff --git a/app/views/search/_project_results.html.haml b/app/views/search/_project_results.html.haml index dd63d5d04cb9951f238c6f17a20559c710040c4e..fc047637b35bfd997f447bd95bb8abd6fc6057f9 100644 --- a/app/views/search/_project_results.html.haml +++ b/app/views/search/_project_results.html.haml @@ -31,6 +31,6 @@ - if @search_results.empty? = render partial: "search/results/empty", locals: { message: "We couldn't find any matching results" } - %ul.bordered-list + %ul.bordered-list.top-list = render partial: "search/results/#{@scope.singularize}", collection: @objects = paginate @objects, theme: 'gitlab' diff --git a/app/views/search/results/_note.html.haml b/app/views/search/results/_note.html.haml index 97e892bdd4da9a45c8d08b512c3dae9802f44f9a..6316212bc9061bba54669acadd5bf49fac19787b 100644 --- a/app/views/search/results/_note.html.haml +++ b/app/views/search/results/_note.html.haml @@ -1,9 +1,26 @@ +- project = note.project %li - note on issue: - = link_to [note.project, note.noteable] do - %span ##{note.noteable.iid} - %strong.term - = truncate note.noteable.title, length: 50 - %span.light (#{note.project.name_with_namespace}) - - if note.noteable.closed? - %span.label Closed + %h5.note-search-caption + %i.icon-comment + = link_to_member(project, note.author, avatar: false) + commented on + + - if note.for_commit? + = link_to project do + = project.name_with_namespace + · + = link_to project_commit_path(project, note.commit_id, anchor: dom_id(note)) do + Commit #{note.commit_id[0..8]} + - else + = link_to project do + = project.name_with_namespace + · + %span #{note.noteable_type.titleize} ##{note.noteable.iid} + · + = link_to [project, note.noteable, anchor: dom_id(note)] do + = note.noteable.title + + .note-search-result + .term + = preserve do + = search_md_sanitize(markdown(note.note, {no_header_anchors: true})) diff --git a/app/views/search/results/_project.html.haml b/app/views/search/results/_project.html.haml index abc86c72bef1fb53cf5b0d9f91f90d57a1c26f7e..8c8baab8a5b5d1db8a345b66e2e16661d250d9f4 100644 --- a/app/views/search/results/_project.html.haml +++ b/app/views/search/results/_project.html.haml @@ -1,7 +1,6 @@ %li - project: - = link_to project do - %strong.term= project.name_with_namespace + %h4 + = link_to project do + %span.term= project.name_with_namespace - if project.description.present? - – %span.light.term= project.description