fix(edgeless): edgeless portal does not render new added blocks

toeverything / blocksuite

🧩 Content editing tech stack for the web - BlockSuite is a toolkit for building editors and collaborative applications.

https://blocksuite.io

Mozilla Public License 2.0

4.08k stars 352 forks source link

Closed doouding closed 1 week ago

doouding commented 1 week ago

vercel[bot] commented 1 week ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
blocksuite	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 27, 2024 9:07am

1 Ignored Deployment

graphite-app[bot] commented 1 week ago

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

doouding commented 1 week ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @doouding and the rest of your teammates on Graphite

Saul-Mirone commented 1 week ago

Can we add a test case for this?

L-Sun commented 1 week ago

I feel that the code logic before and after the change is almost the same. What caused this bug? 🤔

doouding commented 1 week ago

I feel that the code logic before and after the change is almost the same. What caused this bug? 🤔

I think lit cannot handle the repeat rendering in nested loop correctly. So I flat the array first.

doouding commented 1 week ago

Can we add a test case for this?

Done.

graphite-app[bot] commented 1 week ago

Jun 27, 5:05 AM EDT: doouding added this pull request to the Graphite merge queue.
Jun 27, 5:16 AM EDT: doouding merged this pull request with the Graphite merge queue.