commew / timelogger-web

時間記録アプリ

https://timmew.commew.net

MIT License

2 stars 0 forks source link

#47 ホバー効果付きのカテゴリボタンを実装した #66

Closed kuniyuki-f closed 1 year ago

kuniyuki-f commented 1 year ago

issueURL

47

この PR で対応する範囲

47 の以下を実現するためのコンポーネントを実装する
- 左カラムのカテゴリ名にホバーしたら計測開始ボタンが表示されるようになっていること
実装したstorybookでコンポーネントの見た目を確認できるようにする

このPRで対応しない範囲

テストケースはまだ完全には実装しない →APIリクエストの未実装であるため。
SideBarコンポーネントへの組み込み

Storybook の URL、スクリーンショット

変更点概要

componentsにCategoryButtonを追加しました。 NavLinkではなくBoxで実装しています。理由：mantineのuseHoverを使用したかったため。

レビュアーに重点的にチェックして欲しい点

PRの出し方や粒度は適切か
- 初めてのPRなので、コミットの単位やPRの出し方が適切かどうか教えてほしいです
SideBarコンポーネントではNavLinkで実装されているが、Boxで実装している点について適切かどうか
見た目をデザインに忠実に実装できているか

補足情報

とくになしです。

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
timelogger-web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 1, 2023 9:56am

kuniyuki-f commented 1 year ago

@keitakn レビューありがとうございました！きゅーぶさんからApproveもらったらマージしたいと思います！

@stkzk3110 お疲れ様です！お手隙で最終的なコードのご確認・Approveをお願いします！