Spectrum: Scoped packages are parsed as mentions

[Ponjimon]

They link to https://spectrum.chat/users/storybook

[mxstbr]

Yeah, same thing applies to links :confused: Gotta make sure to only parse mentions in actual text...

[alejandronanez]

This bug doesn't seem too complicated for someone new to the codebase like me. If that sounds good to you, I'd like to work on it @mxstbr.

Thanks :)

[mxstbr]

I'm not so sure about that, but you're welcome to try!

I think the relevant code would be in mention-decorator.js and you'd have to filter the incoming drafjs state to only match if the block type isn't code-block or the entity type isn't link or something, not sure honestly.

[alejandronanez]

I'll give it a try and will post an update here with my findings in the next couple of days. ✌️

[alejandronanez]

Is it possible that you assign this issue to me @mxstbr? I think you have a label for that.

[mxstbr]

@alejandronanez ping me if you need help!

[alejandronanez]

Will do!

[alejandronanez]

FYI: My findings so far:

If you write this in Spectrum:

@scoped/package
@scoped/package
@scoped/package
@scoped/package

You'll end up with this:

I'll keep looking into this over the weekend and I'll come here if I need some guidance. 👍

[alejandronanez]

@mxstbr I created the fix for this issue in #2961 and I think we should create a separate issue for the nested mentions.