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.37k stars 398 forks source link

fix: refactor frame panel to use EditorHost instead of AffineEditorContainer #8349

Closed donteatfriedrice closed 3 weeks ago

donteatfriedrice commented 3 weeks ago

BS-1425 BS-1414

  1. To use editorHost instead of AffineEditorContainer
  2. Remove useless slots.
  3. Add more e2e tests.
vercel[bot] commented 3 weeks 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 Sep 13, 2024 10:23am
1 Skipped Deployment | Name | Status | Preview | Comments | Updated (UTC) | | :--- | :----- | :------ | :------- | :------ | | **blocksuite-docs** | ⬜️ Ignored ([Inspect](https://vercel.com/toeverything/blocksuite-docs/3QnkXHAuCgM4QvS3gdnftVHHzPkM)) | [Visit Preview](https://blocksuite-docs-git-09-13-refactorframepane-c2c630-toeverything.vercel.app) | | Sep 13, 2024 10:23am |
changeset-bot[bot] commented 3 weeks ago

⚠️ No Changeset found

Latest commit: 1ccb5ee7ead798f9a9c3ae632f14711d00a5b206

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

donteatfriedrice commented 3 weeks ago

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

Join @donteatfriedrice and the rest of your teammates on Graphite Graphite

graphite-app[bot] commented 3 weeks ago

Your org has enabled the Graphite merge queue for merging into master

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.

nx-cloud[bot] commented 3 weeks ago

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 1ccb5ee7ead798f9a9c3ae632f14711d00a5b206. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target - [`nx vite:build @blocksuite/playground`](https://cloud.nx.app/runs/mw5iXodGD5?utm_source=pull-request&utm_medium=comment)

Sent with 💌 from NxCloud.

donteatfriedrice commented 3 weeks ago

Merge activity

github-actions[bot] commented 3 weeks ago

size-limit report 📦

Path Size
@blocksuite/affine-block-list 5.94 KB (0%)
@blocksuite/affine-block-paragraph 4.5 KB (0%)
@blocksuite/affine-block-surface 82.11 KB (0%)
@blocksuite/affine-components 366.81 KB (0%)
@blocksuite/data-view 209.99 KB (0%)
@blocksuite/affine-model 30.03 KB (0%)
@blocksuite/affine-shared 24.28 KB (0%)
@blocksuite/blocks 3.49 MB (-0.01% 🔽)
@blocksuite/block-std 105.8 KB (0%)
@blocksuite/global 28.4 KB (0%)
@blocksuite/inline 23.76 KB (0%)
@blocksuite/store 64 KB (0%)
@blocksuite/sync 95.99 KB (0%)
@blocksuite/presets 200.68 KB (-0.57% 🔽)