Merge branch 'compliance-framework-badge' into 'master'

Compliance framework badge remove custom styles

See merge request https://gitlab.com/gitlab-org/gitlab/-/merge_requests/139058



Merged-by: Coung Ngo <cngo@gitlab.com>
Approved-by: Coung Ngo <cngo@gitlab.com>
Approved-by: Diana Zubova <dzubova@gitlab.com>
Approved-by: Julia Miocene <jmiocene@gitlab.com>
Reviewed-by: Diana Zubova <dzubova@gitlab.com>
Co-authored-by: Sascha Eggenberger <seggenberger@gitlab.com>

ee/app/assets/stylesheets/framework/badges.scss

-.badge.compliance-framework-pill {
-  border-radius: $label-border-radius;
-  display: inline-block;
-  font-size: $gl-font-size-sm;
-  font-weight: $gl-font-weight-normal;
-  line-height: $gl-line-height-20;
-  padding: 0 $label-padding;
-  vertical-align: middle;
-}

ee/app/assets/stylesheets/page_bundles/projects.scss

@@ -34,14 +34,6 @@
   }
 }
 
-.projects-list {
-  .project-details {
-    .compliance-framework-pill {
-      margin: 1px 0;
-    }
-  }
-}
-
 .git-clone-holder {
   .btn-geo {
     border-left: 0;

ee/app/views/compliance_management/compliance_framework/_compliance_framework_badge.html.haml

@@ -4,5 +4,5 @@
 - if show_compliance_framework_badge?(project)
   - framework = project.compliance_framework_setting.compliance_management_framework
 
-  = gl_badge_tag(nil, { class: "compliance-framework-pill has-tooltip #{additional_classes}", style: "background-color: #{framework.color}; color: #{text_color_for_bg(framework.color)}", data: { container: 'body' }, title: framework.description}) do
+  = gl_badge_tag(nil, { class: "has-tooltip #{additional_classes}", style: "background-color: #{framework.color}; color: #{text_color_for_bg(framework.color)}", data: { container: 'body', testid: 'compliance-framework-badge' }, title: framework.description}) do
     = framework.name

ee/spec/views/compliance_management/compliance_framework/_compliance_framework_badge.html.haml_spec.rb

@@ -15,7 +15,7 @@
     it 'renders a badge' do
       render('compliance_management/compliance_framework/compliance_framework_badge', project: project)
 
-      expect(rendered).to have_selector('.compliance-framework-pill')
+      expect(rendered).to have_selector('[data-testid="compliance-framework-badge"]')
     end
   end
 
@@ -25,7 +25,7 @@
     it 'does not render any badge' do
       render('compliance_management/compliance_framework/compliance_framework_badge', project: project)
 
-      expect(rendered).not_to have_selector('.compliance-framework-pill')
+      expect(rendered).not_to have_selector('[data-testid="compliance-framework-badge"]')
     end
   end
 end