Empty block is unnecessarily created

[moroshko]

Describe the bug
Empty block is unnecessarily created. I'm wondering why is this and whether there is a way to avoid this?

To Reproduce

1. Visit https://www.blocknotejs.org/examples/basic/minimal
2. Focus the editor
3. Press "a"
   You'll see that a new empty block is added under the "a" (you can press the Down arrow to go there)

Misc

• Node version: 20
• Package manager: bun
• Browser: Chrome
• I'm a sponsor and would appreciate if you could look into this sooner than later 💖

[YousefED]

This is because we're adding a so-called trailing node. This makes it easier for users to use their mouse and click below the document and start adding content. It's similar to what notion does.

If you want to disable this, you could try creating a PR. We could add an editor option that would disable the extension here:

BlockNote/packages/core/src/editor/BlockNoteExtensions.ts

Line 134 in 8c22f79

TrailingNode,

[jaredly]

ooh yes I want this disabled as well

[jaredly]

@YousefED I can imagine three different levels of intervention:

(1) add an option noTrailingNode: boolean, simple & only solves the present issue
(2) add an option customizeExtensions: (extensions: AnyExtension[]) => AnyExtension[] -- allows the user to remove or modify any of the list of default extensions, allowing users to customize things further
(3) add an option customizeTipTapOptions: (options: TipTapEditorOptions) => TipTapEditorOptions, allowing the greatest amount of control, not just over the extensions that are loaded, but a bunch of other things as well. This would superceded the _tipTapOptions option that's currently available.

Do you have a preference?

[jaredly]

Alternatively, @moroshko here's the hack that I've figured out, while waiting for PRs to happen and such:

const ext = editor._tiptapEditor.extensionManager.extensions.find((e) => e.name === 'trailingNode');
if (ext) {
    // Disable the trailingNode extension
    ext.config.addProseMirrorPlugins = () => [];
}

[YousefED]

``
Thanks @jaredly! I feel like this should be a first-class option without needing to know about TipTap, so I'd be in favor of:

(1) add an option noTrailingNode: boolean, simple & only solves the present issue

Small caveat: I'd use trailingBlock: boolean which defaults to true ("Block" is more BlockNote like, and I think it's a bit nicer to not have a negation ("no") ).

[YousefED]

/bounty 50

[algora-pbc]

💎 $50 bounty • BlockNote

Steps to solve:

1. Start working: Comment /attempt #631 with your implementation plan
2. Submit work: Create a pull request including /claim #631 in the PR body to claim the bounty
3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to TypeCellOS/BlockNote!

Add a bounty • Share on socials

Attempt	Started (GMT+0)	Solution
🔴 @35C4n0r	Mar 27, 2024, 3:04:17 PM	WIP
🔴 @ezhil56x	Mar 27, 2024, 3:33:14 PM	WIP
🟢 @Kashifwahaj	Apr 2, 2024, 5:15:51 PM	#679

[35C4n0r]

/attempt 631

Algora profile	Completed bounties	Tech	Active attempts	Options
@35C4n0r	13 bounties from 4 projects	Python, JavaScript, HTML & more		Cancel attempt

[ezhil56x]

/attempt #631

Algora profile	Completed bounties	Tech	Active attempts	Options
@ezhil56x	16 bounties from 10 projects	JavaScript, TypeScript, Rust & more		Cancel attempt