Fix UI misalignment on user setting page (#25629) (#25656)

Backport #25629 by @wxiaoguang

Fix #25628

Diff with ignoring space:
https://github.com/go-gitea/gitea/pull/25629/files?diff=unified&w=1

The "modal" shouldn't appear between "ui attached segment", otherwise
these segments lose margin-top.

After the fix:

<details>


![image](https://github.com/go-gitea/gitea/assets/2114189/ac15e73d-4ca8-416a-950d-ffc912c6ab61)


![image](https://github.com/go-gitea/gitea/assets/2114189/eb431336-4d21-4e44-8beb-8919595a83c8)


![image](https://github.com/go-gitea/gitea/assets/2114189/dffaed88-5ba2-419d-a241-24cb200f757c)


![image](https://github.com/go-gitea/gitea/assets/2114189/e8c5a03e-e16a-4c94-a1a5-7845d4e1a824)

</details>

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>

templates/user/settings/account.tmpl

@@ -39,21 +39,15 @@
 			{{.locale.Tr "settings.manage_emails"}}
 		</h4>
 		<div class="ui attached segment">
-			<div class="ui email list">
+			<div class="ui list">
 				{{if $.EnableNotifyMail}}
 				<div class="item">
+					<div class="gt-mb-3">{{.locale.Tr "settings.email_desc"}}</div>
 					<form action="{{AppSubUrl}}/user/settings/account/email" class="ui form" method="post">
-						{{.locale.Tr "settings.email_desc"}}
-						<div class="right floated content">
-							<div class="field">
-								<button class="ui green button">{{$.locale.Tr "settings.email_notifications.submit"}}</button>
-							</div>
-						</div>
-						<div class="right floated content">
 						{{$.CsrfTokenHtml}}
 						<input name="_method" type="hidden" value="NOTIFICATION">
-							<div class="field">
+						<div class="gt-df gt-fw gt-gap-3">
-								<div class="ui selection dropdown" tabindex="0">
+							<div class="ui selection dropdown">
 								<input name="preference" type="hidden" value="{{.EmailNotificationsPreference}}">
 								{{svg "octicon-triangle-down" 14 "dropdown icon"}}
 								<div class="text"></div>
@@ -64,7 +58,7 @@
 									<div data-value="disabled" class="{{if eq .EmailNotificationsPreference "disabled"}}active selected {{end}}item">{{$.locale.Tr "settings.email_notifications.disable"}}</div>
 								</div>
 							</div>
-							</div>
+							<button class="ui green button">{{$.locale.Tr "settings.email_notifications.submit"}}</button>
 						</div>
 					</form>
 				</div>
@@ -102,7 +96,7 @@
 								</form>
 							</div>
 						{{end}}
-						<div class="content">
+						<div class="content gt-py-3">
 							<strong>{{.Email}}</strong>
 							{{if .IsPrimary}}
 								<div class="ui primary label">{{$.locale.Tr "settings.primary"}}</div>

templates/user/settings/applications_oauth2_list.tmpl

@@ -22,7 +22,19 @@
 			</div>
 		{{end}}
 	</div>
+
+	<div class="ui g-modal-confirm delete modal" id="remove-gitea-oauth2-application">
+		<div class="header">
+			{{svg "octicon-trash"}}
+			{{.locale.Tr "settings.remove_oauth2_application"}}
 		</div>
+		<div class="content">
+			<p>{{.locale.Tr "settings.oauth2_application_remove_description"}}</p>
+		</div>
+		{{template "base/modal_actions_confirm" .}}
+	</div>
+</div>
+
 <div class="ui attached bottom segment">
 	<h5 class="ui top header">
 		{{.locale.Tr "settings.create_oauth2_application"}}
@@ -46,14 +58,3 @@
 		</button>
 	</form>
 </div>
-
-<div class="ui g-modal-confirm delete modal" id="remove-gitea-oauth2-application">
-	<div class="header">
-		{{svg "octicon-trash"}}
-		{{.locale.Tr "settings.remove_oauth2_application"}}
-	</div>
-	<div class="content">
-		<p>{{.locale.Tr "settings.oauth2_application_remove_description"}}</p>
-	</div>
-	{{template "base/modal_actions_confirm" .}}
-</div>

templates/user/settings/grants_oauth2.tmpl

@@ -26,7 +26,6 @@
 			</div>
 		{{end}}
 	</div>
-</div>
 
 	<div class="ui g-modal-confirm delete modal" id="revoke-gitea-oauth2-grant">
 		<div class="header">
@@ -38,3 +37,4 @@
 		</div>
 		{{template "base/modal_actions_confirm" .}}
 	</div>
+</div>

templates/user/settings/security/accountlinks.tmpl

@@ -39,7 +39,6 @@
 			{{end}}
 		{{end}}
 	</div>
-</div>
 
 	<div class="ui g-modal-confirm delete modal" id="delete-account-link">
 		<div class="header">
@@ -51,3 +50,4 @@
 		</div>
 		{{template "base/modal_actions_confirm" .}}
 	</div>
+</div>

templates/user/settings/security/openid.tmpl

@@ -49,7 +49,6 @@
 			{{.locale.Tr "settings.add_openid"}}
 		</button>
 	</form>
-</div>
 
 	<div class="ui g-modal-confirm delete modal" id="delete-openid">
 		<div class="header">
@@ -61,3 +60,4 @@
 		</div>
 		{{template "base/modal_actions_confirm" .}}
 	</div>
+</div>

templates/user/settings/security/twofa.tmpl

@@ -21,7 +21,6 @@
 		<a class="ui green button" href="{{AppSubUrl}}/user/settings/security/two_factor/enroll">{{$.locale.Tr "settings.twofa_enroll"}}</a>
 	</div>
 	{{end}}
-</div>
 
 	<div class="ui g-modal-confirm delete modal" id="disable-twofa">
 		<div class="header">
@@ -33,3 +32,4 @@
 		</div>
 		{{template "base/modal_actions_confirm" .}}
 	</div>
+</div>

templates/user/settings/security/webauthn.tmpl

@@ -24,8 +24,6 @@
 		</div>
 		<button id="register-webauthn" class="ui green button">{{svg "octicon-key"}} {{.locale.Tr "settings.webauthn_register_key"}}</button>
 	</div>
-</div>
-
 	<div class="ui g-modal-confirm delete modal" id="delete-registration">
 		<div class="header">
 			{{svg "octicon-trash"}}
@@ -36,3 +34,4 @@
 		</div>
 		{{template "base/modal_actions_confirm" .}}
 	</div>
+</div>

web_src/css/repo.css

@@ -2365,10 +2365,6 @@
   color: var(--color-text-light-2);
 }
 
-.settings .list.email > .item:not(:first-child) {
-  min-height: 60px;
-}
-
 .settings .list.collaborator > .item {
   padding: 0;
 }