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

Fix for method parsing in computed binding #2295

Merged
merged 1 commit into from
Dec 4, 2015

Conversation

nazar-pc
Copy link
Contributor

Quantifier should be greedy, otherwise it fails to parse correctly such construction:

<span attr="[[computed('(', foo)]]">

nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Aug 17, 2015
nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Aug 20, 2015
@tjsavage
Copy link
Contributor

Could you create a corresponding issue for this? It seems clear in the PR, but an issue will help the conversation.

@nazar-pc
Copy link
Contributor Author

Issue: #2506

@samccone
Copy link
Contributor

samccone commented Oct 6, 2015

@tjsavage any word on this one?

nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Oct 8, 2015
nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Nov 1, 2015
nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Nov 6, 2015
nazar-pc added a commit to nazar-pc/CleverStyle-Framework that referenced this pull request Nov 13, 2015
@dfreedm
Copy link
Member

dfreedm commented Dec 4, 2015

LGTM

dfreedm added a commit that referenced this pull request Dec 4, 2015
Fix for method parsing in computed binding
@dfreedm dfreedm merged commit 9c012a1 into Polymer:master Dec 4, 2015
@nazar-pc nazar-pc deleted the method-parsing-fix branch December 16, 2015 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants