atiaomar1978-hub commented on PR #25408: URL: https://github.com/apache/camel/pull/25408#issuecomment-5226803148
Thanks @davsclaus for the review — addressed both confirmed issues in `8559ab1`: ### Confirmed issues 1. **`n`/`N` find navigation in edit mode** — Edit mode now uses **Ctrl+N** / **Ctrl+Shift+N** to step matches while a find term is active. Plain `n`/`N` are no longer consumed, so users can type freely. Footer hint updated to show `Ctrl+N` for next match. Test: `plainNInsertsWhileFindTermActive`. 2. **`moveBlockDown` cursor** — Cursor now follows the moved block (same intent as the `moveBlockUp` fix). Test: `moveBlockDownCursorFollowsMovedBlock`. ### Questions - **`editText()` / `editState()`** — Yes, these are package-private for tests that drive the edit buffer directly. Added brief javadoc on both. - **`setValidateOnSave(false)`** — Already on `main` in `SourceViewer` (also used in `SourceViewerEditTest`); not introduced by this PR. Ready for another look when you have time. _AI-generated comment on behalf of atiaomar1978-hub._ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
