misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix(frontend): 画面表示後最初の音声再生が爆音になることがある問題を修正

Open zyoshoka opened this issue 1 year ago • 1 comments

What

画面表示後最初の音声再生が爆音になることがある問題を修正します。

Why

Closes #12532

Additional info (optional)

Checklist

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

zyoshoka avatar Feb 18 '24 10:02 zyoshoka

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (39c4e3a) 77.51% compared to head (985d39b) 77.46%.

:exclamation: Current head 985d39b differs from pull request most recent head 2d77ccb. Consider uploading reports for the commit 2d77ccb to get more accurate results

Files Patch % Lines
packages/frontend/src/scripts/sound.ts 0.00% 1 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13379      +/-   ##
===========================================
- Coverage    77.51%   77.46%   -0.06%     
===========================================
  Files          185      184       -1     
  Lines        24789    24725      -64     
  Branches       463      463              
===========================================
- Hits         19216    19152      -64     
  Misses        5566     5566              
  Partials         7        7              

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

codecov[bot] avatar Feb 18 '24 10:02 codecov[bot]

🙏

syuilo avatar Feb 20 '24 06:02 syuilo

参考情報 userActivationについて Firefoxではバージョン120(2023-11-21)から実装されていて Firefoxの通常リリースは現状バージョン123 Extended Support Releaseは現状バージョン115となっており ESR版でこのAPI(navigator.userActivation)を使うと

Uncaught TypeError: navigator.userActivation is undefined

というエラーが発生して通知が来なくなるように見えます Chromeでは2019年1月, Edgeでは2020年1月に対応しているので 基本的には問題になることはありませんが navigator.userActivationundefinedならスキップにしてもいいのかもしれません

atsu1125 avatar Feb 23 '24 13:02 atsu1125

ESRでは上記の問題に加え、リアクション等の操作が行えなくなるようです。

creeper-0910 avatar Feb 23 '24 17:02 creeper-0910