Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix #97 #98 #99

Merged
merged 1 commit into from
Oct 20, 2024
Merged

fix #97 #98 #99

merged 1 commit into from
Oct 20, 2024

Conversation

hamachi25
Copy link
Contributor

@hamachi25 hamachi25 commented Oct 20, 2024

workIdを比較することで、エピソード移動後にon_partId_changeが2回実行されるのを修正しました。

これにより前のエピソードでコメント表示していない場合でも、setWorkInfo()を実行しウィンドウタイトルを更新するようになりました。 #97

また「連続再生時に自動で次の動画のコメントを読み込む」が、自動で動画検索がオフの時にも実行されるようになりました。#98

@sopisoft
Copy link
Owner

PR ありがとうございます!

エピソード移動後にon_partId_changeが2回実行される

いつか直そうと思ってはいたのですが、長い間放置してしまっていました。修正ありがとうございます!
周辺コードが非常に読みづらいので、今後リファクタリングしようと思います。

LGTMです !!

@sopisoft sopisoft merged commit 760f72c into sopisoft:main Oct 20, 2024
2 checks passed
@hamachi25 hamachi25 deleted the #97_#98 branch October 20, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants