From 2908331bff340dbc89ae163f731c1245e712ca41 Mon Sep 17 00:00:00 2001 From: Seif Soliman Date: Fri, 11 Jul 2025 17:11:54 +0300 Subject: [PATCH] fix(zen): remove input length check to prevent jump to hidden line (@byseif21, @NadAlaba) (#6714) Closes #6697 --- frontend/src/ts/controllers/input-controller.ts | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/frontend/src/ts/controllers/input-controller.ts b/frontend/src/ts/controllers/input-controller.ts index 8b1ff4bfb..183de6d59 100644 --- a/frontend/src/ts/controllers/input-controller.ts +++ b/frontend/src/ts/controllers/input-controller.ts @@ -731,10 +731,19 @@ async function handleChar( //stop the word jump by slicing off the last character, update word again // dont do it in replace typos, because it might trigger in the middle of a wrd // when using non monospace fonts + /** + * NOTE: this input length > 1 guard, added in commit bc94a64, + * aimed to prevent some input blocking issue after test restarts. + * + * This check was found to cause a jump to a hidden 3rd line bug in zen mode (#6697) + * So commented due to the zen bug and the original issue not being reproducible, + * + * REVISIT this logic if any INPUT or WORD JUMP issues reappear. + */ if ( activeWordTopBeforeJump < newActiveTop && - !TestUI.lineTransition && - TestInput.input.current.length > 1 + !TestUI.lineTransition + // TestInput.input.current.length > 1 ) { if (Config.mode === "zen" || Config.indicateTypos === "replace") { if (!Config.showAllLines) void TestUI.lineJump(activeWordTopBeforeJump);