move comment, add one

Author: meganrogge

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

@@ -145,6 +145,7 @@ export interface IHoverOptions {
 	 * Whether to trap focus in the following ways:
 	 * - When the hover closes, focus goes to the element that had focus before the hover opened
 	 * - If there are elements in the hover to focus, focus stays inside of the hover when tabbing
+	 * Note that this is overridden to true when in screen reader optimized mode.
 	 */
 	trapFocus?: boolean;
 

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

@@ -44,8 +44,8 @@ export class HoverService implements IHoverService {
 		}
 		this._currentHoverOptions = options;
 		this._lastHoverOptions = options;
-		// HACK, remove when #189076 is fixed
 		const trapFocus = options.trapFocus || this._accessibilityService.isScreenReaderOptimized();
+		// HACK, remove this check when #189076 is fixed
 		if (!skipLastFocusedUpdate) {
 			if (trapFocus && document.activeElement) {
 				this._lastFocusedElementBeforeOpen = document.activeElement as HTMLElement;