Skip to content

Fixes for static#2

Open
tristanMatthias wants to merge 6 commits into
Cu3PO42:masterfrom
tristanMatthias:master
Open

Fixes for static#2
tristanMatthias wants to merge 6 commits into
Cu3PO42:masterfrom
tristanMatthias:master

Conversation

@tristanMatthias

Copy link
Copy Markdown

I noticed that you were referencing properties and listeners to the element/target. These should actually be referenced via (target.constructor as Element).properties, as they are static variables in Polymer.

class Test extends PolymerElement {
    static get properties() {...}
}

As a result, I moved them over to the target.constructor which now works in Polymer@^3.0.2

There is also a few tidyups with typings.

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

Successfully merging this pull request may close these issues.

2 participants