Skip to content
This repository has been archived by the owner on Sep 15, 2021. It is now read-only.

Shouldn't @ObserveProperty and @ComputedProperty live here? #13

Open
fabiosimeoni opened this issue Apr 14, 2015 · 0 comments
Open

Shouldn't @ObserveProperty and @ComputedProperty live here? #13

fabiosimeoni opened this issue Apr 14, 2015 · 0 comments

Comments

@fabiosimeoni
Copy link

It'd appear that @ObserveProperty and @ComputedProperty are available only in view models, as Polymer facilities. Can you help explaining the rationale for that placement?

Both seem general facilities over lower-level PathObserver and listen() mechanisms, and as such seem well suited to live in this package.

Ok, maybe @ComputedProperty has a bit more of a model-view application slant, but not that strongly. For example, I find it very practical to avoid train-wrecks in client code, be it in templates, view models, or generic models.

Overall, the annotations seem to give fairly general OO help I regret giving up anytime I'm "below" view models. Couldn't you offer them as companions to, say, onPropertyChange()?

cheers

(and thanks for the good work!)

@fabiosimeoni fabiosimeoni changed the title Shouldn't @ObserveProperty and @ComputedProperty live here? Shouldn't @ObserveProperty and @ComputedProperty live here? Apr 14, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

1 participant