Revert "Always focus the panel when clicked to ensure menu buttons are visible" (#4592)

Revert "Always focus the panel when clicked to ensure menu buttons are visibl…"

This reverts commit 1b1e5a2.

Author: mrubens

.changeset/khaki-clocks-float.md

@@ -173,7 +173,7 @@ export class TelemetryService {
 			itemType,
 			itemName,
 			target,
-			...(properties || {}),
+			... (properties || {}),
 		})
 	}
 

packages/telemetry/src/TelemetryService.ts

@@ -51,7 +51,6 @@ export const commandIds = [
 
 	"focusInput",
 	"acceptInput",
-	"focusPanel",
 ] as const
 
 export type CommandId = (typeof commandIds)[number]

packages/types/src/vscode.ts

@@ -8,7 +8,6 @@ import { Package } from "../shared/package"
 import { getCommand } from "../utils/commands"
 import { ClineProvider } from "../core/webview/ClineProvider"
 import { ContextProxy } from "../core/config/ContextProxy"
-import { focusPanel } from "../utils/focusPanel"
 
 import { registerHumanRelayCallback, unregisterHumanRelayCallback, handleHumanRelayResponse } from "./humanRelay"
 import { handleNewTask } from "./handleTask"
@@ -173,23 +172,20 @@ const getCommandsMap = ({ context, outputChannel, provider }: RegisterCommandOpt
 	},
 	focusInput: async () => {
 		try {
-			await focusPanel(tabPanel, sidebarPanel)
-
-			// Send focus input message only for sidebar panels
-			if (sidebarPanel && getPanel() === sidebarPanel) {
+			const panel = getPanel()
+
+			if (!panel) {
+				await vscode.commands.executeCommand(`workbench.view.extension.${Package.name}-ActivityBar`)
+			} else if (panel === tabPanel) {
+				panel.reveal(vscode.ViewColumn.Active, false)
+			} else if (panel === sidebarPanel) {
+				await vscode.commands.executeCommand(`${ClineProvider.sideBarId}.focus`)
 				provider.postMessageToWebview({ type: "action", action: "focusInput" })
 			}
 		} catch (error) {
 			outputChannel.appendLine(`Error focusing input: ${error}`)
 		}
 	},
-	focusPanel: async () => {
-		try {
-			await focusPanel(tabPanel, sidebarPanel)
-		} catch (error) {
-			outputChannel.appendLine(`Error focusing panel: ${error}`)
-		}
-	},
 	acceptInput: () => {
 		const visibleProvider = getVisibleProviderOrLog(outputChannel)
 

src/activate/registerCommands.ts

@@ -1475,11 +1475,6 @@ export const webviewMessageHandler = async (
 			}
 			break
 		}
-		case "focusPanelRequest": {
-			// Execute the focusPanel command to focus the WebView
-			await vscode.commands.executeCommand(getCommand("focusPanel"))
-			break
-		}
 		case "filterMarketplaceItems": {
 			// Check if marketplace is enabled before making API calls
 			const { experiments } = await provider.getState()

src/core/webview/webviewMessageHandler.ts

@@ -150,7 +150,6 @@ export interface WebviewMessage {
 		| "clearIndexData"
 		| "indexingStatusUpdate"
 		| "indexCleared"
-		| "focusPanelRequest"
 		| "codebaseIndexConfig"
 		| "setHistoryPreviewCollapsed"
 		| "openExternal"

src/shared/WebviewMessage.ts

@@ -18,7 +18,6 @@ import { MarketplaceView } from "./components/marketplace/MarketplaceView"
 import ModesView from "./components/modes/ModesView"
 import { HumanRelayDialog } from "./components/human-relay/HumanRelayDialog"
 import { AccountView } from "./components/account/AccountView"
-import { useAddNonInteractiveClickListener } from "./components/ui/hooks/useNonInteractiveClick"
 
 type Tab = "settings" | "history" | "mcp" | "modes" | "chat" | "marketplace" | "account"
 
@@ -136,13 +135,6 @@ const App = () => {
 	// Tell the extension that we are ready to receive messages.
 	useEffect(() => vscode.postMessage({ type: "webviewDidLaunch" }), [])
 
-	// Focus the WebView when non-interactive content is clicked
-	useAddNonInteractiveClickListener(
-		useCallback(() => {
-			vscode.postMessage({ type: "focusPanelRequest" })
-		}, []),
-	)
-
 	if (!didHydrateState) {
 		return null
 	}

src/utils/focusPanel.ts

@@ -1,3 +1,2 @@
 export * from "./useClipboard"
 export * from "./useRooPortal"
-export * from "./useNonInteractiveClick"