Skip to content

feat: add Stellar onboarding tour#39

Closed
kaycke1337 wants to merge 1 commit into
wraith-protocol:developfrom
kaycke1337:add-stellar-onboarding-tour
Closed

feat: add Stellar onboarding tour#39
kaycke1337 wants to merge 1 commit into
wraith-protocol:developfrom
kaycke1337:add-stellar-onboarding-tour

Conversation

@kaycke1337

Copy link
Copy Markdown

Summary

  • add a first-run Stellar onboarding tour with five steps for the send flow
  • persist dismissal in localStorage and add a footer restart entry point
  • keep tour copy in src/i18n/en.json and anchor the tour to header, wallet, recipient, amount, and submit UI
  • add Playwright e2e coverage for completion, restart, Escape dismissal, and refresh persistence

I used a small in-app tour controller instead of adding a tour dependency, keeping the shipped tour code minimal and avoiding another UI overlay package. The overlay does not trap clicks, supports Escape dismissal, restores focus after closing, and respects prefers-reduced-motion for scroll behavior.

Closes #17

Validation

  • npx prettier --check .
  • npx tsc --noEmit
  • git diff --check
  • npx vite build
  • npx playwright test

Build/test completed successfully. Vite still reports existing dependency/chunk-size warnings from bundled wallet SDK dependencies.

@vercel

vercel Bot commented May 29, 2026

Copy link
Copy Markdown

@kaycke1337 is attempting to deploy a commit to the truthixify's projects Team on Vercel.

A member of the Team first needs to authorize it.

@truthixify truthixify changed the base branch from main to develop June 1, 2026 15:47
@truthixify

Copy link
Copy Markdown
Contributor

The tour itself (OnboardingTour.tsx, 5 steps, dismiss + restart, en.json strings, Playwright e2e) matches #17 well. Same rebase blocker, branched from pre-develop. Rebase, drop reverts to StellarSend/Receive:

git fetch origin
git rebase origin/develop
git push --force-with-lease

Thanks @kaycke1337.

@truthixify

Copy link
Copy Markdown
Contributor

The 5-step OnboardingTour, i18n strings, dismiss + restart flow, and Playwright spec all match issue #17.

Real conflicts now in:

git fetch origin
git rebase origin/develop
git push --force-with-lease

For App.tsx keep both mounts (HelpButton AND OnboardingTour). For playwright.config.ts keep develop's existing config and add your onboarding spec alongside others. For StellarSend.tsx keep develop's content and add any tour-target data attributes you need.

Thanks @kaycke1337.

@truthixify

Copy link
Copy Markdown
Contributor

Closing as we wrap up this round of the Wraith × Drips × Stellar Wave program.

Thanks for the contribution. The work and review history stays on this PR for reference. A new set of issues will go up shortly under the same Stellar Wave label if you want to pick something fresh next round.

If you specifically want to land the work that's already in this PR (post-wave), feel free to reopen and rebase against develop. Otherwise, no action needed.

@truthixify truthixify closed this Jun 18, 2026
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.

First-time user onboarding tour for Stellar flows

2 participants