Fix combobox targetRange for mention input

[JannesMeyer]

Description

Fixes #1843
Fixes #2533

After investigating #1843 I discovered that Slate inserts an empty text node before the mention input to fulfil this normalization constraint:

4. Inline nodes cannot be the first or last child of a parent block, nor can it be next to another inline node in the children array. If this is the case, an empty text node will be added to correct this to be in compliance with the constraint.

Source: https://docs.slatejs.org/concepts/11-normalizing

Is there any better way to wait for normalization without setTimeout()?

[changeset-bot]

🦋 Changeset detected

Latest commit: 3acfad9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@udecode/plate-mention	Patch
@udecode/plate	Patch

Not sure what this means? Click here to learn what changesets are.

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

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
plate	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 19, 2023 7:37pm

[reviewpad]

Thank you @JannesMeyer for this first contribution!

[12joan]

Thanks for the PR!

Is there any better way to wait for normalization without setTimeout()?

In the future, perhaps we could add a helper function to Plate core (or Slate core) that queues up a callback until the next time onChange gets called to signify the editor state is stable. For now, however, I think using setTimeout is the simplest approach.

[JannesMeyer]

Thanks for the reply, @12joan :)

I just noticed the failing test which expects the combobox state to be updated immediately. I don't know how to resolve that yet, but I will try to think of a solution.