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

Enable _removeListener after Polymer/polymer#1740 #36

Merged
merged 1 commit into from
Sep 12, 2015

Conversation

adalinesimonian
Copy link
Contributor

Polymer/polymer#1740 adds unlisten functionality, thus fixing blocking issue Polymer/polymer#1639. Now iron-selector can properly remove the event listener for the old activate event when it is changed instead of being forced to ignore unwanted handler calls.

Polymer/polymer#1740 adds `unlisten` functionality, thus fixing blocking issue Polymer/polymer#1639. Now `iron-selector` can properly remove the event listener for the old activate event when it is changed instead of being forced to ignore unwanted calls handler calls.
@dfreedm
Copy link
Contributor

dfreedm commented Jun 12, 2015

That API hasn't landed in a release yet. Wait for Polymer 1.0.4.

@adalinesimonian
Copy link
Contributor Author

Ah! Didn't even notice. Should I just leave this up until then?

@blasten
Copy link
Contributor

blasten commented Sep 8, 2015

LGTM! cc @notwaldorf @frankiefu

@addyosmani
Copy link
Contributor

LGTM2.

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.

5 participants