Adds gl-icon to icons in buttons

This change adds gl-icon class to icons in buttons so they
maintain the secondary class color.

app/views/projects/artifacts/_artifact.html.haml

@@ -50,12 +50,12 @@
     .table-action-buttons
       .btn-group
         - if can?(current_user, :read_build, @project)
-          = link_to download_project_job_artifacts_path(@project, artifact.job), rel: 'nofollow', download: '', title: _('Download artifacts'), data: { placement: 'top', container: 'body' }, ref: 'tooltip', aria: { label: _('Download artifacts') }, class: 'gl-button btn btn-default btn-default-secondary btn-icon has-tooltip' do
-            = sprite_icon('download')
+          = link_to download_project_job_artifacts_path(@project, artifact.job), rel: 'nofollow', download: '', title: _('Download artifacts'), data: { placement: 'top', container: 'body' }, ref: 'tooltip', aria: { label: _('Download artifacts') }, class: 'gl-button btn btn-default btn-icon has-tooltip' do
+            = sprite_icon('download', css_class: 'gl-icon')
 
-          = link_to browse_project_job_artifacts_path(@project, artifact.job), rel: 'nofollow', title: _('Browse artifacts'), data: { placement: 'top', container: 'body' }, ref: 'tooltip', aria: { label: _('Browse artifacts') }, class: 'gl-button btn btn-default btn-default-secondary btn-icon has-tooltip' do
-            = sprite_icon('folder-open')
+          = link_to browse_project_job_artifacts_path(@project, artifact.job), rel: 'nofollow', title: _('Browse artifacts'), data: { placement: 'top', container: 'body' }, ref: 'tooltip', aria: { label: _('Browse artifacts') }, class: 'gl-button btn btn-default btn-icon has-tooltip' do
+            = sprite_icon('folder-open', css_class: 'gl-icon')
 
         - if can?(current_user, :destroy_artifacts, @project)
           = link_to project_artifact_path(@project, artifact), data: { placement: 'top', container: 'body', confirm: _('Are you sure you want to delete these artifacts?') }, method: :delete, title: _('Delete artifacts'), ref: 'tooltip', aria: { label: _('Delete artifacts') }, class: 'gl-button btn btn-danger btn-icon has-tooltip' do
-            = sprite_icon('remove')
+            = sprite_icon('remove', css_class: 'gl-icon')

app/views/projects/ci/builds/_build.html.haml

@@ -100,33 +100,33 @@
     .gl-display-flex
       .btn-group
         - if can?(current_user, :read_job_artifacts, job) && job.artifacts?
-          = link_to download_project_job_artifacts_path(job.project, job), rel: 'nofollow', download: '', title: _('Download artifacts'), class: 'gl-button btn btn-default btn-default-secondary btn-icon' do
-            = sprite_icon('download')
+          = link_to download_project_job_artifacts_path(job.project, job), rel: 'nofollow', download: '', title: _('Download artifacts'), class: 'gl-button btn btn-default btn-icon' do
+            = sprite_icon('download', css_class: 'gl-icon')
         - if can?(current_user, :update_build, job)
           - if job.active?
-            = link_to cancel_project_job_path(job.project, job, continue: { to: request.fullpath }), method: :post, title: _('Cancel'), class: 'gl-button btn btn-default btn-default-secondary btn-icon' do
-              = sprite_icon('close')
+            = link_to cancel_project_job_path(job.project, job, continue: { to: request.fullpath }), method: :post, title: _('Cancel'), class: 'gl-button btn btn-default btn-icon' do
+              = sprite_icon('close', css_class: 'gl-icon')
           - elsif job.scheduled?
-            .gl-button.btn.btn-default.btn-default-secondary.btn-icon.disabled{ disabled: true }
-              = sprite_icon('planning')
+            .gl-button.btn.btn-default.btn-icon.disabled{ disabled: true }
+              = sprite_icon('planning', css_class: 'gl-icon')
               %time.js-remaining-time{ datetime: job.scheduled_at.utc.iso8601 }
                 = duration_in_numbers(job.execute_in)
             - confirmation_message = s_("DelayedJobs|Are you sure you want to run %{job_name} immediately? This job will run automatically after it's timer finishes.") % { job_name: job.name }
             = link_to play_project_job_path(job.project, job, return_to: request.original_url),
               method: :post,
               title: s_('DelayedJobs|Start now'),
-              class: 'gl-button btn btn-default btn-default-secondary btn-icon has-tooltip',
+              class: 'gl-button btn btn-default btn-icon has-tooltip',
               data: { confirm: confirmation_message } do
-              = sprite_icon('play')
+              = sprite_icon('play', css_class: 'gl-icon')
             = link_to unschedule_project_job_path(job.project, job, return_to: request.original_url),
               method: :post,
               title: s_('DelayedJobs|Unschedule'),
-              class: 'gl-button btn btn-default btn-default-secondary btn-icon has-tooltip' do
-              = sprite_icon('time-out')
+              class: 'gl-button btn btn-default btn-icon has-tooltip' do
+              = sprite_icon('time-out', css_class: 'gl-icon')
           - elsif allow_retry
             - if job.playable? && !admin && can?(current_user, :update_build, job)
-              = link_to play_project_job_path(job.project, job, return_to: request.original_url), method: :post, title: _('Play'), class: 'gl-button btn btn-default btn-default-secondary btn-icon' do
-                = sprite_icon('play')
+              = link_to play_project_job_path(job.project, job, return_to: request.original_url), method: :post, title: _('Play'), class: 'gl-button btn btn-default btn-icon' do
+                = sprite_icon('play', css_class: 'gl-icon')
             - elsif job.retryable?
-              = link_to retry_project_job_path(job.project, job, return_to: request.original_url), method: :post, title: _('Retry'), class: 'gl-button btn btn-default btn-default-secondary btn-icon' do
+              = link_to retry_project_job_path(job.project, job, return_to: request.original_url), method: :post, title: _('Retry'), class: 'gl-button btn btn-default btn-icon' do
                 = sprite_icon('repeat', css_class: 'gl-icon')

app/views/projects/deployments/_rollback.haml

 - if deployment.deployable && can?(current_user, :create_deployment, deployment)
   - tooltip = deployment.last? ? s_('Environments|Re-deploy to environment') : s_('Environments|Rollback environment')
-  = button_tag class: 'gl-button btn btn-default btn-default-secondary btn-icon has-tooltip', type: 'button', data: { toggle: 'modal', target: "#confirm-rollback-modal-#{deployment.id}" }, title: tooltip do
+  = button_tag class: 'gl-button btn btn-default btn-icon has-tooltip', type: 'button', data: { toggle: 'modal', target: "#confirm-rollback-modal-#{deployment.id}" }, title: tooltip do
     - if deployment.last?
-      = sprite_icon('repeat')
+      = sprite_icon('repeat', css_class: 'gl-icon')
     - else
-      = sprite_icon('redo')
+      = sprite_icon('redo', css_class: 'gl-icon')
   = render 'projects/deployments/confirm_rollback_modal', deployment: deployment

app/views/projects/pipelines/_with_tabs.html.haml

@@ -67,8 +67,8 @@
                 = build.present.callout_failure_message
               %td.responsive-table-cell.build-actions
                 - if can?(current_user, :update_build, job) && job.retryable?
-                  = link_to retry_project_job_path(build.project, build, return_to: request.original_url), method: :post, title: _('Retry'), class: 'gl-button btn btn-default btn-default-secondary btn-icon' do
-                    = sprite_icon('repeat')
+                  = link_to retry_project_job_path(build.project, build, return_to: request.original_url), method: :post, title: _('Retry'), class: 'gl-button btn btn-default btn-icon' do
+                    = sprite_icon('repeat', css_class: 'gl-icon')
             - if can?(current_user, :read_build, job)
               %tr.build-trace-row.responsive-table-border-end
                 %td