bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

絞り込みパーツをタブの下に移動

Open yuki-snow1823 opened this issue 2 years ago • 1 comments

Issue

  • #4964

概要

Q&Aをプラクティスの内容で絞り検索をするときの検索バーを、タブより下に配置したい。

変更確認方法

  1. ブランチfeature/move-narrow-down-partsをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. ログイン
  4. http://localhost:3000/questions?target=not_solved にアクセスして確認

変更前

before

変更後

after

yuki-snow1823 avatar Sep 01 '22 03:09 yuki-snow1823

@choco0809 初めまして、お世話になります堀越です🙇‍♂️ 可能でしたら、お手隙の際にレビューお願いします。

yuki-snow1823 avatar Sep 01 '22 05:09 yuki-snow1823

@yuki-snow1823 お疲れ様です🙇‍♂️ 初めてのレビューだったので、お時間がかかってしまいました... コード、動作ともに問題ないことを確認しましたので、Approveさせていただきます!

choco0809 avatar Sep 03 '22 00:09 choco0809

@choco0809 ご丁寧にありがとうございます!自分の方でもできることがあればいつでもメンションしてください。

yuki-snow1823 avatar Sep 05 '22 00:09 yuki-snow1823

@komagata @machida 特に問題なさそうでしたので、移動のみの変更となっております。 レビューよろしくお願いいたします。

issueを渡していただいた際に @machida さんからもお話があったと思うので、一応メンションつけさせていただきました。

yuki-snow1823 avatar Sep 05 '22 00:09 yuki-snow1823

こちらこのままだとデザイン的におかしいかもなので @machida さんにデザインを依頼して頂ければ〜

komagata avatar Sep 05 '22 16:09 komagata

かしこまりました!失礼いたしました(__) 本日の振り返りの際にご依頼します。

yuki-snow1823 avatar Sep 06 '22 01:09 yuki-snow1823

@machida お疲れ様です。こちらのプルリクエストに関してデザインのご確認をお願いします。

また、カリキュラムの下記の説明でよくわからなかった部分があるので、本日の質問タイムに質問させていただきます(__)

注意 もし、デザイナーがデザインを入れる際に、すでに main ブランチや他のブランチに入っているデザインを流用するため、今あなたとデザイナーが作業しているブランチを rebase などで git の履歴に変更が発生する作業を行う場合があります。その場合、デザインが完了した後にそのブランチを pull するとエラーが生じ、pull ができません。 その場合は、手元のブランチを念のため別の名前を付けて別ブランチにした上で、手元のブランチを削除して、リモートブランチをチェックアウトします。もし、自分の手元で行った作業がそこに含まれていなかったら、別の名前を付けて別ブランチにしたブランチの中の取り込みたい作業をチェックアウトしたブランチの上にチェリーピックします。

yuki-snow1823 avatar Sep 06 '22 01:09 yuki-snow1823

@machida 先程はありがとうございました。改めてメンションお送りいたします。 デザインレビューをよろしくお願いします。

yuki-snow1823 avatar Sep 06 '22 07:09 yuki-snow1823

@machida お世話になっております。ちょっと間が空いてしまったので一応、再度メンションさせていただきます。 お忙しいところ恐縮ですが、こちらのデザインレビューをお願いいたします。

yuki-snow1823 avatar Sep 16 '22 06:09 yuki-snow1823

@yuki-snow1823 すいません、遅くなりました🙇‍♂️ デザインの修正をしましたー

@komagata 今回の変更で不要なパーツができたので、CSSの変更が色々入ってます。CSSの変更は無視してレビューをお願いします🙏

machida avatar Sep 20 '22 07:09 machida

@machida お忙しいところありがとうございました!

yuki-snow1823 avatar Sep 20 '22 23:09 yuki-snow1823

長い行に改行を入れるだけの修正なのでマージしてしまいます〜

machida avatar Sep 28 '22 14:09 machida

@machida すみません!反応遅れておりました。こちらありがとうございました! 元のissueをcloseしておきます。

yuki-snow1823 avatar Oct 18 '22 21:10 yuki-snow1823