Skip to content

Conversation

@HeikoKlare
Copy link
Contributor

When changing position and/or size of the editor of a FindReplaceOverlay while it is hidden, e.g., because another editor in the same editor stack is active, the position and size of the overlay is not updated accordingly. Thus, when making the editor of that overlay active again, its position relative to the editor and size is still the same as before another editor was set active, i.e., it is usually wrong.

With this change, the size and position of an overlay is always updated upon resize operations of the target editor, even if the editor is not visible.

Fixes #2478

findreplace_update_position_when_hidden_fixed

@github-actions
Copy link
Contributor

github-actions bot commented Nov 1, 2024

Test Results

 1 821 files  ±0   1 821 suites  ±0   1h 44m 11s ⏱️ - 9m 32s
 7 724 tests ±0   7 496 ✅ +1  228 💤 ±0  0 ❌  - 1 
24 333 runs  ±0  23 586 ✅ +1  747 💤 ±0  0 ❌  - 1 

Results for commit 818c920. ± Comparison against base commit eacd0b4.

♻️ This comment has been updated with latest results.

@HeikoKlare HeikoKlare marked this pull request as ready for review November 2, 2024 08:22
…e-platform#2478

When changing position and/or size of the editor of a FindReplaceOverlay
while it is hidden, e.g., because another editor in the same editor
stack is active, the position and size of the overlay is not updated
accordingly. Thus, when making the editor of that overlay active again,
its position relative to the editor and size is still the same as before
another editor was set active, i.e., it is usually wrong.

With this change, the size and position of an overlay is always updated
upon resize operations of the target editor, even if the editor is not
visible.

Fixes eclipse-platform#2478
@HeikoKlare HeikoKlare merged commit c1c681a into eclipse-platform:master Nov 3, 2024
17 checks passed
@HeikoKlare HeikoKlare deleted the issue-2478 branch November 3, 2024 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Find/replace overlay: no placement update while not visible

1 participant