Xterm.js: Link Matcher funktionieren nicht mit mehrzeiligen Links

Gibt es jemanden, der sich mit diesem Thema befasst? Es ist ein Killer für das integrierte Terminal von VS Code.

+1

Hallo @martellaj , im

Wir werden uns darum kümmern, sobald wir die erforderliche Zeit gefunden haben.

Und PRs sind natürlich immer willkommen 😁.

Ich glaube, dass jetzt alle Voraussetzungen dafür erfüllt sind:

• Der Puffer weiß, wann Zeilen umbrochen werden (zumindest unter Linux / MacOS).
• Es gibt eine relativ schnelle Methode, um die gesamte Zeile als Zeichenfolge zu erfassen
• Links müssen nicht mehr mit dem DOM jonglieren, sodass es möglich ist, mehrzeilige Linien zu erhalten, bei denen beide Teile beim Schweben reagieren

Ich werde das bald nicht mehr abholen können. Folgendes muss für Interessierte getan werden:

• Fragen Sie isWrapped , um Zeilen zu identifizieren, die beim Verknüpfen von Zeilen umbrochen werden.
• Ändern Sie MouseZone und MouseZoneManager , um das Umbrechen zu unterstützen. Dies geschieht wahrscheinlich am besten, indem Sie MouseZone.y durch y1 und y2 ersetzen.

Wir möchten auch sicherstellen, dass Links funktionieren, wenn die Daten rechts vom Bildschirm verschwinden. Dies sollte gleichzeitig mit mehrzeiligen Verbindungen angegangen werden

Ich würde gerne einen Stich machen. Ich habe in den letzten 2 Jahren einiges an js gemacht und freue mich auf meine erste Pull-Anfrage. Ich kann das am kommenden Wochenende abholen, wenn das für alle in Ordnung ist 😊

@chirgjn auf jeden Fall. Als Hinweis darauf, wo Sie anfangen sollen, müssen Sie die Unterstützungsbereiche von MouseZone , die sich über mehrere Zeilen erstrecken können, und dann prüfen, ob Zeilen mit isWrapped markiert sind, und wenn ja, prüfen Sie, ob Zeilen aktiviert sind die gesamte umbrochene Zeile und erstellen Sie die entsprechenden MouseZone . Dies beinhaltet hauptsächlich die Arbeit in Linkifier.ts und MouseZoneManager.ts

Danke, ich habe das dieses Wochenende abgeholt 😃

@chirgjn irgendwelche Fortschritte in diesem Bereich?

@ Bastichelaar Es ist eines der wenigen Dinge, die ich als nächstes erreichen möchte, wenn keine PR

Hey, entschuldige die späte Antwort. Ich werde heute eine PR fertig machen.

Am Dienstag, den 27. Februar 2018 um 20:59 Uhr schrieb Daniel Imms, [email protected] :

@bastichelaar https://github.com/bastichelaar es ist eine der wenigen
von Dingen, die ich als nächstes erreichen möchte, wenn keine PR hereinkommt.

- -
Sie erhalten dies, weil Sie erwähnt wurden.
Antworte direkt auf diese E-Mail und sieh sie dir auf GitHub an
https://github.com/xtermjs/xterm.js/issues/24#issuecomment-368916869 ,
oder schalten Sie den Thread stumm
https://github.com/notifications/unsubscribe-auth/ADeQIqmu9RJPm29PJHZe0XU2trHAV8WWks5tZB9wgaJpZM4EKjoM
.

@chirgjn FYI Ich habe begonnen, daran zu arbeiten und plane, es wahrscheinlich in der nächsten Woche oder in 2 https://github.com/xtermjs/xterm.js/pull/1303 fertigzustellen

@ Tyriar, danke für die Arbeit daran. Ich werde deine Änderungen überprüfen. Vielleicht kann ich beim nächsten Mal helfen! 👍