fix: ensure spread attribute events are attached synchronously

[trueadm]

Fixes #14385. Interestingly, it seems the tests in #11230 still pass here so I assume that we've fixed this issue through some other changes in the codebase since.

[Rich-Harris]

preview: https://svelte-dev-git-preview-svelte-14387-svelte.vercel.app/

this is an automated message

[github-actions]

Playground

pnpm add https://pkg.pr.new/svelte@14387

[dummdidumm]

It works because input and change events are both delegated now. It will break on bindings that have listeners that are not delegated, like bind:paused: works on main, but breaks in this PR. That said, it already did not work for bindings combined with events that aren't delegated in the case of not spreading (example) which is arguably more common. So I think I'm still ok with this change.
Thoughts?

[trueadm]

So I think I'm still ok with this change. Thoughts?

I agree with you here.

[Rich-Harris]

I think I'm still ok with this change.

same. if it causes problems we can try and revisit it in a way that works for both spread and non-spread

[changeset-bot]

🦋 Changeset detected

Latest commit: 33b5434

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

This PR includes changesets to release 1 package

Name	Type
svelte	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