microsoft / winfile

Original Windows File Manager (winfile) with enhancements
MIT License
6.82k stars 706 forks source link

TypeAhead Fix #333

Closed schinagl closed 2 years ago

schinagl commented 2 years ago

General

With #290 TypeAhead was fixed, but we all overlooked one use-case

Problem

Assume you have the following files in a directory

as
bla
san
system
syxsser
zzz

The attached .bat file creates such a situation.

About the changes

With #290 we overlooked, that only consecutive same characters shall move the cursor ahead by one. The #290 solution also moves ahead by one, if any character typed within a word is the same as the first, but other characters are in between.

I now cross checked the fixed behavior with explorer and it is exactly the same.

craigwims commented 2 years ago

The behavior looks good. A couple of small things to fix.

schinagl commented 2 years ago

code review incooperated