Skip to content

Conversation

@sevenc-nanashi
Copy link
Member

内容

タイトル通りです。
将来のブラウザ版対応や、開発時にブラウザを使って軽量化みたいなことをしてる人に役立ちます。

関連 Issue

(なし)

スクリーンショット・動画など

(なし)

その他

(なし)

@sevenc-nanashi sevenc-nanashi requested a review from a team as a code owner January 29, 2024 12:13
@sevenc-nanashi sevenc-nanashi requested review from y-chan and removed request for a team January 29, 2024 12:13
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

別ブラウザで動くかはベストエフォートでよくて、まあ最悪動かなくてもスルーで良いかなと思ってます。
キャンセルできないと不都合あるかもですが、まあ良いのかなーと。

でも場合によってはワークアラウンド的な実装になりがちで、中途半端な実装になってしまうのも事実です。
お時間ある時で大丈夫ですので @sigprogramming さん的にどう思われるか伺いたいです!🙏
(ほんとに時間ある時でOKだと思います)

@sevenc-nanashi sevenc-nanashi changed the title Fix: Firefoxだと音が止まらないのを修正 [ソング] Fix: Firefoxだと音が止まらないのを修正 Feb 2, 2024
@sigprogramming
Copy link
Collaborator

sigprogramming commented Feb 5, 2024

cancelAndHoldAtTime?.()で呼び出すように変更すると、ノートオフのタイミングによっては音量変化が不連続になるかもしれませんが、ひとまず動くようにはなるので良いかなと思います!

ただ、Chrome(Chromium)以外のブラウザで未実装・挙動が違う機能は他にもいくつかあり(audioContext.setSinkId()など)、クロスブラウザ対応をどこまで行うか(フォールバックの実装など)は要議論かもと思いました。

@Hiroshiba
Copy link
Member

@sigprogramming ありがとうございます!!
議論が確かに必要かなと思ったのでissue作ってみました!

@Hiroshiba
Copy link
Member

(テストが回らないのでちょっとdraftにして戻してみます)

@Hiroshiba Hiroshiba marked this pull request as draft February 6, 2024 01:27
@Hiroshiba Hiroshiba marked this pull request as ready for review February 6, 2024 01:27
@Hiroshiba
Copy link
Member

テストがなぜか走りませんが、大丈夫だと思うのでマージします!

@Hiroshiba Hiroshiba merged commit 54e8b72 into VOICEVOX:main Feb 6, 2024
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