waveterm/xterm.js
0
0
Fork 0
mirror of https://github.com/wavetermdev/xterm.js.git synced 2026-08-05 13:43:48 -07:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2912 from xtermjs/tyriar/2911

Browse Source 
Workaround for hover firing multiple times
This commit is contained in:
Daniel Imms
2020-05-06 04:59:33 -07:00
committed by GitHub
parent 7ab0a42b24 f886698dc3
commit f9adf8ad83
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/browser/Linkifier2.ts
+10 -3
View File
@@ -132,9 +132,16 @@ export class Linkifier2 extends Disposable implements ILinkifier2 {
// There is no link cached, so ask for one
this._linkProviders.forEach((linkProvider, i) => {
const existingReply = this._activeProviderReplies?.get(i);
if (existingReply) {
linkProvided = this._checkLinkProviderResult(i, position, linkProvided);
if (useLineCache) {
const existingReply = this._activeProviderReplies?.get(i);
// If there isn't a reply, the provider hasn't responded yet.
// TODO: If there isn't a reply yet it means that the provider is still resolving. Ensuring
// provideLinks isn't triggered again saves ILink.hover firing twice though. This probably
// needs promises to get fixed
if (existingReply) {
linkProvided = this._checkLinkProviderResult(i, position, linkProvided);
}
} else {
linkProvider.provideLinks(position.y, (links: ILink[] | undefined) => {
if (this._isMouseOut) {