fix: deck uiの通知音が重なる問題

[anatawa12]

What

Why

Additional info (optional)

Checklist

• [x] Read the contribution guide
• [ ] Test working in a local environment
• [ ] (If needed) Add story of storybook
• [x] (If needed) Update CHANGELOG.md
• [ ] (If possible) Add tests

[codecov[bot]]

Codecov Report

Attention: Patch coverage is 37.50000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 20.50%. Comparing base (8959ff8) to head (0478cd8).

Files	Patch %	Lines
packages/frontend/src/scripts/sound.ts	37.50%	10 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## develop #14029 +/- ## ============================================ - Coverage 40.18% 20.50% -19.69% ============================================ Files 1524 702 -822 Lines 188773 98746 -90027 Branches 3516 1027 -2489 ============================================ - Hits 75856 20245 -55611 + Misses 112345 77980 -34365 + Partials 572 521 -51 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[Sayamame-beans]

Conflict解消

[syuilo]

「通知音が重なる」が具体的にどういう条件で発生するのかとかの情報が欲しいわね

[Sayamame-beans]

現象としては前回のものと同じだったはずで、その際は #12433, #13379 で修正されています。 今回は #13867 で追加した範囲にて、新たに「ページを開いてから一度も操作していなかったら音が鳴らず、初めて操作したタイミングで全部鳴る」ような状態が発生していたような?

重複防止のない再生の仕組みを呼び出してた。ありがとう。

https://misskey.niri.la/notes/9u3smehs5s

このPRでは、12433の方で追加された重複回避をplayMisskeySfxFile(13867で使用している)の方にも含めるようにしている感じですかね? (合ってます?)

[anatawa12]

さやまめさんの説明で正しいです

[Sayamame-beans]

(13379の方の対処は含んでなくて問題ないのかは少し気になってます(未検証))

[anatawa12]

!navigator.userActivation.hasBeenActiveも関係あるのか。入れたほうが良さそう

[anatawa12]

JSDOCの説明がplayMisskeySfxInternalのほうについてるのでそれの移動も必要

0478cd8f17b92aeb5be6eb99c497f3dc32aeb730 でやりました

[syuilo]

🙏🏻