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 next' and 'Replace all' to Find command in Script Editor #898

Merged
merged 7 commits into from Feb 7, 2022

Conversation

melvingelbard
Copy link
Contributor

  • Added 'Replace next' and 'Replace all' to Find command
  • Fixed 'Previous' button that would search for the next occurrence instead of the previous one
  • Improved UX
  • Fixed find command window losing its owner whenever the Script Editor is closed

melvingelbard and others added 7 commits January 24, 2022 17:17
- Added 'Replace next' and 'Replace all' to Find command
- Fixed 'Previous' button that would search for the next occurrence instead of the previous one
- Improved UX
- Fixed find command window losing its owner whenever the Script Editor would be closed
- 'Find' window is now called 'Find/Replace'
- Improved the user experience of using the 'Find/Replace' command (mainly 'Next' button is looking like it's focused to indicate what the action will be when pressing Enter on a textfield)
- Fixed behaviour to work both on Windows and MacOS
- Fixed 'Replace all'
- Changed 'Replace next' by 'Replace/Find' for better UX
- Renamed "Find" to "Next" for consistency
- Replace all now replace all the text at once. Ctrl/CMD + Z now undoes all the replacements at once, instead of one by one
- Ignore case checkbox state is now saved when reopening the "Find/replace" window 
- Fixed minor bug where hitting Ctrl/CMD + F on a tab would throw an Exception
- Fixed 'undo' after 'Replace all' (it would undo one query at a time, now it undoes all at once)
- Added label next to "Next" button in case 'Previous' or 'Next' doesn't find any matching query.
- 'Replace all' does not match regex anymore (inputting '.' would have replaced all the characters in the text, now it only replaces all the occurrences of '.' in the text)
@petebankhead petebankhead merged commit 0459d5f into qupath:main Feb 7, 2022
@petebankhead petebankhead added this to the v0.4.0 milestone Feb 7, 2022
@melvingelbard melvingelbard deleted the findCommand branch February 9, 2022 11:27
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.

None yet

2 participants