Skip to content

Remove the landing tagline and format note; say "medical scan" on the dropzone - #356

Merged
aperson30 merged 1 commit into
mainfrom
fix/medical-scan-wording
Nov 13, 2026
Merged

aperson30 merged 1 commit into
mainfrom
fix/medical-scan-wording

Conversation

@aperson30

Copy link
Copy Markdown
Collaborator

What

  • Workspace landing page: removes the paragraph under the stats row ("Browse the library in 2D and 3D, upload your own CT for AI segmentation, and annotate or refine the results."), including its text constant, its styles and the same sentence in the crawler summary in index.html. The hero is a centered column, so the stats row and the upload card stay centered.
  • Upload dropzone: "Drop a CT scan here" becomes "Drop a medical scan here", and the "NIfTI (.nii, .nii.gz) or a DICOM series" line under it is removed.

Tests

  • landing.test.tsx: the stats row ends the hero and the tagline is gone.
  • crawler.test.ts, routes.smoke.test.tsx: pin the removed text and the new wording.
  • Ablation: restoring the format line fails the upload smoke test.
  • Locally: typecheck, ESLint (0 errors), the affected vitest files and npm run build pass. Full CI not run locally.

Not changed

  • The "CT Volumes" stat label, the site title/description, llms.txt, and the "Upload a CT" wording in the hidden crawler summary still say CT.
  • No access-policy or backend change.

… dropzone

On the workspace landing page, remove the paragraph under the stats row
("Browse the library in 2D and 3D, ...") along with its constant,
styles and the matching sentence in the crawler summary in index.html.
The hero is a centered column, so the stats row and the upload card stay
centered.

On the upload dropzone, change "Drop a CT scan here" to "Drop a medical
scan here" and remove the "NIfTI (.nii, .nii.gz) or a DICOM series"
line under it.

Tests: the landing test now checks that the stats row ends the hero and
the tagline is gone; the crawler and upload smoke tests pin the removed
text. Ablation: restoring the format line fails the upload smoke test.
@aperson30
aperson30 merged commit 4fe16bb into main Nov 13, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant