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

TL上のサーバー情報をアイコン表示に切り替えられるやつ (中身の変更なし) #15

Merged
merged 8 commits into from
Jul 9, 2024

Conversation

MattyaDaihuku
Copy link

@MattyaDaihuku MattyaDaihuku commented Jul 6, 2024

What

GTLやSTLでのリモートユーザーのインスタンスをロゴのみの表示に切り替え可能にしました。

  • 表示例

Screenshot 2024-07-06 20 34 28

 GTL上ではこのように表示される。
 従来の表示よりTLが見やすくなり、リモートユーザーとローカルユーザーをアイコンで
 見分けられる。

  • トグルの実装

Screenshot 2024-07-06 20 34 56

 従来の表示が望まれる場合を考慮し、切り替えできるように実装
 ノートのサーバー情報にて、表示しないを選択している場合はトグルを非表示に

Screenshot 2024-07-06 20 35 19

Why

 TLの見やすさ向上のため

Fix misskey-dev#13623

Additional info (optional)

 ローカル環境のみで動作を確認しているため、GTLでの挙動は未確認

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

@MattyaDaihuku
Copy link
Author

MattyaDaihuku commented Jul 6, 2024

あとで独自機能のフラグ付けるます

@MattyaDaihuku
Copy link
Author

(フラグって言い方、正しいのだろうか)

@chan-mai chan-mai self-requested a review July 9, 2024 09:20
@chan-mai chan-mai merged commit e260847 into team-shahu:develop Jul 9, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

リモートユーザーの所属サーバーをアイコンで表示する方式
2 participants