Skip to content

style(claim): remove duplicate auth pills and apply status styling to top auth actions#284

Merged
GsCommand merged 1 commit into
mainfrom
codex/clean-up-step-1-in-public/claim.html
May 22, 2026
Merged

style(claim): remove duplicate auth pills and apply status styling to top auth actions#284
GsCommand merged 1 commit into
mainfrom
codex/clean-up-step-1-in-public/claim.html

Conversation

@GsCommand
Copy link
Copy Markdown
Contributor

Motivation

  • Simplify Step 1 UI by removing redundant lower colored status pills and surface connection/auth status on the existing top controls.
  • Make the top-row Connect Wallet / Sign-In with Ethereum controls visually represent completed status (blue/green) while preserving their action semantics.

Description

  • Removed the second/lower auth pill row in public/claim.html and kept the status text block (Authentication verified., Wallet connected: 0x..., and mode hint) intact.
  • Added reusable CSS classes .auth-action, .auth-action.pending, .auth-action.connected, and .auth-action.authenticated and applied them to the top wallet/SIWE buttons so they show pending/connected/authenticated styling (blue/green) without changing sizing/padding.
  • Updated setAuthStatusClasses to toggle the new classes on the top buttons (walletConnectBtn and siweAuthBtn) instead of updating removed lower-pill elements, leaving authentication logic unchanged.
  • No behavior/logic changes were made to connectWallet, signInWithEthereum, SIWE message construction, /api/auth/* interactions, step flow, or claim submission; this is a visual-only cleanup in public/claim.html.

Testing

  • Ran npm test; all tests passed (42/42 passing).
  • No backend or auth logic tests were modified, and existing test suite completed successfully.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

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

Project Deployment Actions Updated (UTC)
commandlayer-commandlayer-org Ready Ready Preview, Comment May 22, 2026 8:29pm
commandlayer-org Ready Ready Preview, Comment May 22, 2026 8:29pm
commandlayer-org111 Ready Ready Preview, Comment May 22, 2026 8:29pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant