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

Maybe: Model/DomView builder react() declaration #142

Open
issa-tseng opened this issue Mar 17, 2019 · 0 comments
Open

Maybe: Model/DomView builder react() declaration #142

issa-tseng opened this issue Mar 17, 2019 · 0 comments

Comments

@issa-tseng
Copy link
Owner

issa-tseng commented Mar 17, 2019

Most of the time when _initialize or _wireEvents are used, the reason is to write a series of (…).get(…).react(…) statements (or, more correctly, this.reactTo(this.get(…), …)).

I dislike needing to actually override the method body and I dislike both these method names (one more than the other can you guess which?) but sometimes you just have to and try as I might I can't come up with better names.

If we add a react() type declaration to subsume these occasions, we move a lot of that need away and so I don't have to see those names as often.

@issa-tseng issa-tseng changed the title Model/DomView react() declaration Maybe: Model/DomView react() declaration Mar 17, 2019
@issa-tseng issa-tseng changed the title Maybe: Model/DomView react() declaration Maybe: Model/DomView builder react() declaration Jul 23, 2019
@issa-tseng issa-tseng added this to the Janus v0.6 milestone Aug 20, 2019
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