From 081c4e401705198f44f058502c1dcfc6de489a5d Mon Sep 17 00:00:00 2001
From: Xinacod <kajaobinna@gmail.com>
Date: Tue, 17 Mar 2026 12:35:21 +0100
Subject: [PATCH 1/2] Hide back button on home page and edit option on deleted
 reviews

NavBar: the back arrow was showing on every page including the root
home page where there's nowhere to go back to. Added a pathname check
so it only renders when you're not on '/'.

ReviewCard: the edit pencil was visible to the review author even after
the review had been soft-deleted (deleted_at set). Added a !deleted_at
guard so the edit option only shows up on reviews that haven't been
deleted.

Fixes #312, #290
---
 src/components/articles/ReviewCard.tsx |  2 +-
 src/components/common/NavBar.tsx       | 27 ++++++++++++--------------
 2 files changed, 13 insertions(+), 16 deletions(-)

diff --git a/src/components/articles/ReviewCard.tsx b/src/components/articles/ReviewCard.tsx
index 611036af..9d9cba22 100644
--- a/src/components/articles/ReviewCard.tsx
+++ b/src/components/articles/ReviewCard.tsx
@@ -168,7 +168,7 @@ const ReviewCard: FC<ReviewCardProps> = ({ review, refetch }) => {
               <div className="flex flex-col">
                 <span className="flex items-center gap-2 text-sm font-bold text-text-secondary">
                   {review.user.username}
-                  {review.is_author && (
+                  {review.is_author && !review.deleted_at && (
                     <>
                       <span className="text-[10px] font-normal text-text-tertiary">(You)</span>
                       {/* Fixed by Codex on 2026-02-15
diff --git a/src/components/common/NavBar.tsx b/src/components/common/NavBar.tsx
index 8b06e079..4824bb52 100644
--- a/src/components/common/NavBar.tsx
+++ b/src/components/common/NavBar.tsx
@@ -225,21 +225,18 @@ const NavBar: React.FC = () => {
     <header className="sticky top-0 z-[1000] w-full border-b border-common-contrast/40 bg-common-background/70 text-text-primary backdrop-blur-md">
       <nav className="container mx-auto flex items-center justify-between px-4 py-2">
         <div className="flex items-center">
-          {/* Fixed by Codex on 2026-02-15
-              Who: Codex
-              What: Convert the back icon into a real button.
-              Why: Icon-only divs are not keyboard accessible or announced by screen readers.
-              How: Wrap the icon in a button with an aria-label and click handler. */}
-          <button
-            type="button"
-            aria-label="Go back"
-            className="mr-4 flex size-8 items-center justify-center rounded-full text-primary hover:bg-common-minimal/60 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-functional-green/60"
-            onClick={() => {
-              router.back();
-            }}
-          >
-            <MoveLeft className="size-5" strokeWidth={1.5} />
-          </button>
+          {pathname !== '/' && (
+            <button
+              type="button"
+              aria-label="Go back"
+              className="mr-4 flex size-8 items-center justify-center rounded-full text-primary hover:bg-common-minimal/60 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-functional-green/60"
+              onClick={() => {
+                router.back();
+              }}
+            >
+              <MoveLeft className="size-5" strokeWidth={1.5} />
+            </button>
+          )}
           <div className="flex items-center gap-2">
             <Link href="/" className="flex items-center">
               <Image

From fcaa61797396346b8caa812438dcd06ecaef18f4 Mon Sep 17 00:00:00 2001
From: Xinacod <kajaobinna@gmail.com>
Date: Wed, 25 Mar 2026 20:04:03 +0100
Subject: [PATCH 2/2] address review: revert navbar change, fix deleted
 discussion edit button
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Revert the pathname !== '/' back button guard per reviewer feedback —
the back button is fine to show on the home page.

Also fixes the same deleted-item edit button issue in DiscussionCard:
canEditDiscussion was true for any author regardless of deleted_at.
Added !discussion.deleted_at guard to match the ReviewCard fix.

Closes #290
---
 src/components/articles/DiscussionCard.tsx |  2 +-
 src/components/common/NavBar.tsx           | 22 ++++++++++------------
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/src/components/articles/DiscussionCard.tsx b/src/components/articles/DiscussionCard.tsx
index e43a3b20..af740fe7 100644
--- a/src/components/articles/DiscussionCard.tsx
+++ b/src/components/articles/DiscussionCard.tsx
@@ -190,7 +190,7 @@ const DiscussionCard: React.FC<DiscussionCardProps> = ({
 
   // Check if user can resolve/unresolve (admin or discussion author)
   const canResolve = isCommunityArticle && (isAdmin || discussion.is_author);
-  const canEditDiscussion = !!discussion.is_author;
+  const canEditDiscussion = !!discussion.is_author && !discussion.deleted_at;
   const shouldShowActions = (canResolve || canEditDiscussion) && !isEditing;
 
   const { mutate: toggleResolved, isPending: isToggling } =
diff --git a/src/components/common/NavBar.tsx b/src/components/common/NavBar.tsx
index 4824bb52..e7353766 100644
--- a/src/components/common/NavBar.tsx
+++ b/src/components/common/NavBar.tsx
@@ -225,18 +225,16 @@ const NavBar: React.FC = () => {
     <header className="sticky top-0 z-[1000] w-full border-b border-common-contrast/40 bg-common-background/70 text-text-primary backdrop-blur-md">
       <nav className="container mx-auto flex items-center justify-between px-4 py-2">
         <div className="flex items-center">
-          {pathname !== '/' && (
-            <button
-              type="button"
-              aria-label="Go back"
-              className="mr-4 flex size-8 items-center justify-center rounded-full text-primary hover:bg-common-minimal/60 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-functional-green/60"
-              onClick={() => {
-                router.back();
-              }}
-            >
-              <MoveLeft className="size-5" strokeWidth={1.5} />
-            </button>
-          )}
+          <button
+            type="button"
+            aria-label="Go back"
+            className="mr-4 flex size-8 items-center justify-center rounded-full text-primary hover:bg-common-minimal/60 focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-functional-green/60"
+            onClick={() => {
+              router.back();
+            }}
+          >
+            <MoveLeft className="size-5" strokeWidth={1.5} />
+          </button>
           <div className="flex items-center gap-2">
             <Link href="/" className="flex items-center">
               <Image