Merge pull request #5171 from Tyriar/links

Fix links sometimes not activating
xtermjs · Sep 27, 2024 · 9186c72 · 9186c72
2 parents f56675f + f764c0a
commit 9186c72
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/src/browser/CoreBrowserTerminal.ts b/src/browser/CoreBrowserTerminal.ts
@@ -69,7 +69,8 @@ export class CoreBrowserTerminal extends CoreTerminal implements ITerminal {
  private _helperContainer: HTMLElement | undefined;
  private _compositionView: HTMLElement | undefined;
 
- public linkifier: ILinkifier2 | undefined;
+ private readonly _linkifier: MutableDisposable<ILinkifier2> = this._register(new MutableDisposable());
+ public get linkifier(): ILinkifier2 | undefined { return this._linkifier.value; }
  private _overviewRulerRenderer: OverviewRulerRenderer | undefined;
  private _viewport: Viewport | undefined;
 
@@ -485,7 +486,7 @@ export class CoreBrowserTerminal extends CoreTerminal implements ITerminal {
  this._mouseService = this._instantiationService.createInstance(MouseService);
  this._instantiationService.setService(IMouseService, this._mouseService);
 
- this.linkifier = this._register(this._instantiationService.createInstance(Linkifier, this.screenElement));
+ const linkifier = this._linkifier.value = this._register(this._instantiationService.createInstance(Linkifier, this.screenElement));
 
  // Performance: Add viewport and helper elements from the fragment
  this.element.appendChild(fragment);
@@ -515,7 +516,7 @@ export class CoreBrowserTerminal extends CoreTerminal implements ITerminal {
  this._selectionService = this._register(this._instantiationService.createInstance(SelectionService,
  this.element,
  this.screenElement,
- this.linkifier
+ linkifier
  ));
  this._instantiationService.setService(ISelectionService, this._selectionService);
  this._register(this._selectionService.onRequestScrollLines(e => this.scrollLines(e.amount, e.suppressScrollEvent)));

diff --git a/src/browser/Linkifier.ts b/src/browser/Linkifier.ts
@@ -227,7 +227,7 @@ export class Linkifier extends Disposable implements ILinkifier2 {
  return;
  }
 
- if (this._mouseDownLink === this._currentLink && this._linkAtPosition(this._currentLink.link, position)) {
+ if (this._mouseDownLink && linkEquals(this._mouseDownLink.link, this._currentLink.link) && this._linkAtPosition(this._currentLink.link, position)) {
  this._currentLink.link.activate(event, this._currentLink.link.text);
  }
  }
@@ -391,3 +391,13 @@ export class Linkifier extends Disposable implements ILinkifier2 {
  return { x1, y1, x2, y2, cols: this._bufferService.cols, fg };
  }
 }
+
+function linkEquals(a: ILink, b: ILink): boolean {
+ return (
+ a.text === b.text &&
+ a.range.start.x === b.range.start.x &&
+ a.range.start.y === b.range.start.y &&
+ a.range.end.x === b.range.end.x &&
+ a.range.end.y === b.range.end.y
+ );
+}