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

Can't bind:this a nested component to a dynamic object property #2333

Closed
PixievoltNo1 opened this issue Mar 29, 2019 · 7 comments · Fixed by #3079
Closed

Can't bind:this a nested component to a dynamic object property #2333

PixievoltNo1 opened this issue Mar 29, 2019 · 7 comments · Fixed by #3079
Labels
bug

Comments

@PixievoltNo1
Copy link

PixievoltNo1 commented Mar 29, 2019 •
edited
Loading

I get a compilation error when I try to do so:
https://v3.svelte.technology/repl?version=3.0.0-beta.21&gist=9b88ce63138ea76b7b64df38a685587f

This bug doesn't show when the bind:this is on an HTML element, or when its right-hand side isn't using square brackets to reference an object property - it specifically needs to be a nested component and a dynamic object property.
@PixievoltNo1
Copy link
Author

Bug is still present in beta 26: https://v3.svelte.technology/repl?version=3.0.0-beta.26&gist=9b88ce63138ea76b7b64df38a685587f

@PixievoltNo1
Copy link
Author

Congrats on leaving beta, but 3.0.1 still has this bug: https://svelte.dev/repl?version=3.0.1&gist=9b88ce63138ea76b7b64df38a685587f

@Conduitry Conduitry added the bug label May 4, 2019
@mrkishi mrkishi mentioned this issue May 15, 2019
Closed
@Conduitry Conduitry mentioned this issue May 28, 2019
Closed
@Conduitry Conduitry mentioned this issue Jun 17, 2019
Closed
@arxpoetica
Copy link
Member

Adding another example for comparison: https://svelte.dev/repl/ff9fac1693924826ba3600c287ff3516?version=3.5.3

@PixievoltNo1 PixievoltNo1 mentioned this issue Jun 23, 2019
Closed
@Bakino
Copy link

Bakino commented Jun 23, 2019

Is there any workaround for this ?

Not beeing able the access to components is quite an issue in complex layout

@btk5h btk5h mentioned this issue Jun 23, 2019
Merged
Conduitry pushed a commit that referenced this issue Jun 23, 2019
@btk5h @Conduitry
Fix dynamic bind:this on components (#2333)
01676aa
@PixievoltNo1
Copy link
Author

Huzzah for @btk5h! 😄

@Conduitry
Copy link
Member

Huzzah indeed! Fixed in 3.5.4

@arxpoetica
Copy link
Member

Awesome: https://svelte.dev/repl/ff9fac1693924826ba3600c287ff3516?version=3.5.4

This was referenced Jun 24, 2019
Closed
Closed
@CanRau CanRau mentioned this issue Nov 3, 2019
Closed
This was referenced Jan 16, 2022
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix binding this to a computed property btk5h/svelte
4 participants
@arxpoetica @PixievoltNo1 @Conduitry @Bakino