Create new directory modal: Replace toggle with commit changes UI
Proposal
After discussions in #444302 (closed), the piece of UI that allows users to commit to a new/existing branch has been updated for the Edit/Delete file flow in !170512 (merged). This branch selection UI inside modals that allow users to commit changes should also be percolated to: New file, Upload file, and New directory. The design proposal is:
- Replace the "Start a new merge request with these changes" toggle with the checkbox UI from the commit changes modal.
- Change modal header text "Create New Directory" with "Create new directory" to be consistent with Pajamas guidelines.
Current | Proposed |
---|---|
Problem
When you go to create a new directory, the option to start a merge request is a toggle. This is inconsistent with the changes made to the edit file blob in !170512 (merged) and #444302 (closed).
Steps
- Go to "Code > Repository"
- Click on the "+"
- Click on "New directory"
Scenarios
The following conditions should be considered (see !170512 (merged)):
- Commit to the current branch and user can push to the branch
- Commit to the current branch and user can NOT push to the branch
- Commit to new branch and open a MR
- Commit to new branch and NOT open a MR
- Cancel button pops a confirmation modal
- Mobile screen
Related UI
These are areas of the UI that should all be using the same commit changes modal UI:
UI element | Issues/MRs | Reproduction steps |
---|---|---|
Edit file |
!170512 (merged) (in review) |
Code > Repository > Select a file > Edit > Edit single file |
Delete file |
#503405 (closed) (open) |
Code > Repository > Select a file > Delete |
New file |
#502460 (open) |
Code > Repository > New file |
Upload file |
#479249 (closed) (open) |
Code > Repository > Upload file |
New directory |
#502483 (this, open) |
Code > Repository > New directory |