Skip to content

fix(web): render textarea spread values as content - #3287

Open
javascript-unsafe wants to merge 1 commit into
solidjs:nextfrom
javascript-unsafe:fix/ssr-textarea-spread-value-3286
Open

fix(web): render textarea spread values as content#3287
javascript-unsafe wants to merge 1 commit into
solidjs:nextfrom
javascript-unsafe:fix/ssr-textarea-spread-value-3286

Conversation

@javascript-unsafe

@javascript-unsafe javascript-unsafe commented Sep 5, 2026

Copy link
Copy Markdown

Summary

  • Render textarea value and defaultValue props from SSR spreads as text content.
  • Keep the SSR runtime path aligned with compiler-specialized textarea output.
  • Cover declared and spread-supplied values in both attribute orderings.

Test plan

  • pnpm --dir packages/web exec vitest run --config vite.config.server.mjs
  • pnpm --dir packages/web exec vitest run --config vite.config.hydrate.mjs
  • pnpm --dir packages/web exec vitest run
  • JSX_COMPILER=babel pnpm --dir packages/web exec vitest run --config vite.config.server.mjs test/server/spread-function-source.spec.tsx

@changeset-bot

changeset-bot Bot commented Sep 5, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c5477e6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 11 packages
Name Type
@solidjs/web Patch
@solidjs/babel-plugin Patch
@solidjs/element Patch
@solidjs/h Patch
@solidjs/html Patch
test-integration Patch
@solidjs/compiler Patch
@solidjs/diagnostics Patch
@solidjs/signals Patch
solid-js Patch
@solidjs/universal Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codspeed-hq

codspeed-hq Bot commented Sep 5, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 136 untouched benchmarks


Comparing javascript-unsafe:fix/ssr-textarea-spread-value-3286 (c5477e6) with next (ace227e)

Open in CodSpeed

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