[Bug] Search is not working properly and destructing the text

[unbreakabl3]

Bug Description

When searching for a word and looping with Enter, the following is happening each click on Enter

Screen.Recording.2024-06-07.at.15.37.13.mov

When searching for a word and looping with arrows, the following is happening each click on arrow. Once the jump down happened, there is no way to go back.

Screen.Recording.2024-06-07.at.15.39.57.mov

How to Reproduce

Try to find some word, which appears more than once.

Expected Behavior

Looping across should happen and formatting of the text should not be done.

Operating System

MacOS 14.5

AppFlowy Version(s)

0.5.9

Screenshots

No response

Additional Context

No response

[Xazin]

Since this was originally implemented by you @MayurSMahajan , I wonder if you want to also work on improvements and bug fixes for inline document search?

[MayurSMahajan]

I can take a look at this one, but if something is priority, I request the team to handle it due to my time constraints.

[MayurSMahajan]

Hey @Xazin,
So after observing this issue, I have noticed two things:

1. For scroll freeze: There is a jumpTo() method call which causes the editor to shift all content down and get frozen there.
2. For selected text getting replaced after pressing enter: This only happens when the focus is not on the Find Text Input field. Thus the Enter key event causes the editor to replace the selected text, rather than navigating to the next match.

I have solved the first part of the issue by removing the jumpTo() call, since anyway making a new selection causes the editor to scroll to the selected content. I will create a PR for this in the AppFlowyEditor repo.

I need some help tackling the second part.
Approach 1: I tried implementing Shortcuts, Actions, and Intent to listen to the Enter key event on the FindReplace widget to kind of intercept the Enter key event before it is handled by the Editor itself, which as expected, deletes the selected content. But I am not able to do it.
Approach 2: I wrapped the FindReplace widget with KeyboardEventListener, this too causes the editor to take over and insert a newline after handling the onKeyEvent.

Can you give me a hint for deciding a strategy towards stopping the editor from deleting the content?
Anyway, I will continue brainstorming, just updating everyone on the current progress of the issue.

[annieappflowy]

Fixed