この問題を調べている人はいますか? これは、VSCodeの統合端末のキラーです。
+1
こんにちは@martellaj 、今これは誰にも割り当てられていないので、誰もこれを調べていません。
必要な時間が見つかり次第、これを乗り越えていきます。
そしてもちろん、PRはいつでも大歓迎です😁。
これに関するすべての前提条件が満たされていると思います。
私はこれをすぐに拾うことができなくなります。 興味のある人のために行う必要があることは次のとおりです。
isWrapped
をクエリして、行をリンクするときにます。MouseZone
とMouseZoneManager
を変更します。 これはおそらく、 MouseZone.y
をy1
とy2
置き換えることによって最もよく行われます。また、データが画面の右側に表示されなくなったときにリンクが機能することを確認する必要があります。 これは、複数行のリンクと同時に取り組む必要があります
これを突き刺したいです。 私は過去2年間、かなりのjsを実行しており、最初のプルリクエストを受け取るのを楽しみにしています。 みんなで大丈夫なら、来週末にこれを拾うことができます😊
ぜひ@chirgjn 。 どこから始めるかについてのヒントとして、複数行にまたがることができるMouseZone
のサポート範囲を用意し、行がisWrapped
マークされているかどうかを確認し、マークされている場合は上の行を確認する必要があります。折り返し行全体を作成し、適切なMouseZone
を作成します。 これには主にLinkifier.tsとMouseZoneManager.tsでの作業が含まれます
ありがとう、今週末これを拾います😃
@chirgjnこれに関する進展はありますか?
@bastichelaarこれは、PRが入らない場合に次に行きたい数少ないことの1つです。
ねえ、返信が遅れてすみません。 今日はPRの準備をします。
2018年2月27日火曜日、午後8時59分、Daniel Imms、 notifications @ github.comは次のように書いています。
@bastichelaarhttps ://github.com/bastichelaarそれは一握りの1つです
PRが入らなかったら次に行きたいことの—
あなたが言及されたのであなたはこれを受け取っています。
このメールに直接返信し、GitHubで表示してください
https://github.com/xtermjs/xterm.js/issues/24#issuecomment-368916869 、
またはスレッドをミュートします
https://github.com/notifications/unsubscribe-auth/ADeQIqmu9RJPm29PJHZe0XU2trHAV8WWks5tZB9wgaJpZM4EKjoM
。
@chirgjn参考までに作業を開始し、おそらく来週か2週間で終了する予定ですhttps://github.com/xtermjs/xterm.js/pull/1303
@Tyriarはその作業に感謝します、私はあなたの変更をチェックします、多分私は次回助けることができるでしょう! 👍
最も参考になるコメント
これを突き刺したいです。 私は過去2年間、かなりのjsを実行しており、最初のプルリクエストを受け取るのを楽しみにしています。 みんなで大丈夫なら、来週末にこれを拾うことができます😊