fix(md): update floatWith handlers in link menu to match mentions menu

[sedson]

Summary

Screenshots, GIFs, and Videos

[github-actions]

Preview: https://seamus-fix-link-menu-float-pos-7ssi42.preview.macro.com/app (dbbcc5c)

[claude]

Code review

Found 1 issue:

Bug: Both positioning directives can be active simultaneously

File: js/app/packages/core/component/LexicalMarkdown/component/menu/FloatingLinkMenu.tsx (line 344)

When clicking on an existing link, the handleClick function sets both linkRef and selection on the ILinkInfo object. This means both floatWithElementProps() and floatWithSelectionProps() will return valid options objects, causing both directives to execute simultaneously.

This leads to:

1. Position style conflicts: floatWithElement sets position: absolute while floatWithSelection sets position: fixed
2. Multiple autoUpdate listeners that will compete
3. Potential flickering or incorrect positioning

The MentionsMenu pattern correctly handles this by using mutual exclusion with !props.anchor in the selection props condition.

Suggested fix: Change line 344 from:

    linkInfo()?.selection

to:

    linkInfo()?.selection && !linkInfo()?.linkRef

This ensures only one directive is active at a time, matching the MentionsMenu pattern.

[gbirman]

remove console logs