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

FindReplaceLogic: incremental doesn't skip over current selection #2109

Merged

Conversation

Wittmaxi
Copy link
Contributor

Incremental search would start incremental search immediately after the
current word. This meant that the incremental search would not go back
to the word where the incremental search initially started.

fixes #2107

@Wittmaxi Wittmaxi changed the title Mw incremental search same word indReplaceLogic: incremental doesn't skip over current selection Jul 18, 2024
Copy link
Contributor

github-actions bot commented Jul 18, 2024

Test Results

 1 815 files  ±0   1 815 suites  ±0   1h 30m 13s ⏱️ - 1m 47s
 7 682 tests +2   7 454 ✅ +2  228 💤 ±0  0 ❌ ±0 
24 207 runs  +6  23 458 ✅ +6  749 💤 ±0  0 ❌ ±0 

Results for commit 9b7a707. ± Comparison against base commit 74970b3.

♻️ This comment has been updated with latest results.

@Wittmaxi
Copy link
Contributor Author

Wittmaxi commented Jul 19, 2024

This PR does not adhere to the specification introduced in #2104 (comment) anymore

@Wittmaxi Wittmaxi force-pushed the MW_incremental_search_same_word branch 4 times, most recently from 3932c36 to 0d748ed Compare July 22, 2024 09:10
@Wittmaxi
Copy link
Contributor Author

#1517

Random Fail documented here. @HeikoKlare from my side, this is ready to merge

Incremental search would start incremental search immediately after the
current word. This meant that the incremental search would not go back
to the word where the incremental search initially started.

fixes eclipse-platform#2107
@HeikoKlare HeikoKlare force-pushed the MW_incremental_search_same_word branch from 0d748ed to 9b7a707 Compare July 22, 2024 14:30
@HeikoKlare HeikoKlare changed the title indReplaceLogic: incremental doesn't skip over current selection FindReplaceLogic: incremental doesn't skip over current selection Jul 22, 2024
Copy link
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

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

Works as expected. Makes the usage of incremental search feel much more "natural".

@HeikoKlare HeikoKlare merged commit e34fec9 into eclipse-platform:master Jul 22, 2024
16 checks passed
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.

FindReplaceLogic: incremental search jumps to next word if selection already on search word
2 participants