From 0c1b441ed260ae70fe442cfa4f9fb0ab4025be69 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C5=ABdolfs=20O=C5=A1i=C5=86=C5=A1?= <rudolfs@osins.org>
Date: Mon, 17 Mar 2025 16:27:43 -0600
Subject: [PATCH] Show commit list in review view

---
 src/components/Changes.svelte | 63 +++++++++++++++--------------------
 src/components/Review.svelte  |  1 -
 2 files changed, 27 insertions(+), 37 deletions(-)

diff --git a/src/components/Changes.svelte b/src/components/Changes.svelte
index d216f49ba..066995d9a 100644
--- a/src/components/Changes.svelte
+++ b/src/components/Changes.svelte
@@ -17,17 +17,10 @@
     patchId: string;
     revision: Revision;
     rid: string;
-    showCommits?: boolean;
     codeComments?: CodeComments;
   }
 
-  const {
-    patchId,
-    revision,
-    rid,
-    showCommits = true,
-    codeComments,
-  }: Props = $props();
+  const { patchId, revision, rid, codeComments }: Props = $props();
 
   let hideChanges = $state(false);
   let filesExpanded = $state(true);
@@ -118,35 +111,33 @@
 </div>
 
 <div class:hide={hideChanges}>
-  {#if showCommits}
-    {#await loadCommits(rid, revision.base, revision.head) then commits}
-      <div style:margin-bottom="1rem">
-        <CommitsContainer expanded={filesExpanded}>
-          {#snippet leftHeader()}
-            <div class="txt-semibold">Commits</div>
-          {/snippet}
-          {#snippet children()}
-            <div style:padding="0 1rem">
-              <div
-                class="global-flex txt-small"
-                style:color="var(--color-foreground-dim)">
-                <Icon name="branch" /><Id id={revision.base} variant="commit" />
-                <div class="global-counter">base</div>
-              </div>
-              <div class="commits">
-                {#each commits.reverse() as commit}
-                  <div class="commit" style:position="relative">
-                    <div class="commit-dot"></div>
-                    <CobCommitTeaser {commit} />
-                  </div>
-                {/each}
-              </div>
+  {#await loadCommits(rid, revision.base, revision.head) then commits}
+    <div style:margin-bottom="1rem">
+      <CommitsContainer expanded={filesExpanded}>
+        {#snippet leftHeader()}
+          <div class="txt-semibold">Commits</div>
+        {/snippet}
+        {#snippet children()}
+          <div style:padding="0 1rem">
+            <div
+              class="global-flex txt-small"
+              style:color="var(--color-foreground-dim)">
+              <Icon name="branch" /><Id id={revision.base} variant="commit" />
+              <div class="global-counter">base</div>
             </div>
-          {/snippet}
-        </CommitsContainer>
-      </div>
-    {/await}
-  {/if}
+            <div class="commits">
+              {#each commits.reverse() as commit}
+                <div class="commit" style:position="relative">
+                  <div class="commit-dot"></div>
+                  <CobCommitTeaser {commit} />
+                </div>
+              {/each}
+            </div>
+          </div>
+        {/snippet}
+      </CommitsContainer>
+    </div>
+  {/await}
 
   {#await loadHighlightedDiff(rid, revision.base, revision.head)}
     <span class="txt-small">Loading…</span>
diff --git a/src/components/Review.svelte b/src/components/Review.svelte
index ac8bdddeb..394ff38f5 100644
--- a/src/components/Review.svelte
+++ b/src/components/Review.svelte
@@ -387,7 +387,6 @@
       threads: codeCommentThreads,
     }}
     rid={repo.rid}
-    showCommits={false}
     {patchId}
     {revision} />
 </div>