Skip to content
This repository was archived by the owner on Feb 17, 2025. It is now read-only.

Conversation

@laminne
Copy link
Member

@laminne laminne commented Sep 1, 2024

What does this PR do?

close #76

  • 時刻の相対表示コンポーネントの実装 (Time)
  • お知らせコンポーネントの実装 (Announce)
  • 翻訳ファイルを更新

Additional information

  • radix-iconを導入しました

m1sk9
m1sk9 previously requested changes Sep 1, 2024
Copy link
Contributor

@m1sk9 m1sk9 left a comment

Choose a reason for hiding this comment

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

フロントエンドエンジニアではないので, コンポーネントについてはあまり見れていません.


肝心のデザインについて: アイコンのサイズがテキストと合っていないことでかなり歪な印象があるので合わせるべきだと思います.

CleanShot 2024-09-01 at 22 32 53@2x

@laminne laminne requested review from MikuroXina and m1sk9 September 2, 2024 02:48
@laminne laminne enabled auto-merge September 2, 2024 11:28
@laminne laminne requested a review from MikuroXina September 2, 2024 11:29
@nanai10a nanai10a removed the request for review from m1sk9 November 29, 2024 02:07
@laminne laminne dismissed m1sk9’s stale review January 17, 2025 03:42

reviewerから削除されたためdismiss

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 管理者からのお知らせ コンポーネントを実装する

4 participants