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

Find/Replace Overlay: correctly set scope mode #2057

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

Wittmaxi
Copy link

@Wittmaxi Wittmaxi commented Jul 8, 2024

Depending on the file format, the line separators are different. This PR allows detecting both when deciding whether to set the scoped search mode or not when opening the Overlay with an active selection.

fixes #2054

To test: use a file formatted on UNIX (line separators: \n), select multiple lines, open overaly

grafik

@Wittmaxi Wittmaxi force-pushed the MW_line_separators branch from 666db2f to 88f1f62 Compare July 8, 2024 11:37
Copy link
Contributor

github-actions bot commented Jul 8, 2024

Test Results

 1 815 files  ±0   1 815 suites  ±0   1h 38m 15s ⏱️ + 2m 7s
 7 665 tests +2   7 437 ✅ +2  228 💤 ±0  0 ❌ ±0 
24 156 runs  +6  23 407 ✅ +6  749 💤 ±0  0 ❌ ±0 

Results for commit 3153f2c. ± Comparison against base commit 7cb5523.

♻️ This comment has been updated with latest results.

@Wittmaxi
Copy link
Author

Wittmaxi commented Jul 8, 2024

random fail documented #1808

Wittmaxi pushed a commit to Wittmaxi/eclipse.platform.ui that referenced this pull request Jul 8, 2024
Stores the search options of the overlay and actively reloads them, even
when switching between overlays.

fixes eclipse-platform#2057
@HeikoKlare HeikoKlare force-pushed the MW_line_separators branch 2 times, most recently from ec40478 to 9b7145a Compare July 9, 2024 12:47
Depending on the file format, the line separators are different.
This PR allows detecting both when deciding whether to set the scoped
search mode or not when opening the Overlay with an active selection.

fixes eclipse-platform#2054
@HeikoKlare HeikoKlare force-pushed the MW_line_separators branch from 9b7145a to 3153f2c Compare July 9, 2024 14:22
@HeikoKlare HeikoKlare merged commit f488dac into eclipse-platform:master Jul 9, 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.

Find/Replace overlay: UNIX-line separators are not interpreted correctly on Windows
2 participants