Fix "delete" modal dialog for issue/PR (#27015)
Close #27012 By the way, rename the single-word ID to a long ID.  
This commit is contained in:
		
							parent
							
								
									ebff0513db
								
							
						
					
					
						commit
						1875362383
					
				
					 3 changed files with 15 additions and 3 deletions
				
			
		| 
						 | 
				
			
			@ -30,6 +30,16 @@
 | 
			
		|||
		</form>
 | 
			
		||||
	</div>
 | 
			
		||||
 | 
			
		||||
	<div id="test-modal-form-4" class="ui mini modal">
 | 
			
		||||
		<div class="header">Form dialog (layout 4)</div>
 | 
			
		||||
		<div class="content">
 | 
			
		||||
			<div class="ui input gt-w-100"><input name="user_input"></div>
 | 
			
		||||
		</div>
 | 
			
		||||
		<form method="post">
 | 
			
		||||
			{{template "base/modal_actions_confirm" (dict "locale" $.locale "ModalButtonTypes" "confirm")}}
 | 
			
		||||
		</form>
 | 
			
		||||
	</div>
 | 
			
		||||
 | 
			
		||||
	<div class="ui g-modal-confirm modal" id="test-modal-default">
 | 
			
		||||
		<div class="header">{{svg "octicon-file"}} Default dialog <span>title</span></div>
 | 
			
		||||
		<div class="content">
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -655,11 +655,11 @@
 | 
			
		|||
				</form>
 | 
			
		||||
			</div>
 | 
			
		||||
		</div>
 | 
			
		||||
		<button class="gt-mt-2 fluid ui show-modal button" data-modal="#delete">
 | 
			
		||||
		<button class="gt-mt-2 fluid ui show-modal button" data-modal="#sidebar-delete-issue">
 | 
			
		||||
			{{svg "octicon-trash"}}
 | 
			
		||||
			{{.locale.Tr "repo.issues.delete"}}
 | 
			
		||||
		</button>
 | 
			
		||||
		<div class="ui g-modal-confirm modal" id="delete">
 | 
			
		||||
		<div class="ui g-modal-confirm modal" id="sidebar-delete-issue">
 | 
			
		||||
			<div class="header">
 | 
			
		||||
				{{if .Issue.IsPull}}
 | 
			
		||||
					{{.locale.Tr "repo.pulls.delete.title"}}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -36,6 +36,7 @@ However, Gitea uses the following layouts:
 | 
			
		|||
* <div .modal><div .content><div .actions/></div></div>
 | 
			
		||||
* <div .modal><form><div .content/><div .actions/></form></div>
 | 
			
		||||
* <div .modal><div .content><form><div .actions/></form></div></div>
 | 
			
		||||
* <div .modal><div .content></div><form><div .actions/></form></div>
 | 
			
		||||
* ...
 | 
			
		||||
These inconsistent layouts should be refactored to simple ones.
 | 
			
		||||
*/
 | 
			
		||||
| 
						 | 
				
			
			@ -47,7 +48,8 @@ These inconsistent layouts should be refactored to simple ones.
 | 
			
		|||
}
 | 
			
		||||
 | 
			
		||||
.ui.modal > .actions,
 | 
			
		||||
.ui.modal .content + .actions {
 | 
			
		||||
.ui.modal .content + .actions,
 | 
			
		||||
.ui.modal .content + form > .actions {
 | 
			
		||||
  background: var(--color-secondary-bg);
 | 
			
		||||
  border-color: var(--color-secondary);
 | 
			
		||||
  padding: 1rem;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue