Skip to content

editor: Fix edit prediction popovers painting outside the editor's bounds#50361

Merged
cole-miller merged 2 commits intomainfrom
fix-ep-clipping
Mar 2, 2026
Merged

editor: Fix edit prediction popovers painting outside the editor's bounds#50361
cole-miller merged 2 commits intomainfrom
fix-ep-clipping

Conversation

@cole-miller
Copy link
Copy Markdown
Member

Deferred draws previously didn't use a content mask, so you could horizontally scroll an EP popover all the way out of the containing pane.

This also affects other UI elements that use the deferred draw system; I think in practice it doesn't make much difference because most of those seem to require something in the editor to be hovered, so if you scroll horizontally the element goes away.

Release Notes:

  • Fixed being able to scroll the edit prediction popover out of the containing pane.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 28, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Feb 28, 2026
@cole-miller cole-miller marked this pull request as ready for review March 2, 2026 12:44
@cole-miller cole-miller enabled auto-merge (squash) March 2, 2026 12:44
@cole-miller cole-miller merged commit c0008cf into main Mar 2, 2026
43 of 45 checks passed
@cole-miller cole-miller deleted the fix-ep-clipping branch March 2, 2026 13:38
tahayvr pushed a commit to tahayvr/zed that referenced this pull request Mar 4, 2026
…unds (zed-industries#50361)

Deferred draws previously didn't use a content mask, so you could
horizontally scroll an EP popover all the way out of the containing
pane.

This also affects other UI elements that use the deferred draw system; I
think in practice it doesn't make much difference because most of those
seem to require something in the editor to be hovered, so if you scroll
horizontally the element goes away.

Release Notes:

- Fixed being able to scroll the edit prediction popover out of the
containing pane.
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.

1 participant