search

Mayumi-Nakabayashi / withTone

2 stars 0 forks source link

Create bottom navigation bar #10

Closed y129may9th closed 1 year ago

y129may9th commented 1 year ago

やったこと

ボトムナビゲーションバーの追加

やってないこと

相談

  1. 2階層目以降のページには、ナビゲーションバーが表示されなくなります。必要かどうか確認したいです。 -> 口頭で相談しました。2階層以降は不要とのことでした。
  2. アイコンのずれ調整 303a384 について \n で改行分の調整したのですが、若干気持ち悪さがあるので、どうにかしたいです...良い方法あれば教えてください🙏🏻
before after
Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 19 48 45 Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 20 33 07

キャプチャ

コミュニティ 通知 content 学ぶ プロフェッショナル
Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 19 48 42 Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 19 48 43 Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 19 48 45 Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 19 48 27 Simulator Screenshot - iPhone 14 Pro - 2023-08-15 at 19 48 30
yuuya-1205 commented 1 year ago

キャプチャめっちゃ見やすいですね！！ 勉強になります！！

ken-ty commented 1 year ago

commit message も 今までの prefix 踏襲してくれた感じですかね？ PR ともに見やすくて助かります👍 個人的には https://dev.classmethod.jp/articles/commitizen/ のような prefix つけてくれるやつ使ってますが, お好みで.

y129may9th commented 1 year ago

https://github.com/Mayumi-Nakabayashi/withTone/pull/10#issuecomment-1679176990 @ken-ty ありがとうございますー！ kenty さんが prefix つけていたのと、仕事でも karma に倣って prefix をつけることになっているので！

Commitizen 良さそうでしたので入れてみます🙌🏻

ken-ty commented 1 year ago

LGTM!