xpadev-net / niconicomments

ニコニコ動画の公式プレイヤー互換の高パフォーマンスなコメント描画ライブラリ / High peformance High compatibility comment drawing library

https://xpadev-net.github.io/niconicomments/

MIT License

74 stars 5 forks source link

Closed rinsuki closed 8 months ago

rinsuki commented 8 months ago

概要

とりあえず this.timeline の参照だけ Math.floor したものを使って、それ以外は非整数な vpos を使ったパッチを用意しました (これで何も出ないということはなくなる) https://github.com/rinsuki/niconicomments/commit/97709919071067fa683e103d89f3731c67c4fb24

一見ちゃんと動いているように見えますが、適当に一本の動画で試しているだけなので、プラグインとかCA依存で壊れるかもしれません (内部構造に詳しくないのでこれでいいかわからない)

xpadev-net commented 8 months ago

こちらもありがとうございます！

非整数に対応するという意味であれば drawCanvas で受け取った vpos をすべてMath.floorすれば良いのではと思うのですが、vposFloat を分離したのはコメントをなめらかに動かしたいなどの意図でしょうか?

もしそうであれば main.ts の _drawCollision だけ vposInt に差し替えてPR投げていただけるとありがたいです🙇‍♂️

rinsuki commented 8 months ago

vposFloat を分離したのはコメントをなめらかに動かしたいなどの意図でしょうか?

そうです！ (issueの本文に書くのを忘れていました…… :bow:)

main.ts の _drawCollision だけ vposInt に差し替えてPR投げていただけるとありがたいです🙇‍♂️

わかりました！