feat: remove IRenderer dependencies to DOM - Githubissues

xpadev-net / niconicomments

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

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

MIT License

74 stars 5 forks source link

feat: remove IRenderer dependencies to DOM #96

Closed rinsuki closed 8 months ago

rinsuki commented 8 months ago

チケットへのリンク

なし (ごめんなさい…)

やったこと

IRenderer から DOM に依存したプロパティを削除しました

やらないこと

型の厳格化
- 本来は IRenderer.drawImage の image は this にするべきですが、そうなると IRenderer が関わる全ファイルにジェネリクスを入れる改修が必要になる & そもそも実現可能か怪しい (継承周りの都合) ので、一旦実行時にハネるだけで許容することにしました (複数 renderer が実行時に並列する use-case が思い付かなかったので)

できるようになること（ユーザ目線）

理論上は DOM に依存していないレンダラを実装することが可能になりました

できなくなること（ユーザ目線）

なし

動作確認

pnpm run check-types でエラーが出ないことを確認

その他

レビュワーへの参考情報（実装上の懸念点や注意点などあれば記載）

xpadev-net commented 8 months ago

PRありがとうございます 🙇‍♂️ LGTMです!

チケットへのリンクなし (ごめんなさい…)

PRのテンプレはどっかから引っ張ってきたやつなのでお気になさらず...