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

Support finding and replacing in document #397

Merged

Conversation

RayZhao1998
Copy link
Collaborator

@RayZhao1998 RayZhao1998 commented Apr 4, 2022

Description

Just enable usesFindBar of CodeEditorTextView to support finding and replacing in a single document.

Related Issue

CodeEditApp/CodeEditTextView#31

Checklist

  • I read and understood the contributing guide as well as the code of conduct
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • Review requested

Screenshots

截屏2022-04-04 11 11 11 PM

截屏2022-04-04 11 41 33 PM

@Angelk90
Copy link
Contributor

Angelk90 commented Apr 4, 2022

@RayZhao1998 : If enter is not pressed, it does not search for the first occurrence.

@RayZhao1998
Copy link
Collaborator Author

RayZhao1998 commented Apr 4, 2022

Good catch! Just enable isIncrementalSearchingEnabled.
截屏2022-04-04 11 41 33 PM

@RayZhao1998 RayZhao1998 force-pushed the feature/finding-in-document branch from 6bbdfcd to f54cd97 Compare April 4, 2022 15:43
@RayZhao1998 RayZhao1998 changed the title Enable CodeEditor usesFindBar to support finding in document Support finding and replacing in document Apr 4, 2022
@Angelk90
Copy link
Contributor

Angelk90 commented Apr 4, 2022

@RayZhao1998 : Ok, now it seems to work.
It would be possible to change the graphic style a bit.
For example the find icon is a little too close to the drop-down menu.

@RayZhao1998 RayZhao1998 requested a review from lukepistrol April 4, 2022 15:51
@RayZhao1998 RayZhao1998 force-pushed the feature/finding-in-document branch from f54cd97 to 1a9f121 Compare April 4, 2022 15:53
@RayZhao1998
Copy link
Collaborator Author

I think this PR can be merge now. I'll find out how to customize find bar and make another PR.

@lukepistrol lukepistrol merged commit 69be9a9 into CodeEditApp:main Apr 5, 2022
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.

3 participants