Skip to content

fix(website): improve revocation banner placement and wording#6426

Draft
theosanderson-agent wants to merge 1 commit into
mainfrom
fix/revocation-banner-placement-grammar
Draft

fix(website): improve revocation banner placement and wording#6426
theosanderson-agent wants to merge 1 commit into
mainfrom
fix/revocation-banner-placement-grammar

Conversation

@theosanderson-agent
Copy link
Copy Markdown
Collaborator

@theosanderson-agent theosanderson-agent commented May 13, 2026

Summary

Addresses #6425.

  • Move the version/revocation banner from the page-level banner slot (which sits above the page header area) into the sequence entry's main content, just below the accession title. The banner now reads as part of the entry it describes rather than as a top-of-page notice.
  • Fix the ungrammatical sentence on revocation versions:
    • Before: "It essentially contains no data, it's just a marker that all previous versions have been revoked."
    • After: "It essentially contains no data: it is just a marker that all previous versions have been revoked."

The leading sentence ("This is a revocation version.") is unchanged, so existing Playwright assertions that look for that text continue to pass.

Test plan

  • CI=1 npm run test (website unit tests)
  • npm run check-types
  • npm run format
  • Manual check: visit a revoked accession page; banner appears below the accession header within the content column
  • Manual check: visit a non-latest version page; "not the latest version" banner appears below the title

🤖 Generated with Claude Code

🚀 Preview: Add preview label to enable

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>
@claude claude Bot added the website Tasks related to the web application label May 13, 2026
@theosanderson theosanderson added preview Triggers a deployment to argocd and removed preview Triggers a deployment to argocd labels May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

website Tasks related to the web application

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants