Skip to content

fix: make <Head> work in client#3252

Merged
bartlomieju merged 18 commits into
mainfrom
fix-layout-thrashign
Mar 29, 2026
Merged

fix: make <Head> work in client#3252
bartlomieju merged 18 commits into
mainfrom
fix-layout-thrashign

Conversation

@marvinhagemeister
Copy link
Copy Markdown
Contributor

@marvinhagemeister marvinhagemeister commented Aug 25, 2025

Not sure how I missed this :S

Closes #3310

bartlomieju and others added 3 commits March 25, 2026 19:53
Resolve conflict in dev_server_test.ts: keep all new tests from main
and update Head test to use persistent demoServer pattern.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…op delete

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bartlomieju bartlomieju force-pushed the fix-layout-thrashign branch from f23c26d to c1f90bb Compare March 26, 2026 08:20
bartlomieju and others added 5 commits March 26, 2026 09:32
- Dynamic meta updates: verify meta content changes reactively on re-render
- Link element: verify <link rel="canonical"> works in Head
- Multiple islands with Head: verify two islands can each inject Head elements
- Title updates multiple times: verify repeated state changes update title
- SSR link test: verify link element replacement in server rendering

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Skip href from cache key for <link> elements (mirroring how content is
skipped for <meta>), so a route's <Head> can override link elements
from the app wrapper. Also add client-side matching by rel attribute.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Document dynamic head updates from islands with example
- Add <link> element deduplication by rel attribute to the list
- Fix em dashes in docs to match style guide

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bartlomieju bartlomieju merged commit a79b155 into main Mar 29, 2026
7 of 9 checks passed
@bartlomieju bartlomieju deleted the fix-layout-thrashign branch March 29, 2026 19:21
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.

<Head> component doesn't work client-side

3 participants