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
feat: react adapters #6490
feat: react adapters #6490
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
{ | ||
"name": "@blocksuite/react", | ||
"version": "0.12.0", | ||
"description": "Lit renderer for blocksuite store", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Lit renderer?" 👀
I'm not sure if this package should belong to framework, since it may be used to wrap preset components like editors and fragments. /cc @Saul-Mirone
346fa18
to
1a32351
Compare
This stack of pull requests is managed by Graphite. Learn more about stacking. |
1a32351
to
9772dd0
Compare
9772dd0
to
790ffd6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The view store refactor will have huge influence on framework integration, we'll need to wait for the feature landed.
Closing after discussion. A new react adapter will be introduced. |
Add custom react wrapper, custom react renderer for lit and examples.
This PR migrated codes that are currently in AFFiNE codebase. We free like it is the time to move into blocksuite.
createReactComponentFromLit
that is similar tocreateComponent
from@lit/react
. This was added to mitigate issues that web components in blocksuite are often instantiated via creating web components & appending to dom directly in JS, described in [react] Property values are not available in the connectedCallback when using createComponent from @lit/react lit/lit#4435Kapture.2024-03-18.at.18.57.28.mp4