Refactor: Update padding and margin styles for consistent layout in AutoApproveMenu component

Author: Githubguy132010

src/core/config/__tests__/importExport.test.ts

@@ -227,7 +227,7 @@ describe("importExport", () => {
 
 			expect(result).toEqual({
 				success: false,
-				error: "Expected property name or '}' in JSON at position 2",
+				error: "Expected property name or '}' in JSON at position 2 (line 1 column 3)",
 			})
 			expect(fs.readFile).toHaveBeenCalledWith("/mock/path/settings.json", "utf-8")
 			expect(mockProviderSettingsManager.import).not.toHaveBeenCalled()

src/integrations/terminal/__tests__/ExecaTerminal.spec.ts

@@ -22,7 +22,7 @@ describe("ExecaTerminal", () => {
 			onShellExecutionComplete: vi.fn(),
 		}
 
-		const subprocess = terminal.runCommand("ls -al", callbacks)
+		const subprocess = terminal.runCommand("LANG=C ls -al", callbacks)
 		await subprocess
 
 		expect(callbacks.onLine).toHaveBeenCalled()

webview-ui/eslint.config.mjs

@@ -18,6 +18,15 @@ export default [
 			"@typescript-eslint/no-explicit-any": "off",
 			"react/prop-types": "off",
 			"react/display-name": "off",
+			"@typescript-eslint/no-unused-expressions": [
+				"error",
+				{
+					"allowShortCircuit": true,
+					"allowTernary": true,
+					"allowTaggedTemplates": true,
+					"enforceForJSX": false
+				}
+			],
 		},
 	},
 	{

webview-ui/package.json

@@ -3,7 +3,7 @@
 	"private": true,
 	"type": "module",
 	"scripts": {
-		"lint": "eslint src --ext=ts,tsx --max-warnings=0",
+		"lint": "eslint src --max-warnings=0",
 		"check-types": "tsc",
 		"test": "jest -w=40%",
 		"format": "prettier --write src",

webview-ui/src/components/chat/AutoApproveMenu.tsx

@@ -134,7 +134,7 @@ const AutoApproveMenu = ({ style }: AutoApproveMenuProps) => {
 					display: "flex",
 					alignItems: "center",
 					gap: "8px",
-					padding: isExpanded ? "8px 0" : "8px 0 0 0",
+					padding: "8px 0 0 0", // Ensure consistent padding for the toggle header
 					cursor: "pointer",
 				}}
 				onClick={toggleExpanded}>
@@ -207,8 +207,8 @@ const AutoApproveMenu = ({ style }: AutoApproveMenuProps) => {
 							display: "flex",
 							alignItems: "center",
 							gap: "8px",
-							marginTop: "10px",
-							marginBottom: "8px",
+							// marginTop: "10px", // Removed to use parent's gap for spacing
+							// marginBottom: "8px", // Removed to use parent's gap for spacing
 							color: "var(--vscode-descriptionForeground)",
 						}}>
 						<span style={{ flexShrink: 1, minWidth: 0 }}>