Show terminal link popup for custom link matchers

Fixes #23149
microsoft · Mar 24, 2017 · ba87991 · ba87991
1 parent 5134a2c
commit ba87991
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/src/vs/workbench/parts/terminal/electron-browser/terminalLinkHandler.ts b/src/vs/workbench/parts/terminal/electron-browser/terminalLinkHandler.ts
@@ -52,9 +52,17 @@ export class TerminalLinkHandler {
 	}
 
 	public registerCustomLinkHandler(regex: RegExp, handler: (uri: string) => void, matchIndex?: number, validationCallback?: XtermLinkMatcherValidationCallback): number {
+		const wrappedValidationCallback = (uri, element, callback) => {
+			this._addTooltipEventListeners(element);
+			if (validationCallback) {
+				validationCallback(uri, element, callback);
+			} else {
+				callback(true);
+			}
+		};
 		return this._xterm.registerLinkMatcher(regex, this._wrapLinkHandler(handler), {
 			matchIndex,
-			validationCallback,
+			validationCallback: wrappedValidationCallback,
 			priority: CUSTOM_LINK_PRIORITY
 		});
 	}