From f1622c4e85eccb6b8accee3fa5a5708324ac498a Mon Sep 17 00:00:00 2001
From: Dylan Weijgertze <dylanwprivate@gmail.com>
Date: Mon, 1 Jun 2026 12:14:01 +0200
Subject: [PATCH] fix(ui): adjust sidebar distances and remove empty p elements
 (#12864)

I removed the empty `p` elements for accessibility as mentioned in this https://codeberg.org/forgejo/forgejo/pulls/12786#issuecomment-16247117. I also adjusted the spacing for the "reference" text with the divider to have the copy button have the same spacing as other buttons.

## Testing

Navigate to an issue and observe the spacing of the time tracker, reference and dependencies sections.

<table style="width:100%">
  <tr>
    <td>Before</td>
    <td>After</td>
  </tr>
  <tr>
    <td><img width="250" src="/attachments/a4e14050-1d14-457c-9d4f-ec9ac1ae3bcc" /></td>
    <td><img width="250" src="/attachments/3ab4445c-9f14-49d7-ac77-a7d69e914e58" /></td>
  </tr>
</table>

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/12864
Reviewed-by: 0ko <0ko@noreply.codeberg.org>
---
 .../repo/issue/view_content/sidebar/due_deadline.tmpl    | 3 +--
 templates/repo/issue/view_content/sidebar/reference.tmpl | 9 +++------
 .../repo/issue/view_content/sidebar/timetracking.tmpl    | 3 +--
 3 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/templates/repo/issue/view_content/sidebar/due_deadline.tmpl b/templates/repo/issue/view_content/sidebar/due_deadline.tmpl
index b37c4462a5..e788f1ca61 100644
--- a/templates/repo/issue/view_content/sidebar/due_deadline.tmpl
+++ b/templates/repo/issue/view_content/sidebar/due_deadline.tmpl
@@ -5,8 +5,7 @@
 		{{ctx.Locale.Tr "repo.issues.due_date_invalid"}}
 	</div>
 	{{if ne .Issue.DeadlineUnix 0}}
-		<p></p>
-		<div class="tw-flex tw-justify-between tw-items-center">
+		<div class="tw-flex tw-justify-between tw-items-center tw-mt-4">
 			<div class="due-date {{if .Issue.IsOverdue}}text red{{end}}" {{if .Issue.IsOverdue}}data-tooltip-content="{{ctx.Locale.Tr "repo.issues.due_date_overdue"}}"{{end}}>
 				{{svg "octicon-calendar" 16 "tw-mr-2"}}
 				{{DateUtils.AbsoluteLong .Issue.DeadlineUnix}}
diff --git a/templates/repo/issue/view_content/sidebar/reference.tmpl b/templates/repo/issue/view_content/sidebar/reference.tmpl
index 98e2a19256..15d1add8c3 100644
--- a/templates/repo/issue/view_content/sidebar/reference.tmpl
+++ b/templates/repo/issue/view_content/sidebar/reference.tmpl
@@ -10,11 +10,8 @@
 			{{ctx.Locale.Tr "discussion.sidebar.reference"}}
 		</strong>
 	</span>
-	<p></p>
-	<div class="ui equal width compact grid">
-		<div class="row tw-items-center" data-tooltip-content="{{$issueReferenceLink}}">
-			<span class="text column truncate">{{$issueReferenceLink}}</span>
-			<button class="ui two wide button column tw-p-2" data-tooltip-content="{{ctx.Locale.Tr "copy"}}" data-clipboard-text="{{$issueReferenceLink}}">{{svg "octicon-copy" 14}}</button>
-		</div>
+	<div class="ui tw-flex tw-justify-between tw-items-end tw-mt-1" data-tooltip-content="{{$issueReferenceLink}}">
+		<span class="text column truncate">{{$issueReferenceLink}}</span>
+		<button class="ui two wide button column tw-p-2" data-tooltip-content="{{ctx.Locale.Tr "copy"}}" data-clipboard-text="{{$issueReferenceLink}}">{{svg "octicon-copy" 14}}</button>
 	</div>
 </div>
diff --git a/templates/repo/issue/view_content/sidebar/timetracking.tmpl b/templates/repo/issue/view_content/sidebar/timetracking.tmpl
index 62df3d7f38..7cc10b1a52 100644
--- a/templates/repo/issue/view_content/sidebar/timetracking.tmpl
+++ b/templates/repo/issue/view_content/sidebar/timetracking.tmpl
@@ -53,8 +53,7 @@
 	{{end}}
 	<div class="ui comments">
 		<span class="text"><strong>{{ctx.Locale.Tr "repo.issues.time_spent_from_all_authors" ($.Issue.TotalTrackedTime | Sec2Time)}}</strong></span>
-		<p></p>
-		<div>
+		<div class="tw-mt-4">
 			{{range $user, $trackedtime := .WorkingUsers}}
 				<div class="comment tw-mt-2">
 					<a class="avatar">