Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Fix aside affecting parent popover height (#11859)" #11942

Merged
merged 1 commit into from
May 16, 2024

Conversation

maxdeviant
Copy link
Member

This reverts commit d3dfa91.

This change can cause weird behavior where the completion menu ends up positioned away from the cursor location:

Screenshot 2024-05-16 at 6 43 17 PM

With the change reverted:

Screenshot 2024-05-16 at 6 43 35 PM

Release Notes:

  • Fixed an issue where the completion menu would sometimes appear detached from the cursor location (preview only).

@maxdeviant maxdeviant self-assigned this May 16, 2024
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label May 16, 2024
@maxdeviant
Copy link
Member Author

/cherry-pick v0.136.x

@maxdeviant maxdeviant merged commit 0dd5fe3 into main May 16, 2024
8 checks passed
@maxdeviant maxdeviant deleted the revert-d3dfa91254f8cbc7f4779463b30a3df1677c395d branch May 16, 2024 22:53
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request May 16, 2024
This reverts commit d3dfa91.

This change can cause weird behavior where the completion menu ends up
positioned away from the cursor location:

<img width="1062" alt="Screenshot 2024-05-16 at 6 43 17 PM"
src="https://github.com/zed-industries/zed/assets/1486634/0462a874-4fe3-4ca9-88ce-8d5d0b4009fe">

With the change reverted:

<img width="1026" alt="Screenshot 2024-05-16 at 6 43 35 PM"
src="https://github.com/zed-industries/zed/assets/1486634/9fc7b9a1-0cfb-4a84-8f6b-b481a785ceca">

Release Notes:

- Fixed an issue where the completion menu would sometimes appear
detached from the cursor location (preview only).
maxdeviant added a commit that referenced this pull request May 16, 2024
…ick #11942) (#11944)

Cherry-picked Revert "Fix aside affecting parent popover height
(#11859)" (#11942)

This reverts commit d3dfa91.

This change can cause weird behavior where the completion menu ends up
positioned away from the cursor location:

<img width="1062" alt="Screenshot 2024-05-16 at 6 43 17 PM"

src="https://github.com/zed-industries/zed/assets/1486634/0462a874-4fe3-4ca9-88ce-8d5d0b4009fe">

With the change reverted:

<img width="1026" alt="Screenshot 2024-05-16 at 6 43 35 PM"

src="https://github.com/zed-industries/zed/assets/1486634/9fc7b9a1-0cfb-4a84-8f6b-b481a785ceca">

Release Notes:

- Fixed an issue where the completion menu would sometimes appear
detached from the cursor location (preview only).

Co-authored-by: Marshall Bowers <elliott.codes@gmail.com>
osiewicz pushed a commit to RemcoSmitsDev/zed that referenced this pull request May 18, 2024
…)" (zed-industries#11942)

This reverts commit d3dfa91.

This change can cause weird behavior where the completion menu ends up
positioned away from the cursor location:

<img width="1062" alt="Screenshot 2024-05-16 at 6 43 17 PM"
src="https://github.com/zed-industries/zed/assets/1486634/0462a874-4fe3-4ca9-88ce-8d5d0b4009fe">

With the change reverted:

<img width="1026" alt="Screenshot 2024-05-16 at 6 43 35 PM"
src="https://github.com/zed-industries/zed/assets/1486634/9fc7b9a1-0cfb-4a84-8f6b-b481a785ceca">

Release Notes:

- Fixed an issue where the completion menu would sometimes appear
detached from the cursor location (preview only).
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant