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

[templates] Do not specify template binding syntax in spec, make configurable #697

Closed
vidhill opened this issue Nov 5, 2017 · 3 comments

Comments

@vidhill
Copy link

vidhill commented Nov 5, 2017

Library authors and users will all have their own preferences and opinions on binding syntax,
Rather that specifying {{foo}} or ${bar} within the proposal,
the Constructor should allow the user to configure the syntax..

In the same way that angularJS supports such..
This proposal could easily getting diverted by nitpicking on such issues..

@vidhill vidhill changed the title Do not specify template binding syntax in spec, make configurable [templates] Do not specify template binding syntax in spec, make configurable Nov 5, 2017
@rniwa
Copy link
Collaborator

rniwa commented Nov 6, 2017

I don't think we should do this. One of the major motivation for standardizing template API is to make the parsing easy & predictable. With a configurable syntax, we won't be able to apply various CSP constraints or protect websites against XSS.

@domenic
Copy link
Collaborator

domenic commented Nov 14, 2017

At TPAC, all participants in the discussion that spoke up agreed that we should decide on a standardized syntax. Note that we did not insist on double-curlies, but did insist on nothing that would require escaping when using in conjunction with CSS or JS. (So, single-curlies and dollar-curlies are no good.)

Given @rniwa's good reasons above, let's close this, as we can be pretty confident the proposal will not end up with configurable syntax.

@domenic domenic closed this as completed Nov 14, 2017
@vidhill
Copy link
Author

vidhill commented Nov 14, 2017

Fair enough

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

No branches or pull requests

3 participants