You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Refactors DevModeOverlay from a repeating-linear-gradient approach to an SVG grid mask with DEV_MODE_GRID constants, removes its gradient prop, and updates all consumers (CheckoutForm, TestPaymentMethod, CardClerkAndPagesTag). Adds flush-mode CSS overrides in CardRoot to hide the overlay and center the notice, adjusts spacing in CardFooter and Navbar, and extends CardRoot tests for raised vs. flush dev-mode indicator behavior.
Replaces the repeating-gradient maskImage implementation with a DEV_MODE_GRID-driven SVG tile generator that produces two masked Box layers. Removes the gradient prop entirely. Adds data-clerk-dev-mode-overlay to the wrapper and data-clerk-dev-mode-notice to DevModeNotice, and removes the fontWeight default from DevModeNotice.
Both components import DevModeOverlay, replace their inline gradient overlay blocks with <DevModeOverlay />, and add overflow: 'hidden' to their containers.
Card layout, spacing, and flush-mode suppression packages/ui/src/elements/Card/CardClerkAndPagesTag.tsx, packages/ui/src/elements/Card/CardFooter.tsx, packages/ui/src/elements/Card/CardRoot.tsx, packages/ui/src/elements/Navbar.tsx
CardClerkAndPagesTag removes isolation: 'isolate' and the gradient={0} prop, and increases the Col gap token to $4. CardFooter adds a negative marginBottom calc for the notice. CardRoot.getFlushElements hides [data-clerk-dev-mode-overlay] and centers [data-clerk-dev-mode-notice] with adjusted margin/padding. Navbar passes devModeNoticeSx with matching padding and bottom margin to Card.ClerkAndPagesTag.
CardRoot tests and changeset packages/ui/src/elements/Card/__tests__/CardRoot.test.tsx, .changeset/clear-adults-pump.md
Adds ClerkInstanceContext, EnvironmentProvider, and CardFooter imports, a devEnvironment stub, and a renderCardWithFooter helper. Extends flush tests to assert overlay hidden and notice alignment. Adds a new describe block covering raised (overlay + notice visible) vs flush (notice visible, overlay hidden) behavior. Changeset records a patch bump for @clerk/ui.
Estimated code review effort
🎯 3 (Moderate) | ⏱️ ~20 minutes
Suggested reviewers
kylemac
Poem
🐰 Hop hop, the gradient's gone today,
SVG grids have come to play!
A mask of tiles, a notice pinned below,
In flush or raised, the dev badge glows.
No more gradient={0} in sight —
The bunny refactored overnight! ✨
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
✅ Passed
The title 'feat(ui): Dev mode banner updates' directly describes the main change in the changeset: refactoring the dev mode banner/overlay implementation across multiple UI components with new styling and grid overlay design.
Docstring Coverage
✅ Passed
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check
✅ Passed
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
✅ Passed
Check skipped because no linked issues were found for this pull request.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches📝 Generate docstrings
Create stacked PR
Commit on current branch
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.
Replaces the dev mode banner with a faded pixel-grid overlay behind a
center-peaked accent line. Adds an oval mask (configurable curve, fade
strength, and center softness), matches the "Development mode" text to
the "Secured by" text, and adjusts footer spacing. Also fixes a type
error from the removed DevModeOverlay `gradient` prop.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The text Col already paints above the absolutely-positioned dev mode
overlay via DOM order with position: relative, so the explicit zIndex
and isolation are unnecessary.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Drops the isolation, content wrapper, and z-index lift from the test
card panels. The dots render over the content like the prior striped
pattern did; only position: relative (to anchor the overlay) and
overflow: hidden (to clip the grid) remain.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
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.
Description
Updates dev mode banner implementation.
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change
Summary by CodeRabbit
Bug Fixes
Style
Tests