Skip to content

Commit e37da4f

Browse files
committed
Small clean-up of MethodRow
1 parent 8ec753f commit e37da4f

File tree

1 file changed

+11
-7
lines changed
  • extensions/ql-vscode/src/view/data-extensions-editor

1 file changed

+11
-7
lines changed

extensions/ql-vscode/src/view/data-extensions-editor/MethodRow.tsx

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,10 @@ import { useCallback, useMemo } from "react";
1010
import styled from "styled-components";
1111

1212
import { ExternalApiUsage } from "../../data-extensions-editor/external-api-usage";
13-
import { ModeledMethod } from "../../data-extensions-editor/modeled-method";
13+
import {
14+
ModeledMethod,
15+
ModeledMethodType,
16+
} from "../../data-extensions-editor/modeled-method";
1417

1518
const Dropdown = styled(VSCodeDropdown)`
1619
width: 100%;
@@ -24,7 +27,7 @@ type SupportedUnsupportedSpanProps = {
2427
supported: boolean;
2528
};
2629

27-
const SupportedUnsupportedSpan = styled.span<SupportedUnsupportedSpanProps>`
30+
const SupportSpan = styled.span<SupportedUnsupportedSpanProps>`
2831
color: ${(props) => (props.supported ? "green" : "red")};
2932
`;
3033

@@ -56,11 +59,12 @@ export const MethodRow = ({
5659
const target = e.target as HTMLSelectElement;
5760

5861
onChange(externalApiUsage, {
62+
// If there are no arguments, we will default to "this", which is Argument[-1]
5963
input: argumentsList.length === 0 ? "Argument[-1]" : "Argument[0]",
6064
output: "ReturnType",
6165
kind: "value",
6266
...modeledMethod,
63-
type: target.value as ModeledMethod["type"],
67+
type: target.value as ModeledMethodType,
6468
});
6569
},
6670
[onChange, externalApiUsage, modeledMethod, argumentsList],
@@ -114,15 +118,15 @@ export const MethodRow = ({
114118
return (
115119
<VSCodeDataGridRow>
116120
<VSCodeDataGridCell gridColumn={1}>
117-
<SupportedUnsupportedSpan supported={externalApiUsage.supported}>
121+
<SupportSpan supported={externalApiUsage.supported}>
118122
{externalApiUsage.packageName}.{externalApiUsage.typeName}
119-
</SupportedUnsupportedSpan>
123+
</SupportSpan>
120124
</VSCodeDataGridCell>
121125
<VSCodeDataGridCell gridColumn={2}>
122-
<SupportedUnsupportedSpan supported={externalApiUsage.supported}>
126+
<SupportSpan supported={externalApiUsage.supported}>
123127
{externalApiUsage.methodName}
124128
{externalApiUsage.methodParameters}
125-
</SupportedUnsupportedSpan>
129+
</SupportSpan>
126130
</VSCodeDataGridCell>
127131
<VSCodeDataGridCell gridColumn={3}>
128132
{externalApiUsage.usages.length}

0 commit comments

Comments
 (0)