xtermjs/xterm.js

Issue in Linkifier2._removeIntersectingLinks

Ouverte

#4 296 ouverte le 8 déc. 2022

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)TypeScript (1 574 forks)batch import
area/linkshelp wantedtype/bug

Métriques du dépôt

Stars
 (16 196 étoiles)
Métriques de merge PR
 (Merge moyen 2j 10h) (16 PRs mergées en 30 j)

Description

Coming from https://github.com/xtermjs/xterm.js/pull/4288.

There is an issue with the private method Linkifier2._removeIntersectingLinks not correctly respecting multiple wrapped links in one merged line, which leads to wrong removal of later links.

On a first look it seems the method does not account y-progression, thus later links are mistaken as overlapping from their column position only.

Guide contributeur

Direction de recherche
Étudiez la méthode ` removeIntersectingLinks` pour comprendre comment elle traite la progression y et les liens enveloppés; identifiez pourquoi les liens ultérieurs sont supprimés incorrectement.
Stack technique
typescript
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60