Skip to content

Commit 628f711

Browse files
Copilotd10c
andcommitted
Fix type errors for @vscode-elements/react-elements compatibility
Changed event handler types from specific React event types to generic Event types to match @vscode-elements/react-elements library expectations: - CodeFlowsDropdown: ChangeEvent<HTMLSelectElement> → Event - DataGrid: React.Ref<HTMLElement | undefined> → React.Ref<HTMLDivElement> - TextButton: Make $size prop optional - ModelTypeTextbox: ChangeEvent<HTMLSelectElement> → Event - RepoRow: ChangeEvent<HTMLInputElement> → Event - ModelAlertsSort, RepositoriesFilter, RepositoriesResultFormat, RepositoriesSort: InputEvent → Event This fixes the TypeScript compilation errors in CI. Co-authored-by: d10c <9970661+d10c@users.noreply.github.com>
1 parent 20d7a4e commit 628f711

File tree

9 files changed

+15
-14
lines changed

9 files changed

+15
-14
lines changed

extensions/ql-vscode/src/view/common/CodePaths/CodeFlowsDropdown.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import type { ChangeEvent, SetStateAction } from "react";
1+
import type { SetStateAction } from "react";
22
import { useCallback } from "react";
33
import {
44
VscodeOption,
@@ -25,8 +25,8 @@ export const CodeFlowsDropdown = ({
2525
setSelectedCodeFlow,
2626
}: CodeFlowsDropdownProps) => {
2727
const handleChange = useCallback(
28-
(e: ChangeEvent<HTMLSelectElement>) => {
29-
const selectedOption = e.target;
28+
(e: Event) => {
29+
const selectedOption = e.target as HTMLSelectElement;
3030
const selectedIndex = parseInt(selectedOption.value);
3131
setSelectedCodeFlow(codeFlows[selectedIndex]);
3232
},

extensions/ql-vscode/src/view/common/DataGrid.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ export const DataGridRow = forwardRef(
8383
"data-testid": testId,
8484
onClick,
8585
}: DataGridRowProps,
86-
ref?: React.Ref<HTMLElement | undefined>,
86+
ref?: React.Ref<HTMLDivElement>,
8787
) => (
8888
<StyledDataGridRow
8989
$focused={focused}
@@ -135,7 +135,7 @@ export const DataGridCell = forwardRef(
135135
className,
136136
children,
137137
}: DataGridCellProps,
138-
ref?: React.Ref<HTMLElement | undefined>,
138+
ref?: React.Ref<HTMLDivElement>,
139139
) => {
140140
return (
141141
<StyledDataGridCell

extensions/ql-vscode/src/view/common/TextButton.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { styled } from "styled-components";
22

33
type Size = "x-small" | "small" | "medium" | "large" | "x-large";
44

5-
const StyledButton = styled.button<{ $size: Size }>`
5+
const StyledButton = styled.button<{ $size?: Size }>`
66
background: none;
77
color: var(--vscode-textLink-foreground);
88
border: none;

extensions/ql-vscode/src/view/model-alerts/ModelAlertsSort.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ type Props = {
2020

2121
export const ModelAlertsSort = ({ value, onChange, className }: Props) => {
2222
const handleInput = useCallback(
23-
(e: InputEvent) => {
23+
(e: Event) => {
2424
const target = e.target as HTMLSelectElement;
2525

2626
onChange(target.value as SortKey);

extensions/ql-vscode/src/view/model-editor/ModelTypeTextbox.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export const ModelTypeTextbox = ({
3333
setValue(modeledMethod[typeInfo]);
3434
}, [modeledMethod, typeInfo]);
3535

36-
const handleChange = useCallback((e: ChangeEvent<HTMLSelectElement>) => {
36+
const handleChange = useCallback((e: Event) => {
3737
const target = e.target as HTMLSelectElement;
3838

3939
setValue(target.value);

extensions/ql-vscode/src/view/variant-analysis/RepoRow.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -229,17 +229,18 @@ export const RepoRow = ({
229229
e.stopPropagation();
230230
}, []);
231231
const onChangeCheckbox = useCallback(
232-
(e: ChangeEvent<HTMLInputElement>) => {
232+
(e: Event) => {
233+
const target = e.target as HTMLInputElement;
233234
// This is called on first render, but we don't really care about this value
234-
if (e.target.checked === undefined) {
235+
if (target.checked === undefined) {
235236
return;
236237
}
237238

238239
if (!repository.id) {
239240
return;
240241
}
241242

242-
onSelectedChange?.(repository.id, e.target.checked);
243+
onSelectedChange?.(repository.id, target.checked);
243244
},
244245
[onSelectedChange, repository],
245246
);

extensions/ql-vscode/src/view/variant-analysis/RepositoriesFilter.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ type Props = {
2020

2121
export const RepositoriesFilter = ({ value, onChange, className }: Props) => {
2222
const handleInput = useCallback(
23-
(e: InputEvent) => {
23+
(e: Event) => {
2424
const target = e.target as HTMLSelectElement;
2525

2626
onChange(target.value as FilterKey);

extensions/ql-vscode/src/view/variant-analysis/RepositoriesResultFormat.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ export const RepositoriesResultFormat = ({
2424
className,
2525
}: Props) => {
2626
const handleInput = useCallback(
27-
(e: InputEvent) => {
27+
(e: Event) => {
2828
const target = e.target as HTMLSelectElement;
2929

3030
onChange(target.value as ResultFormat);

extensions/ql-vscode/src/view/variant-analysis/RepositoriesSort.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ type Props = {
2020

2121
export const RepositoriesSort = ({ value, onChange, className }: Props) => {
2222
const handleInput = useCallback(
23-
(e: InputEvent) => {
23+
(e: Event) => {
2424
const target = e.target as HTMLSelectElement;
2525

2626
onChange(target.value as SortKey);

0 commit comments

Comments
 (0)