-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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(frontend): リアクションの横幅を150pxに制限するかどうかユーザーが選べるように #12416
feat(frontend): リアクションの横幅を150pxに制限するかどうかユーザーが選べるように #12416
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #12416 +/- ##
===========================================
- Coverage 78.78% 78.73% -0.05%
===========================================
Files 945 947 +2
Lines 102164 102897 +733
Branches 8262 8289 +27
===========================================
+ Hits 80488 81020 +532
- Misses 21676 21877 +201 ☔ 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.
翻訳はCrowdin経由で行わないと不具合が発生しますのでja-JP.yml
のみ編集を加えてください🙏
ご指摘ありがとうございます。修正いたしました 🙏 |
@@ -11,7 +11,7 @@ SPDX-License-Identifier: AGPL-3.0-only | |||
:class="[$style.root, { [$style.reacted]: note.myReaction == reaction, [$style.canToggle]: canToggle, [$style.small]: defaultStore.state.reactionsDisplaySize === 'small', [$style.large]: defaultStore.state.reactionsDisplaySize === 'large' }]" | |||
@click="toggleReaction()" | |||
> | |||
<MkReactionIcon :class="$style.icon" :reaction="reaction" :emojiUrl="note.reactionEmojis[reaction.substring(1, reaction.length - 1)]"/> | |||
<MkReactionIcon :class="defaultStore.state.limitWidthOfReaction ? $style.icon : ''" :reaction="reaction" :emojiUrl="note.reactionEmojis[reaction.substring(1, reaction.length - 1)]"/> |
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.
クラス名を limitWidth とかに変えた方がわかりやすそう
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.
元々の実装を使いましたが、変えてよければ変えてしまいますね!
👍 |
* feat(frontend): リアクションの横幅を150pxに制限するかどうかユーザーが選べるように * localesの変更をjs-JP.ymlのみに修正し、日本語をより分かりやすく * クラス名を.icon から .limitWidthに変更
CHANGELKOG要りそう |
* feat(frontend): リアクションの横幅を150pxに制限するかどうかユーザーが選べるように * localesの変更をjs-JP.ymlのみに修正し、日本語をより分かりやすく * クラス名を.icon から .limitWidthに変更
What
いつぞやのタイミングで、リアクションの横幅が150pxに制限されるようになりましたが、
ユーザーの設定画面で、横幅を制限して縮小させるか、縮小せずにそのまま表示するかを選べるようにしました。
Why
従来通りの見た目が良いというユーザーが一定数存在するため
Additional info (optional)
https://kasei.ski/notes/9mebe4h6ba 動作の様子
https://dev.kasei.ski/ にて動作確認できます
Checklist