Skip to content

Every screen of the app, in both themes and at both widths - #8

Merged
aledbf merged 1 commit into
mainfrom
screens-catalog
Oct 22, 2026
Merged

aledbf merged 1 commit into
mainfrom
screens-catalog

Conversation

@aledbf

@aledbf aledbf commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

This adds the place where every screen of the dashboard is kept.

  • cmd/screens reads what spin's dashboard/e2e/catalog.spec.ts writes: catalog.json, one *.audit.json per screen and <name>.<theme>.<width>.png. It fails unless every screen has all four images and every finding names a screen in the catalog. It commits screens/screens.json and screens/audit.json. The images, about 300 per run, go up as screens.tar.gz on a screens-<release>-<commit> release instead of into git.
  • -check runs in the Gate. Until the first catalog is taken it says so and passes.
  • On main, Pages unpacks the release named in screens.json into screens/img.
  • screens.html shows each screen in light and dark side by side, at desktop, mobile or both widths. It has a summary of the audit by kind, filters by area, kind and finding, and each screen's findings with the element they are on.
  • The lockup's word is now spinbox.

The spin side is coming in its own PR.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

spin's catalog spec draws every route, tab, dialog and menu at 1440 and 390 px, light and dark,
and lists what each draws that the palette does not have. cmd/screens keeps the catalog and that
audit in screens/ and puts the images, hundreds a release, on a release of this repository rather
than in git; Pages unpacks them, and screens.html shows them side by side with their findings.
The lockup's word is spinbox, the product's name.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aledbf
aledbf merged commit 012c3ec into main Oct 22, 2026
2 checks passed
@aledbf
aledbf deleted the screens-catalog branch October 22, 2026 04:36
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