fix(blocks): database title affect the global contenteditable

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.09k stars 353 forks source link

Closed Flrande closed 1 week ago

Flrande commented 1 week ago

When the title is set to readonly, inlineEditor will set eventSource to contenteditable=false, causing editing functionality issues.

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 25, 2024 10:33am

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.

graphite-app[bot] commented 1 week ago

graphite-app[bot] commented 1 week ago

Jun 25, 6:29 AM EDT: The merge label 'merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Jun 25, 6:29 AM EDT: The merge label 'merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Jun 25, 6:30 AM EDT: zzj3720 added this pull request to the Graphite merge queue.
Jun 25, 6:39 AM EDT: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'Playground E2E test (1)').