Multi-file editor
demoscript: link
Description
Editing more than a single file using the GitLab interface can feel laborious because each file must be edited one by one, requiring at least one commit for each edited file. This makes contributing larger changes multiple files, be they markdown or source code, through the GitLab interface difficult.
We want to make it easy for everyone to collaborate. Removing barriers will help people contribute more than fixes to typos and dead links, but allow people to contribute larger changes, offer more comprehensive code review suggestions and more.
Kudos to @jschatz1 for idea and proof of concept
Proof of concept can be accessed by creating a cookie with the key new_repo
with the value true
.
Proposal
%10.3
BetaThe beta should be available as an alternative for the existing blob editor. It should be focussed on editing multiple text files with a single commit.
- Multi-file editor should allow multiple text files to opened, created, edited and committed as a single commit
- Multi-file editor should co-exist with existing blob viewer and editor, so that users of current interface are not impacted by limitations or performance problems with the new multi-file editor
-
Decouple multi-file editor from file list, to an IDE route #40040 (closed) -
Edit in multi-file editor button #40032 (closed) -
Return to blob editor from multi-file editor #40035 (closed)
-
- Multi-file editor should allow the user to view images
-
Blob viewer #38355 (moved)
-
- Multi-file editor should allow the user to see a list of the changed files staged for commit and add a commit message. The user should also be able to identify which lines have changed in those files
-
Right hand commit panel #40041 (closed) -
Highlight changed lines #40046 (closed)
-
- Multi-file editor should allow the user to adjust the width of panels to suit the information within each panel and make the best use of their screen size
-
Full width responsiveness (task #40040 (closed))
-
- Multi-file editor should attempt to prevent users losing their work by navigating to a new page or closing the browser
-
Show warning trying to close page without committing changes -
Nice to have: Save changes to local storage #40045 (moved)
-
- Multi-file editor should be consistent with GitLab visual language
-
Tree panel header styling #38363 (closed)
-
%10.4
Improvements- Performance and reliability
-
Prevent changes being lost or overwritten when branches diverge #38606 (moved) (backend #38356 (closed)) -
Tab switching performance #38562 (closed)
-
- Usability improvements
-
Improved flow @dimitrieh -
Resizable left/right panels #40042 (closed) -
Create branch when committing to a project with write permissions #40942 (moved) -
Create fork when opening a project without write permissions #38357 (moved) -
Fuzzy file finder
-
- GitLab integration
-
Open all files from merge request #40961 (moved)
-
Backlog
- Popup dialog needs specs #36376 (closed)
- DRY up Repo Editor HTML #36522 (closed)
- Make
blobURLtoParentTree
less brittle #36364 (closed) - Fix
commitFiles
error handling #36365 (moved) - Use proper markup in dropdown_input helper #36370 (closed)
- Remove root v-if's from repo components #36493 (closed)
- Remove deep watchers #36494 (closed)
- Refactor uses of Vue.nextTick to use returned promise and avoid nested callbacks #36371 (closed)
- Fill in or remove empty specs for editClicked #36372 (closed)
- Move branch-specific logic from gl_dropdown into repo editor #36373 (closed)
- Reset RepoStore before each test #36462 (closed)
- Expose proper entities in Repo Entity #36037 (closed)
- Make
scrollTabsRight
less fragile #36362 (closed) - Don't rely on HTML to create whitespace in repo editor #36356 (closed)
- Re-design and refactor store logic & data flow through repo editor #36357 (closed)
- Break up makeCommit & write more tests #36358 (closed)
- Determine a better approach to method use and re-use than repo_helper #36359 (closed)
- Make
getLanguageIDForFile
less brittle #36360 (closed) - Break up
getContent
#36361 (closed) - https://gitlab.com/gitlab-org/gitlab-ce/issues/36102
- https://gitlab.com/gitlab-org/gitlab-ce/issues/36040
- https://gitlab.com/gitlab-org/gitlab-ce/issues/36142
GA requirements
-
Replace existing blob viewer/editor with Monaco editor (subset of multi-file editor) -
Port blob viewer syntax highlighting to Monaco -
Lock and unlock files #38623 (moved) ~"EE Premium" -
Create file from template #38630 (moved) -
i18n #38611 (moved) -
Auxiliary blob viewer #37060 (closed)
Source: https://gitlab.com/gitlab-org/gitlab-ce/issues/31890
Links / references
- https://gitlab.com/gitlab-org/gitlab-ce/issues/13723
- https://gitlab.com/gitlab-org/gitlab-ce/issues/31890
Documentation blurb
Overview
There are users of GitLab who contributor exclusively through the web interface or occasionally through the web interface to commit changes to documentation, static sites, and even source code.
This feature will improve the experience of those existing users by making contributions to more than one file easier.
Use cases
When web-only GitLab users want to make related edits to multiple files, they are obstructed by having to complete multiple manual steps that are separated by many screens.
Feature checklist
Make sure these are completed before closing the issue, with a link to the relevant commit.
-
Feature assurance -
Documentation -
Added to features.yml
Alpha plan
#### 1. Alpha %10.2Building on the proof of concept, the alpha should demonstrate a working editor with no obvious performance regressions. It should work well browsing repositories and be suitable for editing the handbook and other projects commonly edited in the web interface.
Features
-
Hide 'recent commit viewer' -
File tree list #13723 (closed) -
File tree while viewing and editing #13723 (closed) -
Link to file -
Remove source-target branch switcher, @jschatz1 !14558 (merged) -
Create new branch, @jschatz1 #38485 (closed) !14714 (merged) -
Create merge request, @jschatz1 #38626 (closed) !14665 (merged) -
Upgrade Monaco editor, @timzallmann #38607 (closed) !14629 (merged) -
Browser history/back button, @timzallmann #36436 (closed) FE Test Spin next -
Page title, @timzallmann #36029 (closed) FE Done -
Upload file #38629 (closed) -
New file/folder #38614 (closed) -
Prevent locked files from being edited ~"EE Premium" gitlab-org/gitlab-ee#3918
Performance
-
File list, @jschatz1 #38360 (closed) !14458 (closed)
Bugs
-
Link to file line/range, @fatihacet #38254 (closed) -
Create branch issue because of new_branch
#38477 (closed) -
Incorrect default target branch #36367 (closed) -
Focus open file when selected in tree nav #38250 (closed) -
Right align Last Update column #38624 (closed) -
Directories missing latest commit data #38627 (closed) !14458 (closed)
Tech debt
-
Add last_commit_sha
to batch commits API #36038 (closed)
Initial designs
Repository - Initial view of files
Initial View upon entering the repository tab. Double-click a file name to open it.
Repo File Editor - Read-Only View
Once the file is opened, the view of that file appears to slide in from the right. The files view remains and other files can be opened from this menu. Once all files opened are closed, we return to the initial view of the files. The last file closed appears to slide off to the right.
Notes:
Global area above editor:
- The branch can be changed using the branch dropdown.
- Users toggle into ‘Edit’ mode by pressing the
Edit Mode
button. - Users can download all files (same functionality as exists now).
File tree editor:
- This design shows the ability to download all files within the tree editor. This is a duplicate of the download capability in the global bar. Was left in for possible future iterations.
- Ability to create a new folder or file using the icons.
- Ability to lock or unlock files. Shown only on subset of folders - when implemented all folders would be unlocked by default, shown via the light gray lock symbol.
Commit Section:
- There is no commit section visible when the file is in read-only mode.
Repo File Editor - Edit View
Clicking the Edit Mode
button enables the editing of files.
Notes:
Target Branch
- The 'Edit Mode` button text changes to say, 'Cancel Editing'.
- The branch dropdown becomes locked and a new dropdown appears next to it. This new dropdown selects the target branch for the edits. It is set to create a new branch by default. Users can click the drop-down arrow to change the new branch name or search for and select an existing branch.
- If you don’t have permission (or the branch is protected) then you cannot edit the files.
TODO: Needs Mockup
Cancel Edit
Users can cancel their edits by selecting the 'Cancel Editing' button.
A confirmation modal is triggered. This modal should match the design from issue#28849
TODO: Needs Mockup
Edit
When a file has been edited, the x
becomes a green circle to let users know there are edits that need to be committed.
Repo File Editor - Committing changes
When a file (or files) have been edited, the file appears as staged in the commit section.
Notes:
*When you make a change on master (with permissions), you will be encouraged to create a merge request. We should not stop people from making changes directly to master if they wish.
*The target branch appears but cannot be changed here.