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

Changing of behaviour for attributeChanged #483

Closed
treshugart opened this issue Feb 16, 2016 · 0 comments
Closed

Changing of behaviour for attributeChanged #483

treshugart opened this issue Feb 16, 2016 · 0 comments

Comments

@treshugart
Copy link
Member

treshugart commented Feb 16, 2016

The current behaviour is to not fire for existing attributes. See: https://github.com/skatejs/skatejs/blob/master/README.md#attribute-1. However, they've recently decided on some changes.

However, there have been several issues raised that are currently open that we should keep track of and action once they've been closed:

If this list grows, we should split these out but until it's final this may serve as a better aggregation point.

This is also dependent upon #511.

@treshugart treshugart changed the title Update behaviour for attribute() callback. Changing of behaviour for attributeChangedCallback(). Feb 16, 2016
@treshugart treshugart changed the title Changing of behaviour for attributeChangedCallback(). Changing of behaviour for attributeChanged Mar 16, 2016
@treshugart treshugart added this to the 1.0.0 milestone Mar 16, 2016
treshugart added a commit that referenced this issue May 17, 2016
#483 - Ensure attribute lifecycle is invoked for attributes existing on the element at the time of upgrade
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

1 participant