Skip to content

Fix minor styling issues

Savas Vedova requested to merge 382712-fix-button-styling-issues into master

What does this MR do and why?

Minor CSS adjustments for action buttons as requested in the issue.

Screenshots or screen recordings

before after
image image

How to set up and validate locally

Use the following patch to display a button:

diff --git a/ee/app/assets/javascripts/vue_merge_request_widget/extensions/security_reports/mr_widget_security_reports.vue b/ee/app/assets/javascripts/vue_merge_request_widget/extensions/security_reports/mr_widget_security_reports.vue
index a4189d1214e..48aebe0b2f1 100644
--- a/ee/app/assets/javascripts/vue_merge_request_widget/extensions/security_reports/mr_widget_security_reports.vue
+++ b/ee/app/assets/javascripts/vue_merge_request_widget/extensions/security_reports/mr_widget_security_reports.vue
@@ -257,6 +257,7 @@ export default {
         :status-icon-name="statusIconNameReportType(report)"
         :help-popover="helpPopovers[report.reportType]"
         :data-testid="`report-${report.reportType}`"
+        :action-buttons="[{ href: '#', icon: 'download' }]"
       >
         <template #header>
           <div>

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 #382712 (closed)

Edited by Savas Vedova

Merge request reports