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

Change scoped field slots name syntax #4001

Closed
mehran-prs opened this issue Sep 3, 2019 · 5 comments
Closed

Change scoped field slots name syntax #4001

mehran-prs opened this issue Sep 3, 2019 · 5 comments

Comments

@mehran-prs
Copy link

Due to new Dynamic Arguments feature of Vue when we want to define new field slot in table, should add two \ to field name :
e.g

<template #\[field_name\]="row">Some thing...</template>

I think field slots name syntax can be better, something like __field_name or _field_name_.

@tmorehouse
Copy link
Member

See #3862

@tmorehouse
Copy link
Member

Note that \ are not allowed in attribute names.

@mehran-prs
Copy link
Author

mehran-prs commented Sep 3, 2019

See #3862

Yes, after seeing this issue, I closed my issue.

@mehran-prs
Copy link
Author

Note that \ are not allowed in attribute names.

You got it wrong, I don't using \ as attribute name, just escaping [ & ] in attribute name and Vue allow this.

@tmorehouse
Copy link
Member

When used in document DOM templates there can be issues with \. But not when using Vue-loader

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

No branches or pull requests

2 participants