-
-
Notifications
You must be signed in to change notification settings - Fork 760
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
nnn visually hangs when searching again without redrawing #1117
Comments
Did you read the release notes?
Pressing Esc does not clear a filter (read the docs). So searching for something which exists the the dir but not on the current filter will not change anything in the view and you won't see anything typed because there is no match (the char is ignored). |
This is a change in behaviour. Now that you know it, live with it ;). It's better than typing characters even when there is not match. |
Hmm, I see. The experience here is a bit jarring. But if it's intended then I guess there's nothing to be done here. |
@0xACE you type and you don't see anything typed. Doesn't that indicate something went wrong? I can revert this back. However, just confirm if you think it's difficult to get accustomed to the behaviour. |
Or do you think it's better to have it only if type-to-nav mode is on? I would prefer to keep it consistent. |
I think we should revert it... I can see why it won't feel natural to users. |
I don't think people are staring at the bottom bar while searching/typing (I don't, and I'd assume most keyboard driven users are no different), so seeing the list clear up is a more natural and conventional behavior for when there is no match. As for |
And you don't see that the list isn't changing? |
Sure, but its not a conventional behaviour making the experience jarring. Anyhow, I have the commit (3ef50f0) reverted on my system, so this is no longer going to be an issue for me. |
I have reverted this as well. This does disrupt the workflow for the uninitiated. |
Environment details (Put
x
in the checkbox along with the information)nnn
masterExact steps to reproduce the issue
/foo
and then escapeesc
/bar
Pressing
esc
again gets it out of the hanged state.P.S. v4.1.1 does not have this problem.
The text was updated successfully, but these errors were encountered: