-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
docs: Added conceptual view of the Lexical Core Design #5677
docs: Added conceptual view of the Lexical Core Design #5677
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
I think this works, but will give others a chance to review.
It doesn't really cover the asynchronous nature of state updates (i.e., batching), but I'm not sure that's strictly necessary. Another gap would be update listeners, which are also triggered at the end of the update cycle.
There are a few charts in here that may be helpful, though I think your approach works better for written documentation.
Thanks for the feedback! I’ll add listeners for sure! |
I just realized I forgot to add the link: https://docs.google.com/presentation/d/1IanPm9qzLP4Oiqht83weMCZalCaCJpH4QvFqeD29-uQ/edit#slide=id.p I meant to say "there are a few charts in here (above link) that may be helpful". |
416faa1
to
9169690
Compare
@acywatson @ivailop7 PTAL and merge, updated diagram a bit. I'll use info from https://docs.google.com/presentation/d/1IanPm9qzLP4Oiqht83weMCZalCaCJpH4QvFqeD29-uQ/edit#slide=id.p within other pages as within conceptual view we're treating |
This allows the viewer to have a simple visual representation of what Lexical Core actually does and how User Application shall interact with it.
Any suggestions are welcomed!