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(client): データセーバーモードの追加 #10478

Merged
merged 30 commits into from
Apr 15, 2023

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Apr 4, 2023

Fix #10471

What

  • 「データセーバー」設定が追加されます
    • 全メディアを隠す
    • 画像においてBlurhash読み込みを強制
    • メディアが隠れている際にそのメディアのファイルサイズを表示
  • NSFW指定がないメディアを隠しているときに、「⚠️閲覧注意」が表示される問題を修正

Why

  • データ通信量の削減のため
  • NSFWでないメディアを隠したときにNSFWである旨が出るのがおかしいため

Additional info (optional)

【図1】新設の設定項目

image

【図2】非NSFW + データセーバー有効時のメッセージ

image

Checklist

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Apr 4, 2023
@github-actions github-actions bot requested review from rinsuki and syuilo April 4, 2023 15:08
@codecov
Copy link

codecov bot commented Apr 4, 2023

Codecov Report

Merging #10478 (9602624) into develop (a674399) will increase coverage by 0.03%.
The diff coverage is 81.25%.

@@             Coverage Diff             @@
##           develop   #10478      +/-   ##
===========================================
+ Coverage    76.07%   76.10%   +0.03%     
===========================================
  Files          896      899       +3     
  Lines        88490    88665     +175     
  Branches      6247     6247              
===========================================
+ Hits         67317    67481     +164     
- Misses       21173    21184      +11     
Impacted Files Coverage Δ
packages/frontend/src/components/MkMediaImage.vue 92.08% <50.00%> (-0.62%) ⬇️
...ages/frontend/src/components/MkImgWithBlurhash.vue 89.74% <100.00%> (+0.26%) ⬆️
packages/frontend/src/components/MkMediaVideo.vue 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

packages/frontend/src/components/MkMediaImage.vue Outdated Show resolved Hide resolved
packages/frontend/src/components/MkMediaVideo.vue Outdated Show resolved Hide resolved
packages/frontend/src/components/MkMediaImage.vue Outdated Show resolved Hide resolved
packages/frontend/src/components/MkMediaImage.vue Outdated Show resolved Hide resolved
packages/frontend/src/components/MkMediaVideo.vue Outdated Show resolved Hide resolved
Co-authored-by: Acid Chicken (硫酸鶏) <root@acid-chicken.com>
@acid-chicken
Copy link
Member

見た目の点では良さそうな一方、機能の点では非 NSFW だけサイズの情報が得られるのはアンバランス感があり悩ましい

@kakkokari-gtyih
Copy link
Contributor Author

Twitterだと、データセーバー + センシティブ表示では2アクションで画像を見れるけどなんかコレジャナイ感

image

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Apr 4, 2023

閲覧注意のときは「閲覧注意(サイズ)」で表示させると良いかも・・・?

image

locales/ja-JP.yml Outdated Show resolved Hide resolved
@syuilo
Copy link
Member

syuilo commented Apr 5, 2023

メディアの表示設定機能はデータ通信量を削減する目的ではなく、強制的にNSFW設定で表示する機能なので、現状に問題があるようには感じません。また、NSFWはデータ通信量を削減する目的があるわけでもありません(NSFW設定したとしても画像が裏で読み込まれるように改修される可能性も十分にあり、NSFW設定されていると画像が読み込まれないのはあくまで副次的、かつ偶然の効果にすぎません)
そのため「データ通信量を削減できる設定」が必要なのであれば、別機能として実装した方が良いように思いました

@kakkokari-gtyih kakkokari-gtyih marked this pull request as draft April 6, 2023 00:06
@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Apr 6, 2023

メディアの表示設定機能はデータ通信量を削減する目的ではなく、強制的にNSFW設定で表示する機能なので、現状に問題があるようには感じません。また、NSFWはデータ通信量を削減する目的があるわけでもありません(NSFW設定したとしても画像が裏で読み込まれるように改修される可能性も十分にあり、NSFW設定されていると画像が読み込まれないのはあくまで副次的、かつ偶然の効果にすぎません) そのため「データ通信量を削減できる設定」が必要なのであれば、別機能として実装した方が良いように思いました

別機能として実装するようにします

@kakkokari-gtyih kakkokari-gtyih changed the title メディア表示設定の文言変更 + 非NSFWメディアを隠した際の文言変更 [WIP] データセーバーモードの追加 Apr 6, 2023
@kakkokari-gtyih kakkokari-gtyih marked this pull request as ready for review April 6, 2023 01:25
@kakkokari-gtyih kakkokari-gtyih changed the title [WIP] データセーバーモードの追加 データセーバーモードの追加 Apr 6, 2023
@kakkokari-gtyih
Copy link
Contributor Author

コンフリクト解消

@kakkokari-gtyih kakkokari-gtyih changed the title データセーバーモードの追加 feat(client): データセーバーモードの追加 Apr 9, 2023
@tamaina
Copy link
Member

tamaina commented Apr 12, 2023

リモートのファイルでファイルサイズが0の場合があるので、その場合はサイズを表示しないようにした

@tamaina
Copy link
Member

tamaina commented Apr 12, 2023

p1.a9z.devで動作中

@syuilo 確認してみて

@syuilo
Copy link
Member

syuilo commented Apr 12, 2023

画像をデフォルトで隠したとして果たしてデータセーブになるのかよくわかっていない(人間の心理的に隠された画像あったらたいてい開かない?)

@tamaina
Copy link
Member

tamaina commented Apr 12, 2023

webpで20kb前後まで圧縮してるのであんまり効果なさそうではある

(バックグラウンドの読み込みを止めるとかのほうが効果ありそう)

@tamaina
Copy link
Member

tamaina commented Apr 12, 2023

ただ携帯回線の「ギガ」を消費しきって200kbpsになるとかだと20KBでもでかいので意味はある

@syuilo syuilo merged commit bcbf06a into misskey-dev:develop Apr 15, 2023
14 checks passed
@syuilo
Copy link
Member

syuilo commented Apr 15, 2023

🙏🏻 🙏🏻

@kakkokari-gtyih kakkokari-gtyih deleted the datasaver branch April 15, 2023 06:33
na2na-p pushed a commit to na2na-p/misskey that referenced this pull request May 10, 2023
* change nsfw settings

* Update CHANGELOG.md

* (fix) eliminate warning message when manually hide

* Apply suggestions from code review

Co-authored-by: Acid Chicken (硫酸鶏) <root@acid-chicken.com>

* (change) translation key

* revert nsfw settings (partial)

* (add) data saver setting

* Integrate MkMediaBlurhash and MkImgWithBlurhash

* Update CHANGELOG.md

* 🎨

* リモートのファイルでsizeが0の場合は表示しない, refを作らない

* fix

* かっこ

---------

Co-authored-by: Acid Chicken (硫酸鶏) <root@acid-chicken.com>
Co-authored-by: tamaina <tamaina@hotmail.co.jp>
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.

[Client] データセーバーモード(タップするまで画像を読み込まない)
5 participants