atiaomar1978-hub commented on PR #25407: URL: https://github.com/apache/camel/pull/25407#issuecomment-5224300160
## Bugbot review Reviewed \`cursor/CAMEL-24372-tui-yaml-editor-c587\` vs \`main\`. ### Verified fixes - **Blocker (fixed):** Alt+↑/↓, Ctrl+←/→, Ctrl+Backspace/Delete use \`isKey(KeyCode.*)\` with modifiers — tamboui only matches unmodified keys on \`isUp()\`/\`isLeft()\`/\`isDelete*()\`. - **Major (fixed):** Find-in-edit uses \`jumpToNearestMatch\`, \`n\`/\`N\` aligned with view mode, Esc clears find before discard, paste routes to find input when active. - **Major (fixed):** \`moveBlockUp\` cursor lands on the moved block (\`previous.startRow()\`). - **Tests:** Integration coverage for Alt+↓ block move and Ctrl+←/→ word nav via \`handleKeyEvent\`. ### Remaining suggestions (non-blocking) - Plain \`n\` still navigates find when a term is active (consistent with view mode; may conflict with typing). - Edit render does not highlight find matches inside the TextArea widget. - Undo stack is per-keystroke (100 depth); large pastes may exhaust history quickly. _AI-generated review 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]
