Skip to content

Make spinner pajamas compliant

Savas Vedova requested to merge 353732-pajamas-migration into master

What does this MR do and why?

Makes the loading spinner Pajamas-compliant.

Screenshots or screen recordings

before after
image image

How to set up and validate locally

  1. Go to your Project
  2. Click on Settings > General
  3. Apply the patch shown below
  4. Click on Collaps next to Merge Request Approvals
diff --git a/ee/app/assets/javascripts/pages/projects/edit/index.js b/ee/app/assets/javascripts/pages/projects/edit/index.js
index 3c4fd5f5153..62d13674f0c 100644
--- a/ee/app/assets/javascripts/pages/projects/edit/index.js
+++ b/ee/app/assets/javascripts/pages/projects/edit/index.js
@@ -16,7 +16,7 @@ groupsSelect();
 
 new UserCallout({ className: 'js-mr-approval-callout' });
 
-mountApprovals(document.getElementById('js-mr-approvals-settings'));
+// mountApprovals(document.getElementById('js-mr-approvals-settings'));
 mountStatusChecks(document.getElementById('js-status-checks-settings'));
 
 initProjectAdjournedDeleteButton();

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #353732 (closed)

Merge request reports