PR feedback

Author: cte

webview-ui/src/components/ui/__tests__/tooltip.spec.tsx

@@ -80,31 +80,6 @@ describe("Tooltip", () => {
 			{ timeout: 1000 },
 		)
 	})
-
-	it("should apply VSCode native shadow styling", async () => {
-		const user = userEvent.setup()
-
-		render(
-			<TooltipProvider delayDuration={0}>
-				<Tooltip>
-					<TooltipTrigger>Hover me</TooltipTrigger>
-					<TooltipContent>Tooltip with shadow</TooltipContent>
-				</Tooltip>
-			</TooltipProvider>,
-		)
-
-		const trigger = screen.getByText("Hover me")
-		await user.hover(trigger)
-
-		await waitFor(
-			() => {
-				const tooltips = screen.getAllByText("Tooltip with shadow")
-				const visibleTooltip = tooltips.find((el) => el.getAttribute("role") !== "tooltip")
-				expect(visibleTooltip).toHaveClass("shadow-[0_2px_8px_var(--color-vscode-widget-shadow)]")
-			},
-			{ timeout: 1000 },
-		)
-	})
 })
 
 describe("StandardTooltip", () => {

webview-ui/src/index.css

@@ -131,6 +131,7 @@
 	--color-vscode-inputValidation-infoBorder: var(--vscode-inputValidation-infoBorder);
 
 	--color-vscode-widget-border: var(--vscode-widget-border);
+	--color-vscode-widget-shadow: var(--vscode-widget-shadow);
 
 	--color-vscode-textLink-foreground: var(--vscode-textLink-foreground);
 
@@ -139,12 +140,6 @@
 	--color-vscode-editorHoverWidget-foreground: var(--vscode-editorHoverWidget-foreground);
 	--color-vscode-editorHoverWidget-background: var(--vscode-editorHoverWidget-background);
 	--color-vscode-editorHoverWidget-border: var(--vscode-editorHoverWidget-border);
-
-	/**
-	 * VSCode tooltip shadow - matches native VSCode tooltip styling
-	 * Uses a subtle shadow for better visibility and depth
-	 */
-	--color-vscode-widget-shadow: var(--vscode-widget-shadow, rgba(0, 0, 0, 0.36));
 }
 
 @layer base {