Xterm.js: Os matchers de link não funcionam com links multilinhas

Alguém está investigando esse problema? É um assassino para o terminal integrado do VS Code.

+1

Olá @martellaj , neste momento não está atribuído a ninguém, portanto, ninguém está investigando isso.

Cuidaremos disso, assim que encontrarmos o tempo necessário.

E, claro, os RP são sempre bem-vindos 😁.

Acredito que todos os pré-requisitos para isso já foram cumpridos:

• O buffer sabe quando as linhas são quebradas (pelo menos no Linux / macOS)
• Existe um método relativamente rápido para pegar a linha inteira como uma string
• Os links não precisam mais fazer malabarismos com o DOM, então será possível obter linhas multilinhas com ambas as partes reagindo ao pairar

Não vou conseguir pegar isso tão cedo. Aqui está o que precisa ser feito para os interessados:

• Consulte isWrapped para identificar as linhas que são quebradas durante a
• Modifique MouseZone e MouseZoneManager para suportar o empacotamento. Provavelmente, é melhor fazer isso substituindo MouseZone.y por y1 e y2 .

Também queremos ter certeza de que os links funcionam quando os dados saem da tela para a direita. Isso deve ser abordado ao mesmo tempo que os links multilinhas

Eu gostaria de tentar fazer isso. Tenho feito um bom js nos últimos 2 anos e estou ansioso para receber meu primeiro pull request. Posso pegar no próximo fim de semana, se estiver bom para todos 😊

@chirgjn por todos os meios. Como uma dica sobre por onde começar, você precisará ter os intervalos de suporte de MouseZone que podem abranger várias linhas e, em seguida, verificar se as linhas estão marcadas com isWrapped e se sim, verifique se há linhas em a linha quebrada inteira e crie o MouseZone apropriado. Isso envolverá principalmente trabalhar em Linkifier.ts e MouseZoneManager.ts

Obrigado, pegando isso neste fim de semana 😃

@chirgjn algum progresso nisso?

@bastichelaar é uma das poucas coisas que quero

Ei, desculpe pela resposta tardia. Vou preparar uma RP hoje.

Na terça-feira, 27 de fevereiro de 2018, às 20h59, Daniel Imms, [email protected] escreveu:

@bastichelaar https://github.com/bastichelaar é um dos poucos
das coisas que quero abordar a seguir, se não houver um RP.

-
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/xtermjs/xterm.js/issues/24#issuecomment-368916869 ,
ou silenciar o tópico
https://github.com/notifications/unsubscribe-auth/ADeQIqmu9RJPm29PJHZe0XU2trHAV8WWks5tZB9wgaJpZM4EKjoM
.

@chirgjn Para https://github.com/xtermjs/xterm.js/pull/1303

@Tyriar obrigado pelo trabalho nisso, vou verificar suas mudanças, talvez eu possa ajudar na próxima vez! 👍