Skip to content

Commit 74ed506

Browse files
Use interior target in document target
1 parent 3266cc7 commit 74ed506

File tree

2 files changed

+18
-17
lines changed

2 files changed

+18
-17
lines changed

packages/cursorless-engine/src/processTargets/targets/BaseTarget.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ export abstract class BaseTarget<
6969
get editor() {
7070
return this.state.editor;
7171
}
72+
7273
get isReversed() {
7374
return this.state.isReversed;
7475
}
@@ -114,6 +115,7 @@ export abstract class BaseTarget<
114115
getInterior(): Target[] | undefined {
115116
return undefined;
116117
}
118+
117119
getBoundary(): Target[] | undefined {
118120
return undefined;
119121
}

packages/cursorless-engine/src/processTargets/targets/DocumentTarget.ts

Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
import type { Range } from "@cursorless/common";
2-
import { shrinkRangeToFitContent } from "../../util/selectionUtils";
3-
import type { CommonTargetParameters } from "./BaseTarget";
4-
import { BaseTarget } from "./BaseTarget";
5-
import { PlainTarget } from "./PlainTarget";
2+
import { BaseTarget, type CommonTargetParameters } from "./BaseTarget";
3+
import { InteriorTarget } from "./InteriorTarget";
64

75
export class DocumentTarget extends BaseTarget<CommonTargetParameters> {
86
type = "DocumentTarget";
@@ -13,27 +11,28 @@ export class DocumentTarget extends BaseTarget<CommonTargetParameters> {
1311
super(parameters);
1412
}
1513

16-
getLeadingDelimiterTarget() {
17-
return undefined;
18-
}
19-
getTrailingDelimiterTarget() {
20-
return undefined;
21-
}
22-
getRemovalRange(): Range {
23-
return this.contentRange;
24-
}
25-
2614
getInterior() {
2715
return [
28-
// Use plain target instead of interior target since we want the same content and removal range for a document interior.
29-
new PlainTarget({
16+
new InteriorTarget({
3017
editor: this.editor,
3118
isReversed: this.isReversed,
32-
contentRange: shrinkRangeToFitContent(this.editor, this.contentRange),
19+
fullInteriorRange: this.contentRange,
3320
}),
3421
];
3522
}
3623

24+
getRemovalRange(): Range {
25+
return this.contentRange;
26+
}
27+
28+
getLeadingDelimiterTarget() {
29+
return undefined;
30+
}
31+
32+
getTrailingDelimiterTarget() {
33+
return undefined;
34+
}
35+
3736
protected getCloneParameters() {
3837
return this.state;
3938
}

0 commit comments

Comments
 (0)