8000 Props: Should have change detection, observed attributes · Issue #57 · modulojs/modulo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Props: Should have change detection, observed attributes #57

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.

Sign up for GitHub

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

Jump to bottom
Open
michaelpb opened this issue Aug 2, 2023 · 0 comments
Open

Props: Should have change detection, observed attributes #57

michaelpb opened this issue Aug 2, 2023 · 0 comments

Comments

@michaelpb
Copy link
Contributor
michaelpb commented Aug 2, 2023 •
edited
Loading

Important core feature: 1) Props should be observed attributes that trigger rerender, and 2) Props should know when they get changed in general, and be smarter during reconciliation during subcomponent.rerender()

The first has always 5714 been a 0.1.0 prereq, the second is more a requirement that will probably be fulfilled by the first, but should be clearly documented, in terms of techniques for detecting when props change. This is from experience noticing that it's quite clunky as-is (needing to stash away "receivedProps" and compare against current props)

Potentially related: Create a new CPart (or add feature to props) to provide utilities for setting own attributes as well, such as for stashing away important stuff for SSR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@michaelpb
0