Skip to content

Rename the Find in Files function to Replace in Files and then build Replace in Files#906

Closed
matthewyang204 wants to merge 34 commits intodail8859:masterfrom
matthewyang204:master
Closed

Rename the Find in Files function to Replace in Files and then build Replace in Files#906
matthewyang204 wants to merge 34 commits intodail8859:masterfrom
matthewyang204:master

Conversation

@matthewyang204
Copy link
Contributor

@matthewyang204 matthewyang204 commented Dec 16, 2025

Changes:

  1. Rename the Find in Files button to Replace in Files because Find in Files is 1. pretty useless 2. too hard to implement (unless someone else wants to do so)
  2. Build a Replace in Files where the user can replace stuff in a directory of their choice recursively

Please check that you like it, and then make any changes you want. Otherwise, feel free to merge. Also, given the number of commits, you may not want to squash the entire thing into one patch

matthewyang204 and others added 30 commits June 16, 2025 17:37
@matthewyang204
Update updates.json
75583a4
@matthewyang204
Merge branch 'dail8859:master' into master
68ce33f
@matthewyang204
Merge branch 'dail8859:master' into master
7d5695b
@matthewyang204
Merge branch 'master' of https://github.com/dail8859/NotepadNext
bdf9ed5
@matthewyang204
Need to ignore EVERY build folder
550ec39
@matthewyang204
Add Replace In Files elements
6d104b9
@matthewyang204
Fix labels
af9a72f
@matthewyang204
Only show the replace in files options in Replace tab
b79d4c7
@matthewyang204
Add new button for replacing all in files
7a964c0
@matthewyang204
Update the .ui file for the FindReplaceDialog
3aabe22
@matthewyang204
Show on second index & make comboReplaceInFiles editable
1a4d1b5
@matthewyang204
Include some more files for find and replace
25963d8
@matthewyang204
Add button to select directory
4a8d65d
@matthewyang204
Allow user to select directory to search in
e0f582e
@matthewyang204
Accidentally directly pasted from a directory
8c50841 
Still learning Qt C++
@matthewyang204
Otherwise, create another item
8eeeac6
@matthewyang204
Add function to check if file is binary
478c637
@matthewyang204
Move isBinary to top
bd7a346
@matthewyang204
Write function to replace in files
3879438
@matthewyang204
Show match count
2182cb8
@matthewyang204
This is only for files
65e4ffd
@matthewyang204
check binary files *properly*
52a2d8d
@matthewyang204
This should become ReplaceInFiles
c7f117f
@matthewyang204
This goes after Replace
5395d06
@matthewyang204
Merge branch 'dail8859:master' into master
999f258
@matthewyang204
Heighten a little bit
d40ae6a 
Should fix problems with squeezing
@matthewyang204
Set more reasonable size along with setting a minimumSize property
7cc9fb2
@matthewyang204
Merge branch 'master' of github.com:matthewyang204/NotepadNext
61dfef5
@matthewyang204 @github-actions
Update translation files
32a6be7
@matthewyang204
Merge branch 'dail8859:master' into master
ba5819b
@matthewyang204
Copy link
Contributor Author

matthewyang204 commented Dec 17, 2025

@dail8859 Can you please check out this PR? It's been a long time and you have not checked the previous PR I opened, so I re-opened this one with the same commits.

@matthewyang204 matthewyang204 mentioned this pull request Jan 6, 2026
Merged
@matthewyang204 matthewyang204 deleted the branch dail8859:master January 11, 2026 04:55
@matthewyang204 matthewyang204 deleted the master branch January 11, 2026 04:56
@matthewyang204 matthewyang204 mentioned this pull request Jan 11, 2026
Merged
@matthewyang204
Copy link
Contributor Author

matthewyang204 commented Jan 11, 2026

I broke this PR when I renamed the branch as I was sorting out my fork. Please go to #913, which is the reopened PR.

@matthewyang204 matthewyang204 mentioned this pull request Jan 12, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

translation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@matthewyang204