Skip to content

Commit 6042b7f

Browse files
committed
Remove extraneous comments.
1 parent 2fac8c3 commit 6042b7f

File tree

2 files changed

+4
-56
lines changed

2 files changed

+4
-56
lines changed

webview-ui/src/components/mcp/McpView.tsx

Lines changed: 2 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -145,17 +145,6 @@ const ServerRow = ({ server, alwaysAllowMcp }: { server: McpServer; alwaysAllowM
145145
{ value: 3600, label: t("mcp:networkTimeout.options.60minutes") },
146146
]
147147

148-
// const getStatusColor = () => {
149-
// switch (server.status) {
150-
// case "connected":
151-
// return "var(--vscode-testing-iconPassed)"
152-
// case "connecting":
153-
// return "var(--vscode-charts-yellow)"
154-
// case "disconnected":
155-
// return "var(--vscode-testing-iconFailed)"
156-
// }
157-
// }
158-
159148
const handleRowClick = () => {
160149
if (server.status === "connected") {
161150
setIsExpanded(!isExpanded)
@@ -364,16 +353,7 @@ const ServerRow = ({ server, alwaysAllowMcp }: { server: McpServer; alwaysAllowM
364353
)
365354
) : (
366355
<div className="text-[13px] bg-vscode-textCodeBlock-background rounded-b-[4px] w-full">
367-
<div
368-
className="text-vscode-testing-iconFailed mb-[8px] px-[10px] overflow-wrap-break-word break-words"
369-
// style={{
370-
// color: "var(--vscode-testing-iconFailed)",
371-
// marginBottom: "8px",
372-
// padding: "0 10px",
373-
// overflowWrap: "break-word",
374-
// wordBreak: "break-word",
375-
// }}
376-
>
356+
<div className="text-vscode-testing-iconFailed mb-[8px] px-[10px] overflow-wrap-break-word break-words">
377357
{server.error &&
378358
server.error.split("\n").map((item, index) => (
379359
<React.Fragment key={index}>
@@ -386,9 +366,7 @@ const ServerRow = ({ server, alwaysAllowMcp }: { server: McpServer; alwaysAllowM
386366
appearance="secondary"
387367
onClick={handleRestart}
388368
disabled={server.status === "connecting"}
389-
className="w-[calc(100%-20px)] mx-[10px] mb-[10px]"
390-
// style={{ width: "calc(100% - 20px)", margin: "0 10px 10px 10px" }}
391-
>
369+
className="w-[calc(100%-20px)] mx-[10px] mb-[10px]">
392370
{server.status === "connecting" ? "Retrying..." : "Retry Connection"}
393371
</VSCodeButton>
394372
</div>

webview-ui/src/components/settings/providers/OpenAICompatible.tsx

Lines changed: 2 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -472,19 +472,6 @@ export const OpenAICompatible = ({ apiConfiguration, setApiConfigurationField }:
472472
<VSCodeTextField
473473
value={apiConfiguration?.openAiCustomModelInfo?.cacheReadsPrice?.toString() ?? "0"}
474474
type="text"
475-
// style={{
476-
// borderColor: (() => {
477-
// const value = apiConfiguration?.openAiCustomModelInfo?.cacheReadsPrice
478-
//
479-
// if (!value && value !== 0) {
480-
// return "var(--vscode-input-border)"
481-
// }
482-
//
483-
// return value >= 0
484-
// ? "var(--vscode-charts-green)"
485-
// : "var(--vscode-errorForeground)"
486-
// })(),
487-
// }}
488475
className={cn(
489476
"w-full border", // Added base 'border' class
490477
(() => {
@@ -504,17 +491,14 @@ export const OpenAICompatible = ({ apiConfiguration, setApiConfigurationField }:
504491
cacheReadsPrice: isNaN(parsed) ? 0 : parsed,
505492
}
506493
})}
507-
placeholder={t("settings:placeholders.numbers.inputPrice")}
508-
/* className="w-full" */
509-
>
494+
placeholder={t("settings:placeholders.numbers.inputPrice")}>
510495
<div className="flex items-center gap-1">
511496
<span className="font-medium">
512497
{t("settings:providers.customModel.pricing.cacheReads.label")}
513498
</span>
514499
<i
515500
className="codicon codicon-info text-vscode-descriptionForeground text-xs"
516501
title={t("settings:providers.customModel.pricing.cacheReads.description")}
517-
// style={{ fontSize: "12px" }}
518502
/>
519503
</div>
520504
</VSCodeTextField>
@@ -523,19 +507,6 @@ export const OpenAICompatible = ({ apiConfiguration, setApiConfigurationField }:
523507
<VSCodeTextField
524508
value={apiConfiguration?.openAiCustomModelInfo?.cacheWritesPrice?.toString() ?? "0"}
525509
type="text"
526-
// style={{
527-
// borderColor: (() => {
528-
// const value = apiConfiguration?.openAiCustomModelInfo?.cacheWritesPrice
529-
//
530-
// if (!value && value !== 0) {
531-
// return "var(--vscode-input-border)"
532-
// }
533-
//
534-
// return value >= 0
535-
// ? "var(--vscode-charts-green)"
536-
// : "var(--vscode-errorForeground)"
537-
// })(),
538-
// }}
539510
onChange={handleInputChange("openAiCustomModelInfo", (e) => {
540511
const value = (e.target as HTMLInputElement).value
541512
const parsed = parseFloat(value)
@@ -547,7 +518,7 @@ export const OpenAICompatible = ({ apiConfiguration, setApiConfigurationField }:
547518
})}
548519
placeholder={t("settings:placeholders.numbers.cacheWritePrice")}
549520
className={cn(
550-
"w-full border", // Added base 'border' class
521+
"w-full border",
551522
(() => {
552523
const value = apiConfiguration?.openAiCustomModelInfo?.cacheWritesPrice
553524
if (value === undefined || value === null) return "border-vscode-input-border"
@@ -563,7 +534,6 @@ export const OpenAICompatible = ({ apiConfiguration, setApiConfigurationField }:
563534
<i
564535
className="codicon codicon-info text-vscode-descriptionForeground text-xs"
565536
title={t("settings:providers.customModel.pricing.cacheWrites.description")}
566-
// style={{ fontSize: "12px" }}
567537
/>
568538
</div>
569539
</VSCodeTextField>

0 commit comments

Comments
 (0)