Ensure CSS in the blob viewer is consistent
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
During the process of refactoring the blob viewer, while we had the refactored blob viewer behind a feature flag, we had to ensure that both the non-refactored blob viewer and the refactored blob viewer looked and worked as expected.
But because the refactored blob viewer's markup did not match the legacy viewer exactly, we had to override some styles on the refactored viewer, for example, using utility classes on the refactored viewer to override styles that were already defined in app/assets/stylesheets/framework/highlight.scss.
Using utility classes is not incorrect, but we should ideally not be overriding styles that were already defined.
For this issue, the idea is to discover and fix CSS that is being overridden. Ideally, this should be done once the refactored blob viewer has been rolled out to ensure that we don't end up breaking the legacy blob viewer.