-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
ノートを畳む条件を仮想行数、旧式、実際の表示の大きさから選べるようにする #13961
base: develop
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #13961 +/- ##
============================================
- Coverage 39.58% 19.60% -19.98%
============================================
Files 1553 721 -832
Lines 195033 102387 -92646
Branches 3624 990 -2634
============================================
- Hits 77200 20075 -57125
+ Misses 117230 81763 -35467
+ Partials 603 549 -54 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eslintのコーディング規約チェック通ってないですね
misskey/packages/frontend/src/components/MkNote.vue
Error: 234:4 error Missing semicolon semi
misskey/misskey/packages/frontend/src/components/MkSubNoteContent.vue
Error: 55:2 error 'onMounted' is not defined no-undef
Error: 60:4 error Missing semicolon semi
/misskey/packages/frontend/src/pages/settings/general.vue
Error: 101:43 error Attribute "v-if" should go before "v-model" vue/attributes-order
Error: 55:2 error 'onMounted' is not defined no-undef
が出ているとなると手元でも正常に動作してないと思われるのですがPR提出前にこちら動作確認されていますか?
Co-authored-by: taichan <[email protected]>
修正ありがとうございました。
一応動作確認はしましたが、畳み条件を「実際の表示サイズに基づく」に設定した際の被引用ノートの表示の確認に漏れがあったかもしれません。改めて確認しました。 |
仮に新方式2種を実装するとなると、今度は逆に旧式を残す意味があまり感じられない気がする |
@kakkokari-gtyih |
What
#13287 と @kakkokari-gtyih さんの #13761 をマージし、設定項目などを生やしました。
これにより、大きいノートが畳まれる条件を
の三つから選べるようになります。
ついでに畳む大きさを大・中・小から選べるようにしています。
Why
#13266 で挙がっていた懸念点を
という形である程度解決できるようにします。
Additional info (optional)
設定項目の名づけにあまり自信がありません
Checklist