Bug: LinkNode.importDOM doesn't support image content

[roeycohen]

loading the following HTML:
<a href="https://facebook.com"><img src="https://server/img.jpg"/></a>
drops the A element.

this happens because of this:

lexical/packages/lexical-link/src/index.ts

Line 286 in 5595519

if (content !== null && content !== '') {

Lexical version: 0.10

if I may suggest the fix could look like this:
if ((content !== null && content !== '') || domNode.querySelector('img') !== null)

[acywatson]

Thanks for reporting this - fix looks ok, but actually I think we should probably handle other node content as well. Maybe it would be more like domNode.children.length > 0 as an additional condition.

[roeycohen]

it's a possibility... although other children might be none clickable nodes such as <iframe/> or

[utkarsh-shrivastav77]

Hi is this issue still open

[roeycohen]

seems like @utkarsh-shrivastav77

hi @acywatson, would you like me to create PR for the fix?