WIP: Review page

- something is borked in the router, need to investigate why we need the
  $effect to reload patches when switching sidebar

- hover states for review teasers

- start populating the review page

src/components/PatchTeaser.svelte

@@ -81,7 +81,13 @@
     if (loadPatch) {
       loadPatch(rid, patch.id);
     } else {
-      void push({ resource: "repo.patch", rid, patch: patch.id, status });
+      void push({
+        resource: "repo.patch",
+        rid,
+        patch: patch.id,
+        status,
+        reviewId: undefined,
+      });
     }
   }}>
   <div class="global-flex">

src/components/ReviewTeaser.svelte

@@ -1,4 +1,5 @@
 <script lang="ts">
+  import type { PatchStatus } from "@app/views/repo/router";
   import type { Review } from "@bindings/cob/patch/Review";
 
   import {
@@ -10,13 +11,16 @@
   import Icon from "./Icon.svelte";
   import Markdown from "./Markdown.svelte";
   import NodeId from "@app/components/NodeId.svelte";
+  import { push } from "@app/lib/router";
 
   interface Props {
     rid: string;
     review: Review;
+    patchId: string;
+    status: PatchStatus | undefined;
   }
 
-  const { rid, review }: Props = $props();
+  const { rid, review, patchId, status }: Props = $props();
 
   const header = $derived.by(() => {
     if (!review.verdict) {
@@ -86,7 +90,22 @@
   }
 </style>
 
-<div class="review" style:background-color={backgroundColor(review.verdict)}>
+<!-- svelte-ignore a11y_click_events_have_key_events -->
+<div
+  tabindex="0"
+  role="button"
+  class="review"
+  style:cursor="pointer"
+  style:background-color={backgroundColor(review.verdict)}
+  onclick={() => {
+    void push({
+      resource: "repo.patch",
+      rid,
+      patch: patchId,
+      status,
+      reviewId: review.id,
+    });
+  }}>
   <div class="icon" style:color={color(review.verdict)}>
     <Icon name={icon(review.verdict)} />
   </div>

src/components/Revision.svelte

@@ -4,6 +4,7 @@
   import type { Config } from "@bindings/config/Config";
   import type { Diff } from "@bindings/diff/Diff";
   import type { Embed } from "@bindings/cob/thread/Embed";
+  import type { PatchStatus } from "@app/views/repo/router";
   import type { Revision } from "@bindings/cob/patch/Revision";
   import type { Thread } from "@bindings/cob/thread/Thread";
   import type { Verdict } from "@bindings/cob/patch/Verdict";
@@ -39,11 +40,12 @@
     patchId: string;
     revision: Revision;
     config: Config;
+    status: PatchStatus | undefined;
     reload: () => Promise<void>;
   }
 
   /* eslint-disable prefer-const */
-  let { rid, repoDelegates, patchId, revision, config, reload }: Props =
+  let { rid, repoDelegates, patchId, revision, config, status, reload }: Props =
     $props();
   /* eslint-enable prefer-const */
 
@@ -407,7 +409,7 @@
   {#if revision.reviews && revision.reviews.length}
     <div class:hide={hideReviews} style:margin-top="1rem">
       {#each revision.reviews as review}
-        <ReviewTeaser {rid} {review} />
+        <ReviewTeaser {rid} {review} {patchId} {status} />
       {/each}
     </div>
   {/if}