Skip to content

Fix styling for Request CVE button in sidebar

Dheeraj Joshi requested to merge djadmin-fix-request-cve-sidebar into master

Resolves #343059 (closed)

What does this MR do and why?

Fix styling for "Request CVE" button in Issue Sidebar.

It was done by using a commonly used class for the sidebar (block) which makes it consistent with other sections.

Also, did a minor refactor by using helpPagePath helper method.

Screenshots or screen recordings

before after
image image

How to set up and validate locally

  1. Apply the follow patch
diff --git a/ee/app/views/shared/issuable/_sidebar_cve_id_request.haml b/ee/app/views/shared/issuable/_sidebar_cve_id_request.haml
index 335b4dd6e1d..3323ee7a1b3 100644
--- a/ee/app/views/shared/issuable/_sidebar_cve_id_request.haml
+++ b/ee/app/views/shared/issuable/_sidebar_cve_id_request.haml
@@ -1,2 +1 @@
-- if issuable_sidebar[:request_cve_enabled_for_user]
-  #js-sidebar-cve-id-request-entry-point
+#js-sidebar-cve-id-request-entry-point
\ No newline at end of file
  1. Open a confidential issue on GDK

  2. Look for Request CVE button on the sidebar

MR acceptance checklist

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

Edited by Dheeraj Joshi

Merge request reports