fix(dashboard): show Stack Auth logo on left of rebrand modal#1495
Conversation
The Stack Auth → Hexclave rebrand modal in #1493 referenced /logo.svg and /logo-bright.svg expecting the Stack Auth mark, but the visible- rebrand flip in #1481 replaced those public files with the Hexclave benzene mark. The illustration ended up rendering Hexclave on both sides instead of 'Stack Auth → Hexclave'. Preserve the pre-rebrand Stack Auth SVGs under dedicated filenames (stack-auth-logo{,-bright}.svg) so future brand-asset edits to /logo.svg can't silently break the modal, and point the modal's left-hand <Image> at them.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedPull request was closed or merged during review No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe rebrand modal illustration component is updated to reference dedicated Stack Auth logo assets instead of generic logo filenames, ensuring the correct visual presentation of the pre-rebrand branding during the modal flow. ChangesRebrand Modal Illustration
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Greptile SummaryThis PR fixes the rebrand modal illustration, which was showing the Hexclave logo on both sides after PR #1481 replaced
Confidence Score: 5/5Safe to merge — the change is two path swaps in a purely decorative modal illustration with no logic, auth, or data-flow impact. The diff is surgical: two new static SVG assets and two src= string replacements in a decorative component. The dismissal logic, user gate, localStorage handling, and environment checks are completely untouched. The new filenames are isolated from the canonical /logo.svg brand asset, so future brand flips won't regress this again. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[RebrandIllustration renders] --> B{Theme}
B -- light --> C["Image src='/stack-auth-logo.svg' block dark:hidden"]
B -- dark --> D["Image src='/stack-auth-logo-bright.svg' hidden dark:block"]
C --> E[Arrow SVG]
D --> E
E --> F["Image src='/hexclave-icon.svg'"]
F --> G[Modal illustration complete]
Reviews (1): Last reviewed commit: "fix(dashboard): show Stack Auth logo on ..." | Re-trigger Greptile |
1 participant
Summary
The Stack Auth → Hexclave rebrand modal added in #1493 rendered the Hexclave logo on both sides of its illustration instead of Stack Auth → Hexclave.
Root cause
`hexclave-rebrand-modal.tsx` referenced `/logo.svg` and `/logo-bright.svg` for the left-hand "Stack Auth" mark. But the visible-rebrand flip in #1481 replaced those files in `apps/dashboard/public/` with the Hexclave benzene mark — so post-merge, both `![]()
` slots resolved to the Hexclave logo.
Fix
The originals on `origin/dev` (`/logo.svg`, `/logo-bright.svg` = Hexclave) are untouched.
Verification
Summary by cubic
Fixes the rebrand modal to show the Stack Auth logo on the left instead of rendering Hexclave on both sides. Restores pre-rebrand assets as
stack-auth-logo.svgandstack-auth-logo-bright.svgand updates the modal to use them for light/dark themes.Written for commit f4ed262. Summary will update on new commits. Review in cubic
Summary by CodeRabbit