Repository navigation
feat(frontend): onboarding, skeleton loaders, scam interstitial, tx detail modal - #504
Merged
Merged
Conversation
… tx detail modal - Add 3-step OnboardingModal shown on first visit (quickex-onboarded in localStorage) - Replace dashboard/history spinners with DashboardSkeleton and TransactionTableSkeleton - Blocking ScamInterstitial for high-confidence signals, banner with 'Why this warning?' tooltip for low-confidence, analytics events on dismiss/proceed - Clickable, keyboard-activatable history rows opening TransactionDetailModal with Stellar Expert link, plus unit test Closes Viky207#477 Closes Viky207#478 Closes Viky207#479 Closes Viky207#480
|
@Qavahpaul Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Implements four frontend UX issues in one PR.
OnboardingModal(mounted in root layout) shows on first visit: What is QuickEx → Connect wallet (Freighter/Lobstr with install links) → Claim username (live availability check viacheckUsernameAvailability). Back/Skip at every step, completion or dismissal storesquickex-onboarded: true. Focus-trapped dialog, ESC to skip, progress announced to screen readers, bottom-sheet on mobile.DashboardSkeleton,TransactionRowSkeleton,TransactionTableSkeleton(greyanimate-pulse,aria-hiddenwith an sr-only status). Replaces all spinners on dashboard and history pages, including theSuspensefallbacks.ScamInterstitial(alertdialog, focus-trapped, underlying pageinert, pay buttons disabled) with the signal type, ≤60-word copy and a required "I understand the risk — proceed" confirmation. Low-confidence signals show a dismissable banner with a "Why this warning?" tooltip. Shown/dismissed/proceeded events go through a newtrackEventhelper (window.dataLayer+quickex:analyticsDOM event).ScamAlertnow accepts optionalsignalandconfidencefields.TransactionDetailModalshowing hash, amount + asset, sender, recipient, memo, fee, ledger, timestamp, plus a "View on Stellar Expert" link that opens in a new tab. Closes with ESC, a backdrop click or ×.ActivityFeedItemgains optionalfee/ledgerfields, mapped from the backend when it sends them. Unit test added.Notes
Closes #477
Closes #478
Closes #479
Closes #480