Skip to content

feat(web): polish site experience and app states - #41

Merged
brilliantinsane merged 2 commits into
mainfrom
codex/web-experience-polish
Jul 28, 2026
Merged

feat(web): polish site experience and app states#41
brilliantinsane merged 2 commits into
mainfrom
codex/web-experience-polish

Conversation

@brilliantinsane

Copy link
Copy Markdown
Owner

Summary

  • add configurator route loading with layout-matched skeletons while keeping the static hero and footer outside the nuqs Suspense boundary
  • add a 404-aligned error recovery state, clipboard toasts, accessible validation associations, focus treatments, and improved header/mobile navigation semantics
  • add command palette infrastructure and keyboard navigation while intentionally keeping its visual trigger hidden
  • serve the branded Open Graph image through the App Router image convention
  • cover the new loading, error, command palette, copy feedback, accessibility, navigation, and SEO behavior with tests

Why

The web experience had a blank or shifting configurator transition, a generic error card that did not match the site, limited clipboard feedback, and several accessibility and navigation inconsistencies. The Open Graph image also needed to follow the App Router convention without changing route-specific metadata.

Impact

Users get a more stable configurator transition, consistent error recovery, clearer interaction feedback, and stronger keyboard and assistive-technology behavior. The static configurator shell now renders immediately instead of waiting for URL-backed configuration state.

Validation

  • pnpm test - 72 tests passed
  • pnpm typecheck
  • pnpm lint
  • git diff --check

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tenkit-web Ready Ready Preview, Comment Jul 28, 2026 1:48pm

@brilliantinsane

Copy link
Copy Markdown
Owner Author

@codex

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 65a81ee45a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread apps/web/components/ui/command.tsx Outdated
@brilliantinsane

Copy link
Copy Markdown
Owner Author

@codex

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Nice work!

Reviewed commit: 1d16427a67

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@brilliantinsane
brilliantinsane marked this pull request as ready for review July 28, 2026 14:04
@brilliantinsane
brilliantinsane merged commit ea59cba into main Jul 28, 2026
3 checks passed
@brilliantinsane
brilliantinsane deleted the codex/web-experience-polish branch July 28, 2026 14:04
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