From a1769b7ec32a3be2fb4995ced14ec1805a156254 Mon Sep 17 00:00:00 2001
From: theosanderson-agent <theo@theo.io>
Date: Wed, 13 May 2026 15:52:13 +0100
Subject: [PATCH] fix(website): move revocation banner inside sequence entry
 and tidy text
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Place the version/revocation banner below the accession title rather than
above the page header, so it visually belongs to the sequence entry it
describes. Also fix the ungrammatical sentence in the revocation banner
("It essentially contains no data, it's just a marker..." → "It
essentially contains no data: it is just a marker...").

Closes #6425

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../SequenceDetailsPage/SequencesBanner.tsx     |  2 +-
 .../pages/seq/[accessionVersion]/index.astro    | 17 ++++++-----------
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/website/src/components/SequenceDetailsPage/SequencesBanner.tsx b/website/src/components/SequenceDetailsPage/SequencesBanner.tsx
index 20036d6f87..d199752587 100644
--- a/website/src/components/SequenceDetailsPage/SequencesBanner.tsx
+++ b/website/src/components/SequenceDetailsPage/SequencesBanner.tsx
@@ -39,7 +39,7 @@ const SequencesBanner: FC<SequencesBannerProps> = ({ sequenceEntryHistory, acces
             {ownHistoryEntry?.isRevocation && (
                 <div className='bg-red-100 border-l-4 border-red-500 text-red-700 p-3' role='alert'>
                     <p className='font-bold'>
-                        This is a revocation version. It essentially contains no data, it's just a marker that all
+                        This is a revocation version. It essentially contains no data: it is just a marker that all
                         previous versions have been revoked.
                     </p>
                 </div>
diff --git a/website/src/pages/seq/[accessionVersion]/index.astro b/website/src/pages/seq/[accessionVersion]/index.astro
index 5bf2c72664..7ce3b397c6 100644
--- a/website/src/pages/seq/[accessionVersion]/index.astro
+++ b/website/src/pages/seq/[accessionVersion]/index.astro
@@ -42,17 +42,6 @@ const sequenceFlaggingConfig = getWebsiteConfig().sequenceFlagging;
     title={sequenceDetailsTableData.isOk() ? accessionVersion : 'Sequence not found'}
     implicitOrganism={sequenceDetailsTableData.isOk() ? sequenceDetailsTableData.value.organism : undefined}
 >
-    {
-        sequenceDetailsTableData.isOk() &&
-            sequenceDetailsTableData.value.result.type === SequenceDetailsTableResultType.TABLE_DATA && (
-                <div slot='banner'>
-                    <SequencesBanner
-                        sequenceEntryHistory={sequenceDetailsTableData.value.result.sequenceEntryHistory}
-                        accessionVersion={accessionVersion}
-                    />
-                </div>
-            )
-    }
     {
         sequenceDetailsTableData.match(
             ({ organism, result }) => {
@@ -72,6 +61,12 @@ const sequenceFlaggingConfig = getWebsiteConfig().sequenceFlagging;
                                 showDownloadAndReport && getSegmentNames(getReferenceGenomes(organism)).length > 0
                             }
                         />
+                        {result.type === SequenceDetailsTableResultType.TABLE_DATA && (
+                            <SequencesBanner
+                                sequenceEntryHistory={result.sequenceEntryHistory}
+                                accessionVersion={accessionVersion}
+                            />
+                        )}
                         {result.type === SequenceDetailsTableResultType.TABLE_DATA &&
                             (result.isRevocation ? (
                                 <RevocationEntryDataTable