Follow-up from "Add selected state for chart button in vulnerabilities dashboard"
The following discussion from !54763 (merged) should be addressed:
-
@yo started a discussion: (+3 comments) I think a different approach would be to work through gitlab-ui#1000 (closed) to update this in GitLab UI, and then if the instances you opened the MRs for don’t use the component, migrate them to it.
/cc @andyvolpe
Implementation plan
-
https://gitlab.com/gitlab-org/gitlab/-/blob/master/ee/app/assets/javascripts/security_dashboard/components/vulnerability_chart_buttons.vue uses GlButtons
in order to display the image below. Replace the button group with https://gitlab-org.gitlab.io/gitlab-ui/?path=/story/base-segmented-control--default.
Edited by David Pisek