Skip to content

Conversation

@titoBouzout
Copy link
Contributor

@titoBouzout titoBouzout commented Nov 2, 2025

It deopts spreads, aka spreads should work the same as in JavaScript native.

It fixes:

Notes:

  • There may be some possibility for optimization. But I prefer if this works the same as native first, because if not we introduce behaviour that we cant change, (the reason we cannot merge this fix in solid v1). If this works the same as native, then we can see how to optimize it keeping that behaviour in mind.
  • Same should be done in SSR
  • use: and ref arent merged, future of both seem to be undecided.
  • It also overwrites duplicated attributes <div c="1" c="2"/> discards c="1" producing <div c="2"/>

Problems:

  • merging ref and use seems challenging

@titoBouzout
Copy link
Contributor Author

Also merged this <div c="1" c="2"/> that produces <div c="2"/>, to make it consistent.

I am currently being blocked by use:, I am proposing to just use namespaced refs instead. https://discord.com/channels/722131463138705510/1435384712443990096

@titoBouzout titoBouzout marked this pull request as ready for review November 5, 2025 20:19
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