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

feat: 字幕のオン・オフを記憶する #1015

Merged
merged 3 commits into from
Mar 27, 2024
Merged

feat: 字幕のオン・オフを記憶する #1015

merged 3 commits into from
Mar 27, 2024

Conversation

kou029w
Copy link

@kou029w kou029w commented Mar 11, 2024

resolved #861
fixed #860 close #1013

@ties-mitsuhashi
Copy link

macOSのSafariで,字幕をオンにすると,ビデオの画面が真っ暗になります。ご確認いただけますでしょうか。
OS: macOS 12.7.4
ブラウザ:Safari 17.4
image

@kou029w

This comment was marked as outdated.

@kou029w

This comment was marked as outdated.

@ties-makimura
Copy link

日本語と英語が必ずしも決まった順番で登録されないので,インデックス番号が最初に検索されるより,言語を最初に判定して欲しい

3ac7744#diff-1754472701fccef33600584f672f6851bf995a7c27a96e815ad8d5e01ad5e68cR98-R102

@kou029w
Copy link
Author

kou029w commented Mar 18, 2024

例えば 英語(en),日本語(ja) や 日本語(ja),英語(en) のように互い違いに並ぶことがあるということですね。
その場合は後続の 3ac7744#diff-1754472701fccef33600584f672f6851bf995a7c27a96e815ad8d5e01ad5e68cR104-R107 で処理されており、例えば「日本語(ja)」を選択すれば次も「日本語(ja)」が選択されるかと思います。

問題なのは、例えば 日本語,日本語* など同じ言語が並ぶときはどうしましょうか。
言語での判定を優先するということは、日本語,日本語* (2番目を選択) → 次回アクセスしたときや次の動画に進むと 日本語*,日本語 (1番目が選択される)、となり得るかと思います。

@ties-makimura
Copy link

ties-makimura commented Mar 18, 2024

すみません,もう少し表示される言語について確認させてください。

例えば,以下のケースの場合,トピック4からトピック3に移動すると英語が表示されるのは,トピック2で英語に切り替えたのをずっと記憶しているからでしょうか?

ケース

  • トピック1:日本語,英語
  • トピック2:英語,日本語
  • トピック3:日本語,英語
  • トピック4:日本語

再現手順

  1. トピック1を日本語字幕にする
  2. トピック2に移動すると日本語が表示される
  3. トピック2で英語に切り替える
  4. トピック4に移動すると未選択状態になる
  5. トピック3に移動すると英語が表示される

@kou029w
Copy link
Author

kou029w commented Mar 18, 2024

そうです! トピック2で英語に切り替えることで、以降は英語に切り替わります

@ties-makimura
Copy link

なるほど,判定を文章にすると以下の流れですね。

  1. 選択順・字幕表示・言語が「完全に一致する」対象があるか見てあれば採用する
  2. 完全に一致するのがなければ,同じ言語があるか判定してあれば採用する
  3. それもなければ非表示にする。ただし切り替えた情報は更新していないので,次のトピックでは,また,完全一致か同一言語チェックを行い,一致すれば採用する

すみません,混乱していました。
言語の優先変更はまだ考えなくて大丈夫です。もう少し頭整理します。

@ties-mitsuhashi
Copy link

iPhoneのSafariでは,動画を再生すると全画面になってしまい,字幕の表示(あるいは字幕表示UI)がされないのですが,iPhoneのSafariでは字幕の表示はできるでしょうか。

もし,別途対応が必要であれば,これとは別にissueを起票いたします。

それ以外の環境では,問題ありませんでした。

IMG_0020

@kou029w
Copy link
Author

kou029w commented Mar 27, 2024

確認いただきありがとうございます。
iOS環境での字幕の表示についてはSafariの動画プレイヤーの実装が異なることもあり(従来バージョンと同様)動作未サポートとなっており可能かどうか調査含め別途対応の必要あります。すみませんがご了承ください。

@ties-mitsuhashi
Copy link

承知いたしました。別途 issueを起票いたします。
マージしていただいて問題ございません。

@kou029w
Copy link
Author

kou029w commented Mar 27, 2024

では本PRマージさせていただきます!

@kou029w kou029w merged commit c8a7953 into master Mar 27, 2024
2 checks passed
@kou029w kou029w deleted the gh-861 branch March 27, 2024 02:34
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.

字幕のオン・オフを記憶する 動画に設定した字幕が増える
3 participants