Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[svelte 5] Cross of variables in event functions #9653

Closed
CodexKI opened this issue Nov 26, 2023 · 1 comment · Fixed by #10620
Closed

[svelte 5] Cross of variables in event functions #9653

CodexKI opened this issue Nov 26, 2023 · 1 comment · Fixed by #10620
Assignees
@trueadm
Milestone
5.0

Comments

@CodexKI
Copy link

CodexKI commented Nov 26, 2023 •
edited
Loading

Describe the bug

the variable "method" is getting conffused with the destructured variable "method" of object in array methods

Reproduction

svelte 5 playground

Logs

No response

System Info

return undefined

Severity

annoyance
@dummdidumm
Copy link
Member

dummdidumm commented Nov 27, 2023 •
edited
Loading

Another hoisting-related bug: hoisting should not be done here because the variable name is shadowed

@dummdidumm dummdidumm added this to the 5.0 milestone Nov 27, 2023
@trueadm trueadm self-assigned this Feb 23, 2024
@trueadm trueadm mentioned this issue Feb 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trueadm trueadm

Labels
None yet
Projects
None yet
Milestone
5.0
Development

Successfully merging a pull request may close this issue.

fix: improve event delegation with shadowed bindings sveltejs/svelte
3 participants
@trueadm @dummdidumm @CodexKI