hirano-vm4

@machida お疲れ様です！ 本日の開発ミーティングで、デモをしたこちらのデザインについて駒形さんから、町田さんに相談してみると良いと助言をもらってのメンションになります🙏 PCが重く、最後にレビューを回していただいたので、直接ご説明できず申し訳ありません🙇‍♂️ # 今回の修正内容 adminアカウントの相談部屋詳細において、「ユーザーの日報」を表示するタブをクリックした際、ユーザーの日報がない場合にCSSが適用されておらず、そのバグを修正するIssueです。 # 相談内容 修正箇所の要素のマージンが狭く、その修正が必要かどうかの相談になります！ ### 修正箇所 `修正前`  `修正後`  赤枠の部分のアイコン表示と「日報はまだありません。」の表示の上下の隙間があまりないので、そこをこのままでいくか、修正をするかのご相談です。 同じ要素の別タブ「ユーザーメモ」の同じ箇所は上下の隙間が程よく入っているようです。 ## 対象のファイル `app/javascript/components/Reports.jsx` こちらのReactファイルの以下のクラスを追加しました。 ↓今回、このクラスを追加したのは、これが日報があった場合に適用されるクラスで、日報がある場合には正しく「枠線」「バックグラウンドカラー」が表示されていたので、日報がない場合にも、同様に適用した修正になります。 `日報が1以上存在する場合は表示されている` 「日報がある場合」と「日報がない場合」と同じCSSで良いか、またその場合のデザイン的に問題ないかの確認になります🙏

@machida お疲れ様です！1週間が経過したため、念のためのご連絡をさせていただきます🙏 私の手順の不手際で通知が届いていない可能性も考慮し、このメッセージを送ります🙇‍♂️ もしすでにご存じで、お忙しい状況でしたら、急ぎの件ではありませんので、お手すきの際に対応いただけますと幸いです。引き続きどうぞよろしくお願いいたします🙏

@machida お忙しいところ、ありがとうございます🙏引き続き進めたいとおもいます💪

@taco-nantai お疲れ様です🙏 お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか🙆？ 1ポイントのIssueで内容も軽いものなのでそんなにお手間かからない内容かとおもいます🙏 急ぎではありませんので、無理ないタイミングで全く問題ありません！(1~2週間後とかでもOKです) ご検討のほど、よろしくお願いいたします🙇‍♂️

@taco-nantai お忙しいところ確認いただき、ありがとうございました🙏 引き続き、よろしくお願いします！ @komagata お疲れ様です！メンバーによるレビューが終わりました。確認をお願いいたします🙏

@komagata こちらconflictの修正を行いました！

@komagata ありがとうございます！ステージング環境にデプロイされたことを確認次第、動作確認に入ります🙆

@machida @komagata ## 方向性の確認と提案 ### 【提案】休みの理由の表示について >管理者が定期イベント編集ページから休みを登録できるようにする こちらの実装について、1つの定期イベントに対して複数休みの日付が登録されることになるため、以下のように定期イベント休日登録用にテーブルを別に切り出して実装しようと考えています。 `イメージ：regular_event_holidays（仮名）` | カラム名 | データ型 | | |----------------|----------------|--------------------------------------------------| | id | Integer | 主キー | | regular_event_id | Integer | `regular_events`テーブルのレコードを参照する外部キー |...

@komagata >descriptionはstringではなくtextがいいかも。 こちら、たしかに長めに入力されることも想定して`test`型で進めていきます！

@machida ## 「休みの追加」フォームを追加実装する方向性のご相談 >年月日を入力し登録ボタンを押すと、input タグから普通の文字に画面遷移なく変更され、新たなinputタグが出現する。 普通の文字になった際、右側に削除リンクが表示される。削除をクリックすると、画面遷移なく登録した年月日が消える。  こちらの実装ですが、休みに設定する日付をユーザーが入力したあと **「登録」ボタンを押して、次のフォームが出現する** 、というオーダーかと思いますが、以下のような実装もありかな？と思ってのご相談になります🙏 以下のように同じページに「定期開催日」というフォームがあります。 これは開催日程を設定して、追加があれば「定期開催日追加」ボタンを押して複数の開催日を設定し、最後にページ下部のボタンを押して確定します（新規作成であれば「作成」ボタン、更新であれば「内容変更」）。  定期イベントに休日を登録する場合においても、同様の実装でも大丈夫そうでしょうか？ 提案した理由としては2点あります。 - **日付のフォーム入力部分にも「登録」があると、ページ下部の確定ボタン（新規作成であれば「作成」ボタン、更新であれば「内容変更」）を押さないで別ページに移動してしまうなどの、勘違いが発生するかもと考えた（あくまでフォームは設定のみで、確定はページ下部の確定ボタンとしたい）** - **同じフォーム内で統一したほうがユーザーは理解しやすいと考えた** 上記の点を個人的に感じたので、ご相談させていただきました🙏 下記のようなイメージです  町田さんのイメージもあるかと思いますので、ご意見やアドバイスいただけたら嬉しいです🙆