Skip to content
This repository has been archived by the owner on Mar 13, 2018. It is now read-only.

Add array contents observer #83

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

brianchin
Copy link

This new type of observer constructs observers for each element of the array, even as the elements of the array changes. It reports changes to the array itself similar to ArrayObserver, and reports changes to the elements as sent by the user-created element observer type.

@jmesserly
Copy link
Contributor

My usual thought here is if it's worth adding to observe.js, vs another repository/file? Certainly not opposed to having it here, if it's going to be widely used. There was a push at one point to drive observe.js to a small size, because folks are generally very sensitive about code size. Another option is to start building up another .js file with utilities, and folks can include that in their page/build if desired. (A similar question arose around a "DeepObjectObserver" recently.)

@addyosmani
Copy link
Contributor

My two cents are that this type of functionality is best served via additional repositories which we link to or in a utils file/directory. That way we avoid the size of the shim growing with what some may consider optionally useful helpers.

@tilgovi
Copy link

tilgovi commented Oct 24, 2015

👍 @addyosmani

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants