Skip to content

feat(web): add welcome modal for new visitors#1857

Merged
tyler-dane merged 2 commits into
mainfrom
claude/exciting-lederberg-be651d
Jun 8, 2026
Merged

feat(web): add welcome modal for new visitors#1857
tyler-dane merged 2 commits into
mainfrom
claude/exciting-lederberg-be651d

Conversation

@tyler-dane

Copy link
Copy Markdown
Contributor

Summary

  • Adds a WelcomeModal component that shows automatically on first visit (unauthenticated users who haven't seen it before)
  • Dismissed state persists in localStorage via key compass.onboarding.has-seen-welcome
  • Start Now closes the modal; Log In closes the modal and opens the existing auth modal on the login view
  • Includes title, subtitle, FAQ accordion (native <details>/<summary>), X/LinkedIn social links, and Privacy/Terms footer
  • Mounted inside AuthModalProvider in CompassRequiredProviders so auth modal integration works

Test plan

  • Open app in fresh browser (no localStorage key, not logged in) → welcome modal appears
  • Click "Start Now" → modal closes, compass.onboarding.has-seen-welcome set in localStorage
  • Reload → modal does not reappear
  • Clear key, click "Log In" → welcome modal closes, auth login modal opens
  • FAQ items expand/collapse on click
  • External links (X, LinkedIn, Privacy, Terms) open in new tab
  • Logged-in users do not see the modal

🤖 Generated with Claude Code

tyler-dane and others added 2 commits June 8, 2026 18:39
Shows a first-visit modal with title, subtitle, FAQ accordion, social
links, and Start Now / Log In actions. Dismissed state persists via
localStorage (compass.onboarding.has-seen-welcome) and the modal is
suppressed for already-authenticated users.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Pre-set compass.onboarding.has-seen-welcome in storageState so the
WelcomeModal doesn't overlay the UI and block e2e interactions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tyler-dane tyler-dane merged commit ce6ac0d into main Jun 8, 2026
9 checks passed
@tyler-dane tyler-dane deleted the claude/exciting-lederberg-be651d branch June 8, 2026 23:55
@tyler-dane

Copy link
Copy Markdown
Contributor Author

Closes #1852

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