Bug: Mentions triggered somehow for list types

[safa-kajabi]

Lexical version: "@lexical/react": "^0.12.0",

Steps To Reproduce

-- Make sure that you are using the mentions plugin --

1. Create a list
2. Click the first word beside the dot or number type in the list and you will see the mentions

Screen.Recording.2023-09-07.at.13.36.08.mov

Link to code example:

https://playground.lexical.dev/

The current behavior

It shows the mentions when we click the word in the list

The expected behavior

It should not trigger the mentions if it is not a mention.

[acywatson]

I think we should just disable the capital-word trigger for the playground mentions plugin. It's annoying and trivial to implement if someone wants to do it in their own editor.

[safa-kajabi]

I think we should just disable the capital-word trigger for the playground mentions plugin. It's annoying and trivial to implement if someone wants to do it in their own editor.

Yeah this is the problem. What should ı do to disable it in my own editor ? @acywatson

[safa-kajabi]

I think we can disable it with returning the "match" instead of using the checkForCapitalizedNameMentions function. Right?

export const getPossibleQueryMatch = (text: string): MenuTextMatch | null => {
  const match = checkForAtSignMentions(text, 1);
   return match;
  return match === null ? checkForCapitalizedNameMentions(text, 3) : match;
};

[acywatson]

I think we can disable it with returning the "match" instead of using the checkForCapitalizedNameMentions function. Right?

export const getPossibleQueryMatch = (text: string): MenuTextMatch | null => {
  const match = checkForAtSignMentions(text, 1);
   return match;
  return match === null ? checkForCapitalizedNameMentions(text, 3) : match;
};

yep, that works!

lexical/packages/lexical-playground/src/plugins/MentionsPlugin/index.tsx

Line 587 in 75c9ae5

return match === null ? checkForCapitalizedNameMentions(text, 3) : match;

[safa-kajabi]

Nice! Thanks!

[acywatson]

#4991

[safa-kajabi]

#4991

Thanks a lot!!