Skip to content

Commit 5c444eb

Browse files
revert comments
1 parent 111b40c commit 5c444eb

File tree

1 file changed

+8
-12
lines changed
  • packages/cursorless-engine/src/processTargets/modifiers/scopeHandlers/SurroundingPairScopeHandler

1 file changed

+8
-12
lines changed

packages/cursorless-engine/src/processTargets/modifiers/scopeHandlers/SurroundingPairScopeHandler/InteriorScopeHandler.ts

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -79,12 +79,10 @@ export class InteriorScopeHandler extends BaseScopeHandler {
7979
this.languageId,
8080
);
8181

82-
/*
83-
If the scope type is explicit (ie, the user has specified a scope
84-
type), then we want to prioritize language scopes. The user might
85-
have said something like "inside element" and then we don't want to
86-
yield the interior of the `<div>` pair first.
87-
*/
82+
// If the scope type is explicit (ie, the user has specified a scope
83+
// type), then we want to prioritize language scopes. The user might
84+
// have said something like "inside element" and then we don't want to
85+
// yield the interior of the `<div>` pair first.
8886
if (this.scopeType.explicitScopeType) {
8987
if (languageScopeHandler == null) {
9088
return pairScopeHandler;
@@ -114,12 +112,10 @@ export class InteriorScopeHandler extends BaseScopeHandler {
114112
}
115113

116114
private shouldYield(targetDomain: Range, scope: TargetScope): boolean {
117-
/*
118-
For an explicit scope type we only yield scopes that are contained within
119-
the target domain. E.g the user said "inside token", then we don't want
120-
to yield scopes that are larger than the token. The definition of an
121-
interior is that it's inside the scope.
122-
*/
115+
// For an explicit scope type we only yield scopes that are contained within
116+
// the target domain. E.g the user said "inside token", then we don't want
117+
// to yield scopes that are larger than the token. The definition of an
118+
// interior is that it's inside the scope.
123119
return (
124120
!this.scopeType.explicitScopeType || targetDomain.contains(scope.domain)
125121
);

0 commit comments

Comments
 (0)