Skip to content

Commit ca2b732

Browse files
authored
Merge pull request #3983 from github/tuan-nguen/remove-vscode-tags
Remove the use of `VSCodeTag`
2 parents a82dcce + 873ad15 commit ca2b732

File tree

6 files changed

+29
-23
lines changed

6 files changed

+29
-23
lines changed

Diff for: extensions/ql-vscode/src/view/common/CodePaths/ThreadPath.tsx

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { styled } from "styled-components";
2-
import { VSCodeTag } from "@vscode/webview-ui-toolkit/react";
2+
import { Tag } from "../Tag";
33

44
import type {
55
AnalysisMessage,
@@ -56,12 +56,12 @@ export const ThreadPath = ({
5656
</TitleContainer>
5757
{isSource && (
5858
<TagContainer>
59-
<VSCodeTag>Source</VSCodeTag>
59+
<Tag>Source</Tag>
6060
</TagContainer>
6161
)}
6262
{isSink && (
6363
<TagContainer>
64-
<VSCodeTag>Sink</VSCodeTag>
64+
<Tag>Sink</Tag>
6565
</TagContainer>
6666
)}
6767
</HeaderContainer>

Diff for: extensions/ql-vscode/src/view/common/Tag.tsx

+14
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
import { styled } from "styled-components";
2+
3+
export const Tag = styled.span`
4+
background-color: var(--vscode-badge-background);
5+
border: 1px solid var(--vscode-button-border, transparent);
6+
border-radius: 2px;
7+
color: var(--vscode-badge-foreground);
8+
padding: 2px 4px;
9+
text-transform: uppercase;
10+
box-sizing: border-box;
11+
font-family: var(--vscode-font-family);
12+
font-size: 11px;
13+
line-height: 16px;
14+
`;

Diff for: extensions/ql-vscode/src/view/method-modeling/MethodModeling.tsx

+3-5
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { ModelingStatusIndicator } from "../model-editor/ModelingStatusIndicator
44
import type { Method } from "../../model-editor/method";
55
import { MethodName } from "../model-editor/MethodName";
66
import type { ModeledMethod } from "../../model-editor/modeled-method";
7-
import { VSCodeTag } from "@vscode/webview-ui-toolkit/react";
7+
import { Tag } from "../common/Tag";
88
import { ReviewInEditorButton } from "./ReviewInEditorButton";
99
import { MultipleModeledMethodsPanel } from "./MultipleModeledMethodsPanel";
1010
import type { QueryLanguage } from "../../common/query-language";
@@ -39,14 +39,12 @@ const DependencyContainer = styled.div`
3939
margin-bottom: 0.8rem;
4040
`;
4141

42-
const StyledVSCodeTag = styled(VSCodeTag)<{ $visible: boolean }>`
42+
const StyledTag = styled(Tag)<{ $visible: boolean }>`
4343
visibility: ${(props) => (props.$visible ? "visible" : "hidden")};
4444
`;
4545

4646
const UnsavedTag = ({ modelingStatus }: { modelingStatus: ModelingStatus }) => (
47-
<StyledVSCodeTag $visible={modelingStatus === "unsaved"}>
48-
Unsaved
49-
</StyledVSCodeTag>
47+
<StyledTag $visible={modelingStatus === "unsaved"}>Unsaved</StyledTag>
5048
);
5149

5250
export type MethodModelingProps = {

Diff for: extensions/ql-vscode/src/view/model-editor/LibraryRow.tsx

+3-6
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,11 @@ import { calculateModeledPercentage } from "../../model-editor/shared/modeled-pe
77
import { percentFormatter } from "./formatters";
88
import { Codicon } from "../common";
99
import { Mode } from "../../model-editor/shared/mode";
10-
import {
11-
VSCodeButton,
12-
VSCodeDivider,
13-
VSCodeTag,
14-
} from "@vscode/webview-ui-toolkit/react";
10+
import { VSCodeButton, VSCodeDivider } from "@vscode/webview-ui-toolkit/react";
1511
import type { ModelEditorViewState } from "../../model-editor/shared/view-state";
1612
import type { AccessPathSuggestionOptions } from "../../model-editor/suggestions";
1713
import type { ModelEvaluationRunState } from "../../model-editor/shared/model-evaluation-run-state";
14+
import { Tag } from "../common/Tag";
1815

1916
const LibraryContainer = styled.div`
2017
background-color: var(--vscode-peekViewResult-background);
@@ -169,7 +166,7 @@ export const LibraryRow = ({
169166
<ModeledPercentage>
170167
{percentFormatter.format(modeledPercentage / 100)} modeled
171168
</ModeledPercentage>
172-
{hasUnsavedChanges ? <VSCodeTag>UNSAVED</VSCodeTag> : null}
169+
{hasUnsavedChanges ? <Tag>UNSAVED</Tag> : null}
173170
</NameContainer>
174171
{viewState.showGenerateButton &&
175172
viewState.mode === Mode.Application && (

Diff for: extensions/ql-vscode/src/view/model-editor/MethodClassifications.tsx

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
import { useMemo } from "react";
22
import type { Method } from "../../model-editor/method";
33
import { CallClassification } from "../../model-editor/method";
4-
import { VSCodeTag } from "@vscode/webview-ui-toolkit/react";
54
import { styled } from "styled-components";
5+
import { Tag } from "../common/Tag";
66

77
const ClassificationsContainer = styled.div`
88
display: inline-flex;
99
flex-direction: row;
1010
gap: 0.5rem;
1111
`;
1212

13-
const ClassificationTag = styled(VSCodeTag)`
13+
const ClassificationTag = styled(Tag)`
1414
font-size: 0.75em;
1515
white-space: nowrap;
1616
`;

Diff for: extensions/ql-vscode/src/view/model-editor/ModelEditor.tsx

+4-7
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,6 @@
11
import { useCallback, useEffect, useMemo, useState } from "react";
22
import type { ToModelEditorMessage } from "../../common/interface-types";
3-
import {
4-
VSCodeButton,
5-
VSCodeCheckbox,
6-
VSCodeTag,
7-
} from "@vscode/webview-ui-toolkit/react";
3+
import { VSCodeButton, VSCodeCheckbox } from "@vscode/webview-ui-toolkit/react";
84
import { styled } from "styled-components";
95
import type { Method } from "../../model-editor/method";
106
import type { ModeledMethod } from "../../model-editor/modeled-method";
@@ -22,6 +18,7 @@ import type { AccessPathSuggestionOptions } from "../../model-editor/suggestions
2218
import type { ModelEvaluationRunState } from "../../model-editor/shared/model-evaluation-run-state";
2319
import { ModelEvaluation } from "./ModelEvaluation";
2420
import { useMessageFromExtension } from "../common/useMessageFromExtension";
21+
import { Tag } from "../common/Tag";
2522

2623
const LoadingContainer = styled.div`
2724
text-align: center;
@@ -305,9 +302,9 @@ export function ModelEditor({
305302
<ViewTitle>
306303
{getLanguageDisplayName(viewState.extensionPack.language)}
307304
</ViewTitle>
308-
<VSCodeTag>
305+
<Tag>
309306
{percentFormatter.format(modeledPercentage / 100)} modeled
310-
</VSCodeTag>
307+
</Tag>
311308
</HeaderRow>
312309
<HeaderRow>
313310
<>{viewState.extensionPack.name}</>

0 commit comments

Comments
 (0)