Merge pull request 'Fix bad spacing on new release page' (#5261) from maltejur/forgejo:push-nnnyrrvzlqqy into forgejo

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/5261
Reviewed-by: Otto <otto@codeberg.org>

templates/repo/release/new.tmpl

@@ -99,9 +99,9 @@
 						{{ctx.Locale.Tr "remove"}}
 					</a>
 				</div>
-				<a class="ui mini button tw-float-right tw-mb-4 tw-mt-2" id="add-external-link">
+				<button type="button" class="ui mini button tw-float-right tw-mb-4 tw-mt-2" id="add-external-link">
 					{{ctx.Locale.Tr "repo.release.add_external_asset"}}
-				</a>
+				</button>
 				{{if .IsAttachmentEnabled}}
 					<div class="field">
 						{{template "repo/upload" .}}
@@ -111,32 +111,28 @@
 			<div class="divider"></div>
 			<div class="ui">
 				<div>
+					<fieldset>
 						{{if not .PageIsEditRelease}}
-						<div class="tag-message field">
+							<label>
-							<div class="ui checkbox">
 								<input type="checkbox" name="add_tag_msg">
-								<label><strong>{{ctx.Locale.Tr "repo.release.add_tag_msg"}}</strong></label>
+								{{ctx.Locale.Tr "repo.release.add_tag_msg"}}
-							</div>
+							</label>
-						</div>
 						{{else}}
 							<input type="hidden" name="add_tag_msg" value="false">
 						{{end}}
-					<div class="prerelease field">
+						<label>
-						<div class="ui checkbox">
 							<input type="checkbox" name="prerelease" {{if .prerelease}}checked{{end}}>
-							<label><strong>{{ctx.Locale.Tr "repo.release.prerelease_desc"}}</strong></label>
+							{{ctx.Locale.Tr "repo.release.prerelease_desc"}}
-						</div>
-					</div>
 							<span class="help">{{ctx.Locale.Tr "repo.release.prerelease_helper"}}</span>
+						</label>
 						{{if not .DisableDownloadSourceArchives}}
-						<div class="field">
+							<label>
-							<div class="ui checkbox">
 								<input type="checkbox" name="hide_archive_links" {{if .hide_archive_links}}checked{{end}}>
-								<label><strong>{{ctx.Locale.Tr "repo.release.hide_archive_links"}}</strong></label>
+								{{ctx.Locale.Tr "repo.release.hide_archive_links"}}
-							</div>
-						</div>
 								<span class="help">{{ctx.Locale.Tr "repo.release.hide_archive_links_helper"}}</span>
+							</label>
 						{{end}}
+					</fieldset>
 					<div class="divider tw-mt-0"></div>
 					<div class="tw-flex tw-justify-end button-row">
 						{{if .PageIsEditRelease}}

tests/e2e/release.test.e2e.js

@@ -1,6 +1,7 @@
 // @ts-check
 import {expect} from '@playwright/test';
 import {test, login_user, save_visual, load_logged_in_context} from './utils_e2e.js';
+import {validate_form} from './shared/forms.js';
 
 test.beforeAll(async ({browser}, workerInfo) => {
   await login_user(browser, workerInfo, 'user2');
@@ -23,6 +24,7 @@ test('External Release Attachments', async ({browser, isMobile}, workerInfo) =>
 
   // Fill out form and create new release
   await expect(page).toHaveURL('/user2/repo2/releases/new');
+  await validate_form({page}, 'fieldset');
   await page.fill('input[name=tag_name]', '2.0');
   await page.fill('input[name=title]', '2.0');
   await page.click('#add-external-link');
@@ -43,6 +45,7 @@ test('External Release Attachments', async ({browser, isMobile}, workerInfo) =>
 
   // Validate edit page and edit the release
   await expect(page).toHaveURL('/user2/repo2/releases/edit/2.0');
+  await validate_form({page}, 'fieldset');
   await expect(page.locator('.attachment_edit:visible')).toHaveCount(2);
   await expect(page.locator('.attachment_edit:visible').nth(0)).toHaveValue('Test');
   await expect(page.locator('.attachment_edit:visible').nth(1)).toHaveValue('https://forgejo.org/');

web_src/css/form.css

@@ -10,6 +10,7 @@ fieldset legend {
 
 fieldset label {
   display: block;
+  margin-bottom: 0.6em;
 }
 
 fieldset label:has(input[type="text"]),
@@ -19,7 +20,11 @@ fieldset label:has(input[type="number"]) {
 
 fieldset .help {
   font-weight: var(--font-weight-normal);
-  display: block !important; /* overrides another rule in this file, remove when obsolete */
+}
+
+.form fieldset .help { /* overrides other .form .help rules in this file, remove when obsolete */
+  display: block !important;
+  padding-bottom: 0;
 }
 
 fieldset input[type="checkbox"],

web_src/css/repo/release-tag.css

@@ -99,10 +99,6 @@
   padding-bottom: 10px;
 }
 
-.repository.new.release .prerelease.field {
-  margin-bottom: 0;
-}
-
 @media (max-width: 438px) {
   .repository.new.release .field button,
   .repository.new.release .field input {