Merge branch 'spinner-spin' into 'master'

Spin icon next to "Checking for CI status..."

Stationary spinners make me sad.

See merge request !655

CHANGELOG

@@ -64,6 +64,7 @@ v 7.11.0 (unreleased)
   - Use SIGKILL by default in Sidekiq::MemoryKiller
   - Fix mentioning of private groups.
   - Add style for <kbd> element in markdown
+  - Spin spinner icon next to "Checking for CI status..." on MR page.
 v 7.10.2
   - Fix CI links on MR page

app/views/projects/merge_requests/show/_mr_ci.html.haml

 - if @commits.any?
   .ci_widget.ci-success{style: "display:none"}
-    %i.fa.fa-check
+    = icon("check")
     %span CI build passed
     for #{@merge_request.last_commit_short_sha}.
     = link_to "View build page", ci_build_details_path(@merge_request), :"data-no-turbolink" => "data-no-turbolink"
   .ci_widget.ci-failed{style: "display:none"}
-    %i.fa.fa-times
+    = icon("times")
     %span CI build failed
     for #{@merge_request.last_commit_short_sha}.
     = link_to "View build page", ci_build_details_path(@merge_request), :"data-no-turbolink" => "data-no-turbolink"
   - [:running, :pending].each do |status|
     .ci_widget{class: "ci-#{status}", style: "display:none"}
-      %i.fa.fa-clock-o
+      = icon("clock-o")
       %span CI build #{status}
       for #{@merge_request.last_commit_short_sha}.
       = link_to "View build page", ci_build_details_path(@merge_request), :"data-no-turbolink" => "data-no-turbolink"
   .ci_widget
-    %i.fa.fa-spinner
+    = icon("spinner spin")
     Checking for CI status for #{@merge_request.last_commit_short_sha}
   .ci_widget.ci-canceled{style: "display:none"}
-    %i.fa.fa-times
+    = icon("times")
     %span CI build canceled
     for #{@merge_request.last_commit_short_sha}.
     = link_to "View build page", ci_build_details_path(@merge_request), :"data-no-turbolink" => "data-no-turbolink"
   .ci_widget.ci-error{style: "display:none"}
-    %i.fa.fa-times
+    = icon("times")
     %span Cannot connect to the CI server. Please check your settings and try again.