Prepare common Vue app for Repository header area
All threads resolved!
All threads resolved!
Compare changes
Files
5- Paulina Sedlak-Jakubowska authored
@@ -18,8 +18,8 @@ import {
This MR is a part of a bigger effort that aims to consolidate various tiny Vue apps in the header area of tree and blob view into one common Vue application.
Note: There's a potential for refactor, but that part will be handled in the following issues within &13018. The MRs for #497051 (closed) goal is to move everything as is.
Even though the end result should not change any UI and behaviour, it is quite a ride
common_repository_blob_header_app=true
URL paramThe breakdown of #497051 (closed) MRs:
step | status |
---|---|
introduce ff & move first batch of controls, move web ide button for tree header, recreate projects/tree/lock_link in Vue | ![]() |
move code buttons (desktop & mobile) for tree header | ![]() |
move contents of app/views/projects/blob/_breadcrumb.html.haml | ![]() |
turn on feature tests for when the ff is true
|
![]() |
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 are required for UI changes, and strongly recommended for all other merge requests.
Context | Before | After |
---|---|---|
tree: old version with multiple Vue apps | ![]() |
![]() |
blob: old version with multiple Vue apps | ![]() |
![]() |
tree: different ref than root ref (ff off vs. ff on) | ![]() |
![]() |
blob | ![]() |
![]() |
Numbered steps to set up and validate the change are strongly suggested.
common_repository_blob_header_app=true
param to the URL~
prefix for searching for filesnew blob | legacy blob |
---|---|
![]() |
![]() |
Related to #497051 (closed)