Unify when copy file control is disabled
What does this MR do and why?
I've noticed that the mechanism for defining the viewer type in new blob controls is not reactive and does not disables Copy contents control, when on rich viewer. I'm aligning new controls with what we already have in app/assets/javascripts/repository/components/blob_content_viewer.vue
Note: Copy contents control is in the blob overflow menu dropdown only in the mobile layout. Above sm breakpoints, it is in the file viewer's header where it used to be.
References
Screenshots or screen recordings
| Before | After |
|---|---|
 |
 |
 |
 |
How to set up and validate locally
-
In rails console enable the feature flag
Feature.enable(:blob_overflow_menu) -
Open Project / Code /Repository and choose a markdown file.
-
When switching the file to Preview mode, the Copy contents button should be disabled.
-
Make sure your viewport is around 500px wide or less. Verify that Copy contents in the blob overflow menu is also disabled.
-
Switch to Code mode of the viewer and check if both controls are enabled.
MR acceptance checklist
Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.