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

property() decorator should work with Symbol-named properties #288

Closed
justinfagnani opened this issue Nov 4, 2018 · 0 comments
Closed

property() decorator should work with Symbol-named properties #288

justinfagnani opened this issue Nov 4, 2018 · 0 comments
Assignees
@sorvell
Labels
Severity: High Type: Enhancement
Milestone
2.0.0

Comments

@justinfagnani
Copy link
Contributor

Like so:

const protectedField = Symbol();
class MyElement extends LitElement {
  @property()
  [protectedField] = 0;
}
@sorvell sorvell self-assigned this Nov 5, 2018
@sorvell sorvell added this to the 0.7.0 milestone Nov 5, 2018
sorvell pushed a commit that referenced this issue Nov 5, 2018
Update decorator types
9ab8a4f 
Fixes #288 and #291
@sorvell sorvell mentioned this issue Nov 5, 2018
Merged
@dorivaught dorivaught modified the milestones: 0.7.0, 1.0.0 Nov 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sorvell sorvell

Labels
Severity: High Type: Enhancement
Projects
None yet
Milestone
2.0.0
Development

No branches or pull requests
3 participants
@sorvell @justinfagnani @dorivaught