caleb531 / workday-time-calculator / 7000855007

92%

push

github

Fix next completion not canceling when deleting trailing space

This fixes a bug where, when completing a word and then pressing the
Delete key to delete the trailing space, the completion placeholder was
still active. Ironically, this was due to the cancel() call in the
EditorComponent.autocomplete() method; the cancel was resetting the
autocompleter model's (last-computed) 'completionQuery' property, which
was causing the substring optimization in fetchCompletions() to run.

The fix seems to be removing the cancel() call, but then (crucially)
resetting the rendered autocomplete in the DOM to prevent a flicker of
the previous placeholder text when tab-completing past the initial
autocomplete run.

384 of 417 branches covered (0.0%)

Branch coverage included in aggregate %.

0 of 1 new or added line in 1 file covered. (0.0%)

2178 of 2311 relevant lines covered (94.24%)

397.14 hits per line