Bugfix: search backwards with operator (#5041)

This PR removes an unnecessary restriction that prevents search backwards ("?") with an operator.

Fixes #4506
Fixes #4647

src/actions/commands/actions.ts

@@ -1204,13 +1204,6 @@ export class CommandSearchBackwards extends BaseCommand {
   isMotion = true;
   isJump = true;
 
-  public doesActionApply(vimState: VimState, keysPressed: string[]): boolean {
-    // Prevent collision with `g?` (rot13 operator)
-    return (
-      super.doesActionApply(vimState, keysPressed) && vimState.recordedState.operator === undefined
-    );
-  }
-
   public async exec(position: Position, vimState: VimState): Promise<VimState> {
     globalState.searchState = new SearchState(
       SearchDirection.Backward,

test/mode/modeNormal.test.ts

@@ -2375,6 +2375,22 @@ suite('Mode Normal', () => {
     endMode: Mode.Normal,
   });
 
+  newTest({
+    title: 'Can delete with search forward',
+    start: ['foo |junk junk bar'],
+    keysPressed: 'd/bar\n',
+    end: ['foo |bar'],
+    endMode: Mode.Normal,
+  });
+
+  newTest({
+    title: 'Can delete with search backward',
+    start: ['foo junk garbage trash |bar'],
+    keysPressed: 'd?junk\n',
+    end: ['foo |bar'],
+    endMode: Mode.Normal,
+  });
+
   newTest({
     title: 'Can do C',
     start: ['export const options = {', '|', '};'],