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

enhance/feat(frontend): データセーバーの改良・強化 #12526

Merged
merged 13 commits into from
Dec 3, 2023

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Nov 30, 2023

What

  • データセーバーの適用範囲を個別で設定できるように
    image
  • コードハイライトを隠す設定を追加
    image

Why

Fix #12259
Fix #12435

Additional info (optional)

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 30, 2023
@kakkokari-gtyih kakkokari-gtyih marked this pull request as draft November 30, 2023 11:34
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

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

Comparison is base (2eb86e0) 78.79% compared to head (069c210) 78.42%.

❗ Current head 069c210 differs from pull request most recent head 21fed56. Consider uploading reports for the commit 21fed56 to get more accurate results

Files Patch % Lines
packages/frontend/src/components/MkMediaImage.vue 50.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #12526      +/-   ##
===========================================
- Coverage    78.79%   78.42%   -0.38%     
===========================================
  Files          953      180     -773     
  Lines       103748    23857   -79891     
  Branches      8348      495    -7853     
===========================================
- Hits         81750    18709   -63041     
+ Misses       21998     5148   -16850     

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

@kakkokari-gtyih kakkokari-gtyih marked this pull request as ready for review November 30, 2023 11:56
Comment on lines +26 to +32
// なんかtestがうまいこと動かないのでここに書く
dataSaver: {
media: false,
avatar: false,
urlPreview: false,
code: false,
},
Copy link
Contributor Author

Choose a reason for hiding this comment

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

vitestが落ちるのでむりやり初期値を書いて回避しました

@kakkokari-gtyih kakkokari-gtyih changed the title enhance/feat(frontend): データセーバーの改良 enhance/feat(frontend): データセーバーの改良・強化 Dec 1, 2023
@noridev
Copy link
Contributor

noridev commented Dec 1, 2023

こんにちは。全て有効にする/全て無効にするボタンが4つのデータセーバーオプションのいずれかをオンまたはオフにしないと動作しないようですが、もしよろしければご確認いただけますか?

@kakkokari-gtyih
Copy link
Contributor Author

こんにちは。全て有効にする/全て無効にするボタンが4つのデータセーバーオプションのいずれかをオンまたはオフにしないと動作しないようですが、もしよろしければご確認いただけますか?

みてみます

@kakkokari-gtyih
Copy link
Contributor Author

こんにちは。全て有効にする/全て無効にするボタンが4つのデータセーバーオプションのいずれかをオンまたはオフにしないと動作しないようですが、もしよろしければご確認いただけますか?

なおりました

@syuilo syuilo merged commit 5bf7813 into misskey-dev:develop Dec 3, 2023
9 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 3, 2023

👍🏻

@kakkokari-gtyih kakkokari-gtyih deleted the enh-12259-12435 branch December 8, 2023 06:34
camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* enhance(frontend): データセーバーを個別で設定できるように

* Update Changelog

* fix design

* (fix) 設定が当たらない

* fix test(無理やり感)

* (fix) 設定がない状態ですべて有効・向操作が効かない

* fix

* tweak

---------

Co-authored-by: syuilo <Syuilotan@yahoo.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
3 participants