Skip to content

Bootstrap status easter egg#500

Open
pietro909 wants to merge 2 commits intomasterfrom
bootstrap-status-easter-egg-2
Open

Bootstrap status easter egg#500
pietro909 wants to merge 2 commits intomasterfrom
bootstrap-status-easter-egg-2

Conversation

@pietro909
Copy link
Copy Markdown
Contributor

Summary

Bootstrap status messages and git commit hash are now hidden by default during wallet initialization. Triple-tap the loading logo to reveal them—useful for debugging without cluttering the boot screen for end users.

How to use the easter egg

  • Tap the logo 3 times (within 600ms between taps) during boot → messages appear
  • Tap 3 times again → messages disappear (toggle on/off)
  • Tap only 1–2 times → counter resets silently

Changes

  • Added statusRevealed state to LoadingLogo component
  • Added tap counter with 600ms debounce window
  • Made logo tappable by wrapping it in a div with pointerEvents: auto
  • Gated both status text and git commit hash render on the statusRevealed flag

pietro909 and others added 2 commits March 31, 2026 08:36
Status messages and commit hash during wallet initialization are now hidden
by default. Tap the loading logo three times (within 600ms between taps) to
toggle them on/off. This keeps the boot animation clean for normal users while
preserving debug visibility for developers and testers.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@pietro909 pietro909 requested review from Kukks and sahilc0 and removed request for sahilc0 March 31, 2026 06:40
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 31, 2026

Warning

Rate limit exceeded

@pietro909 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 5 minutes and 39 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 5 minutes and 39 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 670b0c70-25d2-4f8c-b63a-87c8159a1d35

📥 Commits

Reviewing files that changed from the base of the PR and between 9300408 and c7e86e0.

📒 Files selected for processing (2)
  • .gitignore
  • src/components/LoadingLogo.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bootstrap-status-easter-egg-2

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@arkanaai
Copy link
Copy Markdown

arkanaai bot commented Mar 31, 2026

🔍 Review — Bootstrap status easter egg

Scope: Hides boot status text + git commit hash behind a triple-tap on the loading logo. UI-only change, no protocol/security impact.

Code review:

Tap detection logic is clean. Counter increments per tap, resets after 600ms idle via setTimeout. Hitting 3 toggles statusRevealed and resets the counter. Timer cleanup on unmount via the dedicated useEffect — no leaks.

useCallback on handleLogoTap — no dependencies needed since it only uses refs and a state setter. Correct.

Conditional rendering gates both text and showBackground on statusRevealed — so the status block and background overlay are fully hidden until revealed.

Minor observations:

  1. The showBackground gate (line 195): this was previously tied to exitMode !== 'none', now it also requires statusRevealed. Is that intentional? If the exit animation relies on the background overlay being visible, gating it behind statusRevealed means the background won't show during exit unless the user has triple-tapped. Worth confirming this doesn't affect the exit animation flow.

  2. .gitignore additions (*.har, *.agents.md): Unrelated to the easter egg but sensible — HAR files and agent markdown files shouldn't be committed.

  3. Accessibility: The tappable wrapper is a plain div with onClick. For keyboard/screen-reader users, consider adding role="button" and tabIndex={0} with an onKeyDown handler — though for a hidden easter egg this is low priority.

No blocking issues. Clean implementation of a nice UX touch.

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