Going through the link with the popup opens the cursor

[d-e-v-esh]

Bug Report 🐛

When we move our cursor forward in a line with the right arrow where one of the words is a hyperlink, the link popup opens up and the cursor jumps to the link URL input field.

Expected Behavior

The pop-up should not open up until we click on the hyperlink text. The cursor should move past the letters in the text one by one without the pop-up opening.

Current Behavior

It jumps to the URL input field as soon as the cursor touches the hyperlink text.

Steps to Reproduce

1. Open up Storybook => Markdown Editor
2. Go to the first line which has a hyperlink text with the right arrow.
3. As soon as the cursor touches the hyperlink text, the link pop-up would open up.

Context (Environment)

Desktop

• OS: Windows 10
• Browser: Chrome
• Version 88.0.4324.182 (Official Build) (64-bit)

Detailed Description

Possible Implementation

[K-Kumar-01]

@irmerk @DianaLease @Michael-Grover
I have added a PR for the issue. This might be able to resolve the issue.
Requesting a review from the maintainers.

[iamsaumya]

I wonder if this is a bug or a feature. As we can see in the source code there is an explicit if condition to show the model when the cursor is inside the link.

web-components/packages/ui-markdown-editor/src/index.js

Lines 146 to 148 in 90973d4

	if (selection && isSelectionLinkBody(editor)) {
	setShowLinkModal(true);
	}

If we let the cursor move forward in the link body, I believe they can also change the text and it will lead to a change in the name of the link. As I can see, the maintainers wanted to only have a single source of changing the link i.e LinkModal.

[mttrbrts]

web-components/packages/ui-markdown-editor/src/index.js

@jeromesimeon I see that you made the change that introduced

web-components/packages/ui-markdown-editor/src/index.js

Lines 146 to 148 in 90973d4

	if (selection && isSelectionLinkBody(editor)) {
	setShowLinkModal(true);
	}

Do you recall why you wanted this behaviour, please?

[jolanglinais]

This seems related to #114. I think this needs more investigation before moving on to a PR (#297)