Ensure legacy viewer is loaded when highlight.js FF disabled + general cleanup
What does this MR do and why?
Ensures that we only load highlight.js behind a FF (being worked on in this epic)
This MR also does some general cleanup in the refactored blob viewer.
:refactor_blob_viewer feature flag.
Screenshots or screen recordings
| viewer | screenshot |
|---|---|
| download |  |
| image |  |
| video |  |
| text |  |
 |
|
| lfs |  |
How to set up and validate locally
- Enable the Blob Viewer Refactor feature flag
Feature.enable(:refactor_blob_viewer)- Open a file in the repository file browser
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Edited by Jacques Erasmus