Xterm.js: Los comparadores de enlaces no funcionan con enlaces de varias líneas

¿Alguien está investigando este problema? Es un asesino para el terminal integrado de VS Code.

+1

Hola @martellaj , en este momento esto no está asignado a nadie, por lo que nadie está investigando esto.

Lo solucionaremos tan pronto como encontremos la cantidad de tiempo necesaria.

Y, por supuesto, los RP siempre son bienvenidos 😁.

Creo que ya se han cumplido todos los requisitos previos para esto:

• El búfer sabe cuándo se envuelven las líneas (al menos en Linux / macOS)
• Existe un método relativamente rápido para tomar toda la línea como una cadena
• Los enlaces ya no necesitan hacer malabarismos con el DOM, por lo que será posible obtener líneas de varias líneas con ambas partes reaccionando al pasar el cursor

No podré retomar esto pronto. Esto es lo que se debe hacer para los interesados:

• Consulta isWrapped para identificar las líneas que se ajustan al
• Modifique MouseZone y MouseZoneManager para admitir el ajuste. Probablemente, esto se haga mejor reemplazando MouseZone.y con y1 y y2 .

También queremos asegurarnos de que los enlaces funcionen cuando los datos desaparecen de la pantalla a la derecha. Esto debe abordarse al mismo tiempo que los enlaces de varias líneas

Me gustaría intentarlo. He hecho un poco de js durante los últimos 2 años y espero obtener mi primera solicitud de extracción. Puedo retomar esto el próximo fin de semana si eso está bien para todos 😊

@chirgjn por MouseZone que pueden abarcar varias líneas, y luego verificar si las líneas están marcadas con isWrapped y si es así, verifique las líneas en toda la línea envuelta y cree el MouseZone apropiado. Esto implicará principalmente trabajar en Linkifier.ts y MouseZoneManager.ts

Gracias, recoger esto este fin de semana 😃

@chirgjn ¿ algún progreso en esto?

@bastichelaar es una de las pocas cosas a las que quiero llegar si no entra un PR.

Oye, perdón por la respuesta tardía. Prepararé un PR hoy.

El martes 27 de febrero de 2018 a las 8:59 p.m., Daniel Imms, [email protected] escribió:

@bastichelaar https://github.com/bastichelaar es uno de los pocos
de las cosas a las que quiero llegar a continuación si un PR no entra.

-
Estás recibiendo esto porque te mencionaron.
Responda a este correo electrónico directamente, véalo en GitHub
https://github.com/xtermjs/xterm.js/issues/24#issuecomment-368916869 ,
o silenciar el hilo
https://github.com/notifications/unsubscribe-auth/ADeQIqmu9RJPm29PJHZe0XU2trHAV8WWks5tZB9wgaJpZM4EKjoM
.

@chirgjn FYI, comencé a trabajar en eso y planeo terminarlo probablemente en la próxima semana o 2 https://github.com/xtermjs/xterm.js/pull/1303

@Tyriar, gracias por el trabajo en ello,