Skip to content

Fix resolved lens causing flickers#56047

Merged
SomeoneToIgnore merged 3 commits into
mainfrom
kb/lens-placeholders
May 8, 2026
Merged

Fix resolved lens causing flickers#56047
SomeoneToIgnore merged 3 commits into
mainfrom
kb/lens-placeholders

Conversation

@SomeoneToIgnore

Copy link
Copy Markdown
Contributor

Based on #54100 (comment)

  • Adjusts the code lens display closer to what VSCode does: have blank placeholders for the code lens need resolving.
    Zed will remove them if resolve returns nothing, so some small amount of jitter is still there.

  • Also reworks LspStore layer to provide a simple resolve method, without any ranges involved, grouping that logic in the editor itself.
    This allows to process each resolve request separately, updating editor blocks as soon as possible.

Before:

before.mov

After:

after.mov

Release Notes:

  • Fixed resolved lens causing flickers

@SomeoneToIgnore SomeoneToIgnore requested a review from Veykril May 7, 2026 12:40
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 7, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label May 7, 2026
@SomeoneToIgnore SomeoneToIgnore added this pull request to the merge queue May 8, 2026
Merged via the queue into main with commit b270b1d May 8, 2026
41 checks passed
@SomeoneToIgnore SomeoneToIgnore deleted the kb/lens-placeholders branch May 8, 2026 10:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants