Merge pull request microsoft#151185 from microsoft/tyriar/150842

Fix releasing alt sometimes not hiding locked hovers

Author: Tyriar

src/vs/workbench/services/hover/browser/hoverService.ts

@@ -38,7 +38,11 @@ export class HoverService implements IHoverService {
 		const hoverDisposables = new DisposableStore();
 		const hover = this._instantiationService.createInstance(HoverWidget, options);
 		hover.onDispose(() => {
-			this._currentHoverOptions = undefined;
+			// Only clear the current options if it's the current hover, the current options help
+			// reduce flickering when the same hover is shown multiple times
+			if (this._currentHoverOptions === options) {
+				this._currentHoverOptions = undefined;
+			}
 			hoverDisposables.dispose();
 		});
 		const provider = this._contextViewService as IContextViewProvider;

src/vs/workbench/services/hover/browser/hoverWidget.ts

@@ -507,7 +507,7 @@ export class HoverWidget extends Widget {
 }
 
 class CompositeMouseTracker extends Widget {
-	private _isMouseIn: boolean = false;
+	private _isMouseIn: boolean = true;
 	private _mouseTimeout: number | undefined;
 
 	private readonly _onMouseOut = this._register(new Emitter<void>());