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

サウンド設定に「サウンドを出力しない」と「Misskeyがアクティブな時のみサウンドを出力する」を追加 #12342

Merged
merged 8 commits into from
Nov 26, 2023

Conversation

samunohito
Copy link
Member

What

#12341 の対応です。
この対応により、設定画面から音声の出力そのものをOFFにするか、非アクティブ時のみOFFにするかなどの設定ができるようになります。

Why

詳細は #12341 を参照願います。

Additional info (optional)

ブラウザとスマホ(Android+Chrome)で実動作確認

Checklist

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Nov 15, 2023
Copy link

codecov bot commented Nov 15, 2023

Codecov Report

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

Comparison is base (755ca97) 78.69% compared to head (5ddff13) 78.72%.

Files Patch % Lines
packages/frontend/src/scripts/sound.ts 12.50% 14 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12342      +/-   ##
===========================================
+ Coverage    78.69%   78.72%   +0.02%     
===========================================
  Files          948      949       +1     
  Lines       103086   103228     +142     
  Branches      8300     8302       +2     
===========================================
+ Hits         81123    81263     +140     
- Misses       21963    21965       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

github-actions bot commented Nov 15, 2023

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

@kakkokari-gtyih
Copy link
Contributor

「マスター音量0」と「サウンドを出力しない」設定がダブるのは気になる

@tai-cha
Copy link
Contributor

tai-cha commented Nov 23, 2023

ミュートボタンと考えれば不自然じゃないかもしれない…?

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Nov 23, 2023

そうかも

@samunohito
Copy link
Member Author

「マスター音量0」と「サウンドを出力しない」設定がダブるのは気になる

思想としては

ミュートボタンと考えれば不自然じゃないかもしれない…?

コレです。「音を出さない」という結果は同じですが、「発音しない」のと「音量をゼロにする」のでは意味が違うかなと思いまして

@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

「マスター音量0」と「サウンドを出力しない」設定がダブるのは気になる

これ

@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

まあこのままでもいいわね

@kakkokari-gtyih
Copy link
Contributor

一括でオンオフできるのはアリかと思ったりもした

@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

この設定を使うとMisskey全体のサウンドがオフになるような印象を受けそう

@kakkokari-gtyih
Copy link
Contributor

あー

@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

動画の再生音量などはまた別に管理されてる

@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

サウンド設定を効果音設定などにした方が良さそうだけどオフトピだわね

@syuilo syuilo merged commit c9503da into misskey-dev:develop Nov 26, 2023
15 checks passed
@syuilo
Copy link
Member

syuilo commented Nov 26, 2023

🙏

@syuilo
Copy link
Member

syuilo commented Dec 22, 2023

これ 2023.11.1 に CHAGELOG追加されてるけど合ってる?

@kakkokari-gtyih
Copy link
Contributor

これ 2023.11.1 に CHAGELOG追加されてるけど合ってる?

たぶんちがいますね(tagにはv2023.12.0-beta.1以降しか含まれてない)

image

@samunohito samunohito deleted the enhance/no_use_sound_option branch January 11, 2024 00:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants