Skip to content

Removes the custom zoom icon for contributors chart

Ezekiel Kigbo requested to merge ek-remove-unused-zoom-svg into master

What does this MR do and why?

Removes the custom zoom icon for contributors chart, the default zoom svg is set in gitlab-ui instead so this is no longer needed

This MR:

  • Updates the storybook stories
  • Removes the getSvgIconPathContent prop that is no longer used

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

NOTE: No visual changes to the chart, the screenshots are to show the behaviour remains the same

Before After
svg-contributors-before svg-contributors-after

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Open gitlab storybook from gdk directory, run $ yarn storybook:start
  2. Select the contributors chart, move the zoom slider below the top line chart
  3. Run your local GDK
  4. Navigate to the contributors page for any project Analyze > Contributor analytics
  5. Interact with the zoom slider on the page

Related to #450450 (closed)

Edited by Ezekiel Kigbo

Merge request reports