Inline Toolbar overrides selected content outside editor when pressing "/"

[litvin0d]

Bug Description

When the Inline Toolbar is enabled in Editor.js, pressing the "/" key after selecting any text or element outside the editor area results in replacing the selected content with a "/". This issue occurs both in custom implementations and on the official Editor.js documentation site.

Steps to Reproduce

1. Go to any page with the editor loaded (e.g., editorjs.io and click "Play With Demo").
2. Select text or any element outside the editor's area.
3. Press the "/" key on your keyboard.

Expected behavior

Pressing "/" should not affect selected content or elements outside the editor area.

Actual Behavior

The "/" keypress unexpectedly replaces selected text and elements outside the editor with "/".

Screenshots

Additional Context

This behavior appears to be triggered by Inline Toolbar, which activates the editor's command palette when "/" is pressed, regardless of the selected area. It disrupts the content and layout outside the editor area.

Environment

Editor.js version: 2.29.x

Browser: e.g., Chrome 130.0, Firefox 132.0

OS: e.g., Ubuntu 24.04, macOS 15

Plugins you use with their versions: Header 2.8.1, ImageTool 2.9.0, NestedList 1.4.2, Embed 2.8.1, Table 2.3.0, Underline 1.1.0

[obafemitayor]

Taking a look at this one

[obafemitayor]

@idebenone I noticed this has been open for over a month so I decided to quickly fix this. I hope you don't mind

@neSpecc Here is the fix for the bug
#2894

Can you help have a look when you have the chance?

[idebenone]

Hey @obafemitayor,

Thank you for your contribution, we will definitely look into it.

[obafemitayor]

Hello @neSpecc, did you find some time to review the latest changes to this PR (#2894)?

[obafemitayor]

Hello @neSpecc, have you had some time to test the changes of this #2894?

[obafemitayor]

Hello @neSpecc, happy new year 🙂🥳. just wanted to inquire if you have had some time to test the changes of this ?

[obafemitayor]

Hello @neSpecc, have had some time to test the changes of #2894 and possibly approve and merge the PR?

[neSpecc]

@obafemitayor I've checked it. The fix works fine, but the test is not fully reproduce the original problem. Would you have time to refine test?

[obafemitayor]

@obafemitayor I've checked it. The fix works fine, but the test is not fully reproduce the original problem. Would you have time to refine test?

@neSpecc Yes, I will work on this now
Can you share the error you got when you ran the test?

[neSpecc]

There is no error, but the text outside editor is not replaced by slash

[obafemitayor]

@neSpecc I have modified the test to fully reproduce the original problem
Here is a video evidence of the test without the bug fix
https://www.loom.com/share/5f74637ad2f140b6a4ddf4052a5db107
and here is a video evidence of the test with the bug fix
https://www.loom.com/share/e9e1c2df0f9947128c28fe6866650dc9