Skip to content

Migrate security_report_download_dropdown to GlDisclosureDropdown

What does this MR do and why?

Relates to #422523 (closed)

Migrate security_report_download_dropdown to GlDisclosureDropdown

  1. Migrate from GlDropdown to GlDisclosureDropdown
  2. Remove title prop and tooltip – unused
  3. Simplify spec

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After
Screenshot_2023-08-23_at_12.36.43 Screenshot_2023-08-23_at_12.36.01

How to set up and validate locally

Prerequisites

  1. You need an EE license
  2. You need to have runners enabled (See $2408961 for setting up a runner)
  3. Import https://gitlab.com/gitlab-examples/security/security-reports
  4. Run a pipeline on master

Validate

  1. Go to pipeline > security tab
  2. Expand reports by clicking "Show details" in the Scan details card
  3. Click on any "Download results" button
  4. Validate that this shows disclosure dropdown and each option can be clicked and will download the scan results

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 Lorenz van Herwaarden

Merge request reports