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

Added 'Replace in files' functionality to text editors #36044

Merged

Conversation

dreamsComeTrue
Copy link
Contributor

Reuses same 'find_in_files' dialog as for regular search in files.
Sets Ctrl + Shift + R as shortcut for bringing up 'Replace in Files' dialog.

Fixes issue #31146

Copy link
Member

@Paulb23 Paulb23 left a comment

Choose a reason for hiding this comment

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

Looks okay, just one question below.

editor/find_in_files.cpp Outdated Show resolved Hide resolved
@dreamsComeTrue dreamsComeTrue force-pushed the fix-replace-all-shortcut branch from 25821cc to 4c80bea Compare February 13, 2020 22:45
@@ -886,7 +951,7 @@ void FindInFilesPanel::apply_replaces_in_file(String fpath, const Vector<Result>
}

String FindInFilesPanel::get_replace_text() {
return _replace_line_edit->get_text().strip_edges();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

That bit was there since forever - now, replacement text is fully set (even with border spaces)

Copy link
Member

@Paulb23 Paulb23 left a comment

Choose a reason for hiding this comment

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

LGTM.

@akien-mga akien-mga merged commit 710d829 into godotengine:master Feb 14, 2020
@akien-mga
Copy link
Member

Thanks!

@akien-mga akien-mga added this to the 4.0 milestone Feb 14, 2020
@dreamsComeTrue
Copy link
Contributor Author

Hoorayy! Thank you, sir :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants