Skip to content

[lexical-playground] fix: close link popup when user clicks out of it - #7670

Merged
etrepum merged 3 commits into
facebook:mainfrom
achaljhawar:close-link-popup
Jul 3, 2025
Merged

etrepum merged 3 commits into
facebook:mainfrom
achaljhawar:close-link-popup

Conversation

@achaljhawar

Copy link
Copy Markdown
Contributor

Description

Added a event listener to the FloatingLinkEditor component to ensure the link popup closes when it loses focus.

Closes #7665

Test plan

Before

Lexical.Playground.6.mp4

After

Lexical.Playground.7.mp4

@vercel

vercel Bot commented Jul 3, 2025 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lexical ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 3, 2025 6:27pm
lexical-playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 3, 2025 6:27pm

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 3, 2025
etrepum
etrepum previously approved these changes Jul 3, 2025
Comment thread packages/lexical-playground/src/plugins/FloatingLinkEditorPlugin/index.tsx Outdated
Co-authored-by: Bob Ippolito <bob@redivi.com>

@etrepum etrepum left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fails the formatting check, npm run prettier:fix should address that

@etrepum
etrepum added this pull request to the merge queue Jul 3, 2025
Merged via the queue into facebook:main with commit 5834946 Jul 3, 2025
@etrepum etrepum mentioned this pull request Jul 3, 2025
@achaljhawar
achaljhawar deleted the close-link-popup branch July 28, 2025 12:39
fantactuka pushed a commit that referenced this pull request Aug 11, 2025

This branch was successfully deployed

2 active deployments
Preview – lexical-playground — 369c2e30 Deployed Jul 3, 2025 by vercel[bot]
Preview – lexical — 369c2e30 Deployed Jul 3, 2025 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The LinkPlugin popup remains open when the link is at the end of the content.

3 participants