Skip to content

Wrap layout with @policyengine/ui-kit PolicyEngineShell#9

Merged
MaxGhenis merged 3 commits into
mainfrom
fix/add-pe-shell
May 15, 2026
Merged

Wrap layout with @policyengine/ui-kit PolicyEngineShell#9
MaxGhenis merged 3 commits into
mainfrom
fix/add-pe-shell

Conversation

@MaxGhenis
Copy link
Copy Markdown
Contributor

Wraps the app's root layout with <PolicyEngineShell country="us"> from @policyengine/ui-kit ^0.12.0 so policyengine.org/us/* renders the PolicyEngine site header + footer around this tool. Adds the ui-kit dependency. Layout edit is mechanical and idempotent.

MaxGhenis and others added 3 commits May 14, 2026 19:40
policyengine.org/us/* serves this app through a multizone
rewrite, but the destination wasn't rendering the PolicyEngine site
header/footer. Wrap the existing layout with PolicyEngineShell so the
PE nav + footer wrap the tool's own UI.

Adds @policyengine/ui-kit ^0.12.0 dependency.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The auto-generated wrap put analytics Scripts and the SEO noscript
inside <PolicyEngineShell>. Move them back to the body root so the
shell only wraps {children}, matching the Coverage-Compass pattern.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@MaxGhenis MaxGhenis merged commit 69ec1f8 into main May 15, 2026
2 checks passed
@MaxGhenis MaxGhenis deleted the fix/add-pe-shell branch May 15, 2026 00:03
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