Use single UserInputHandler for all other _Handler classes

Open ReagentX opened this issue 5 years ago • 4 comments

Currently we initialize a new one every time we want to get input.

Nov 25 '20 21:11 ReagentX

This might be related to why I get double-inputs when dealing with anything interactive. I can't enter "0" at a prompt, only "00".

This is on Windows 10 Terminal with Powershell 7.4.10 , logria 0.4.1 from cargo 1.87.0. Same result on git bash in its own window (not Win Terminal).

Jun 22 '25 00:06 Tzrlk

That issue is caused by https://github.com/crossterm-rs/crossterm/issues/752.

Jun 22 '25 01:06 ReagentX

Amazing that they report the only terminal not to have the bug is the old cmd shell. I didn't test that because it seemed absurd. I guess I'll give it a go.

Would you like me to raise this as a separate issue?

Jun 22 '25 07:06 Tzrlk

Yes please, seems easy enough to resolve.

Jun 22 '25 15:06 ReagentX