feat(frontend): サーバーの表示をアイコンのみに切り替えられるように #14822
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14822 +/- ##
============================================
- Coverage 39.96% 19.35% -20.62%
============================================
Files 1563 728 -835
Lines 197878 103922 -93956
Branches 3646 991 -2655
============================================
- Hits 79080 20113 -58967
+ Misses 118193 83255 -34938
+ Partials 605 554 -51 ☔ View full report in Codecov by Sentry. |
| @@ -47,10 +47,10 @@ SPDX-License-Identifier: AGPL-3.0-only | |||
| </div> | |||
| <article v-else :class="$style.article" @contextmenu.stop="onContextmenu"> | |||
| <div v-if="appearNote.channel" :class="$style.colorBar" :style="{ background: appearNote.channel.color }"></div> | |||
| <MkAvatar :class="$style.avatar" :user="appearNote.user" :link="!mock" :preview="!mock"/> | |||
| <MkAvatar :class="$style.avatar" :user="appearNote.user" :link="!mock" :preview="!mock" :showInstance="!!showInstanceIcon && !!showTicker"/> | |||
There was a problem hiding this comment.
実装しているときにエラーが出ていたのでそのように記述したのですが、どのようなエラーが発生していたか忘れてしまいました...
|
インスタンスアイコンに透過がかかっている理由はどういったものですか？（アイコンとサーバーアイコンが同系色の場合に重なってインスタンスアイコンが見えなくなりそう） |
|
ユーザーアイコンがサーバーアイコンで隠れてしまわないようにしました |
一応ではありますが、サーバーアイコンの背景に |
|
別のスイッチとして持たせるのではなく、サーバーのバナー表示のメニューと統合させるので良さそう |
表示しないリモートユーザーに表示常に表示リモートユーザーに表示(アイコンのみ)常に表示(アイコンのみ)のような感じでしょうか？ |
|
MkSelectに含めました |
|
サーバーアイコンを透過させなかったらこういう感じですね… |
There was a problem hiding this comment.
これはコンポーネントとして切り出さず、MkAvatar内に直接実装した方が自然かつパフォーマンスも良さそうです
|
コンフリクト解消 |
What
GTLやSTL、HTLでのリモートユーザーのインスタンスをロゴのみの表示に切り替え可能にしました。
表示例
GTL上ではこのように表示されます。
PC環境ではインスタンスアイコンのホバー時にインスタンス名が表示されるようにしました。
従来の表示よりTLが見やすくなり、リモートユーザーとローカルユーザーをアイコンで
見分けられるようになります。
また、ノートの詳細表示の際は従来の表示になるようにしています。
メニュー項目の追加
従来の表示が望まれる場合を考慮し、切り替えできるように実装
リモートユーザーに表示(アイコンのみ) 、 常に表示(アイコンのみ) を追加
前回のPRからの変更点
・instanceIconをMkAvatarに含めることで、ユーザーアイコンのクリック可能な領域を減らしてしまう問題を改善
Why
TLの見やすさ向上のため
Fix #13623
Fix #14141
Additional info (optional)
特になし
Checklist