Leverage injected blobInfo in blob controls components
What does this MR do and why?
Use injected blobInfo for overflow menu subcomponents. With each following MR there's more and more blobInfo properties that I need to use. There's no sense in adding additional props. Now I only inject the blobInfo and use selected properties in each component. Defaulting to DEFAULT_BLOB_INFO in case the request fails.
References
Please include cross links to any resources that are relevant to this MR. This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.
- Rearrange and introduce overflow menu to single file page
- Change BlobButtonGroup into a dropdown part 2
- next step after this MR: !180241 (closed)
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
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
| Before | After |
|---|---|
| n/a | n/a |
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
- Make sure all unit tests for
spec/frontend/repository/components/header_areapass - Enable the feature flag
Feature.enable(:blob_overflow_menu)- Go to a project
- Navigate to a chosen file in your repository.
- Check all the actions in the overflow menu and make sure there's no console errors.
Related to #450774 (closed)