Skip to content

Commit 8abc5a4

Browse files
Remove jsx attribute from item scope (#2039)
Fixes #2034 ## Checklist - [x] I have added [tests](https://www.cursorless.org/docs/contributing/test-case-recorder/) - [-] I have updated the [docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and [cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet) - [-] I have not broken the cheatsheet
1 parent a70c889 commit 8abc5a4

File tree

9 files changed

+48
-107
lines changed

9 files changed

+48
-107
lines changed

packages/cursorless-engine/src/languages/getNodeMatcher.ts

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ import { LegacyLanguageId } from "./LegacyLanguageId";
1313
import cpp from "./cpp";
1414
import csharp from "./csharp";
1515
import go from "./go";
16-
import { patternMatchers as html } from "./html";
1716
import java from "./java";
1817
import latex from "./latex";
1918
import markdown from "./markdown";
@@ -59,7 +58,6 @@ export const languageMatchers: Record<
5958
csharp,
6059
clojure,
6160
go,
62-
html,
6361
java,
6462
javascript: typescript,
6563
javascriptreact: typescript,
@@ -73,7 +71,6 @@ export const languageMatchers: Record<
7371
rust,
7472
typescript,
7573
typescriptreact: typescript,
76-
xml: html,
7774
};
7875

7976
function matcherIncludeSiblings(matcher: NodeMatcher): NodeMatcher {

packages/cursorless-engine/src/languages/html.ts

Lines changed: 0 additions & 11 deletions
This file was deleted.

packages/cursorless-engine/src/languages/typescript.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import { argumentMatcher, createPatternMatchers } from "../util/nodeMatchers";
55
const nodeMatchers: Partial<
66
Record<SimpleScopeTypeType, NodeMatcherAlternative>
77
> = {
8-
collectionItem: "jsx_attribute",
98
argumentOrParameter: argumentMatcher("formal_parameters", "arguments"),
109
};
1110

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/html/clearEveryItem.yml

Lines changed: 0 additions & 24 deletions
This file was deleted.

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/html/clearItem.yml

Lines changed: 0 additions & 22 deletions
This file was deleted.
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
languageId: typescriptreact
2+
command:
3+
version: 6
4+
spokenForm: change every item
5+
action:
6+
name: clearAndSetSelection
7+
target:
8+
type: primitive
9+
modifiers:
10+
- type: everyScope
11+
scopeType: {type: collectionItem}
12+
usePrePhraseSnapshot: true
13+
initialState:
14+
documentContents: <div value={[1, 2]}>
15+
selections:
16+
- anchor: {line: 0, character: 13}
17+
active: {line: 0, character: 13}
18+
marks: {}
19+
finalState:
20+
documentContents: <div value={[, ]}>
21+
selections:
22+
- anchor: {line: 0, character: 13}
23+
active: {line: 0, character: 13}
24+
- anchor: {line: 0, character: 15}
25+
active: {line: 0, character: 15}
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
languageId: typescriptreact
2+
command:
3+
version: 6
4+
spokenForm: change item
5+
action:
6+
name: clearAndSetSelection
7+
target:
8+
type: primitive
9+
modifiers:
10+
- type: containingScope
11+
scopeType: {type: collectionItem}
12+
usePrePhraseSnapshot: true
13+
initialState:
14+
documentContents: <div value={[1, 2]}>
15+
selections:
16+
- anchor: {line: 0, character: 13}
17+
active: {line: 0, character: 13}
18+
marks: {}
19+
finalState:
20+
documentContents: <div value={[, 2]}>
21+
selections:
22+
- anchor: {line: 0, character: 13}
23+
active: {line: 0, character: 13}

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/typescript/clearEveryItem.yml

Lines changed: 0 additions & 24 deletions
This file was deleted.

packages/cursorless-vscode-e2e/src/suite/fixtures/recorded/languages/typescript/clearItem4.yml

Lines changed: 0 additions & 22 deletions
This file was deleted.

0 commit comments

Comments
 (0)