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.

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

Throw error when using interpolation instead of JS expression for slot names #9038

Open
revolter opened this issue Nov 7, 2018 · 6 comments · May be fixed by #9050
Open

Throw error when using interpolation instead of JS expression for slot names #9038

revolter opened this issue Nov 7, 2018 · 6 comments · May be fixed by #9050

Comments

@revolter
Copy link

revolter commented Nov 7, 2018

What problem does this feature solve?

For this demo:

https://jsfiddle.net/2os9jken/

opening the console and running it throws the Interpolation inside attributes has been removed. Use v-bind or the colon shorthand instead. error for the span, but it should also throw it for the slot, as it has the exact same issue for its name attribute.

What does the proposed API look like?

N/A

@posva
Copy link
Member

posva commented Nov 7, 2018

Thanks for the proposal, the warning was meant for people coming from v1 where this was allowed. Personally, I don't think it is worth adding any more warnings regarding this as it will rarely help any user

@revolter
Copy link
Author

revolter commented Nov 7, 2018

Well, old references still exist, and it's not that obvious for new users. It surely confused me in #9037.

@prabhatmishra33
Copy link

prabhatmishra33 commented Oct 5, 2021

Hello ! Is this issue fixed or yet open to be worked on ?

@prathameshpcodes
Copy link

Is this issue fixed or yet to be fixed?

@IhsenBen
Copy link

can I work on this?

@DaZuiZui
Copy link

It is not supported to use interpolation {{}} to assign values to attributes, so use the v-bind command (or abbreviated ":") to specify attributes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants