From 1084362f7c53877a912ca9cec7effed52b586a31 Mon Sep 17 00:00:00 2001
From: 0ko <0ko@noreply.codeberg.org>
Date: Sat, 1 Jun 2024 15:20:40 +0000
Subject: [PATCH] CSS improvements around commit status icons (#3948)

- make sure margins are all consistent and good, elements are not too close or too apart
- this also applies to "Show commit body" button
- remove unused code. The class `commit-status-link` doesn't exist in templates, nor I could find it on any related pages in case it's generated in runtime

## Preview

![](/attachments/9cf6d73a-8132-4f30-8094-5687d7dd98e9)

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/3948
Reviewed-by: Beowulf <beowulf@noreply.codeberg.org>
---
 web_src/css/repo.css | 23 ++++++++++++-----------
 1 file changed, 12 insertions(+), 11 deletions(-)

diff --git a/web_src/css/repo.css b/web_src/css/repo.css
index 91a0c8c336..066e81f63c 100644
--- a/web_src/css/repo.css
+++ b/web_src/css/repo.css
@@ -185,6 +185,14 @@ td .commit-summary {
   white-space: normal;
 }
 
+.commit-list .js-toggle-commit-body {
+  margin: 0 0 0 0.25em;
+}
+
+.commit-list .commit-status {
+  margin: 0 0 0 0.25em;
+}
+
 .latest-commit {
   display: flex;
   flex: 1;
@@ -193,6 +201,10 @@ td .commit-summary {
   text-overflow: ellipsis;
 }
 
+.latest-commit .commit-status {
+  margin-right: 0.25em;
+}
+
 @media (max-width: 767.98px) {
   .latest-commit .sha {
     display: none;
@@ -876,12 +888,6 @@ td .commit-summary {
   background: var(--color-orange-badge-hover-bg) !important;
 }
 
-.repository.view.issue .comment-list .timeline-item.event > .commit-status-link {
-  float: right;
-  margin-right: 8px;
-  margin-top: 4px;
-}
-
 .repository.view.issue .comment-list .timeline-item .comparebox {
   line-height: 32px;
   vertical-align: middle;
@@ -2467,11 +2473,6 @@ tbody.commit-list {
   }
 }
 
-.commit-list .commit-status-link {
-  display: inline-block;
-  vertical-align: middle;
-}
-
 .commit-body {
   margin: 0.25em 0;
   white-space: pre-wrap;