bootcamp
bootcamp copied to clipboard
fjordllc
管理者権限からみた相談部屋内の「ユーザー日報」タブの表示バグを修正
Issue
- #7476
概要
adminアカウントの相談部屋詳細において、ユーザーの日報を表示するタブをクリックした際、ユーザーの日報がない場合CSSが適用されておらず、そのバグを修正するIssueです。
変更確認方法
-
bug/reports-tab-missing-borderをローカルに取り込む - komagataさんまたはmachidaさんどちらかの管理者権限でログイン
- http://localhost:3000/talks にアクセス
- 日報がないユーザーの詳細に入る
-
ユーザーの日報タブをクリックし、CSSが正しく適用されていることを確認する
- 枠線が表示されているかの確認
- 枠線内はバックグラウンドカラー適用されているか
Screenshot
変更前
変更後
@machida
お疲れ様です！
本日の開発ミーティングで、デモをしたこちらのデザインについて駒形さんから、町田さんに相談してみると良いと助言をもらってのメンションになります🙏
PCが重く、最後にレビューを回していただいたので、直接ご説明できず申し訳ありません🙇♂️
今回の修正内容
adminアカウントの相談部屋詳細において、「ユーザーの日報」を表示するタブをクリックした際、ユーザーの日報がない場合にCSSが適用されておらず、そのバグを修正するIssueです。
相談内容
修正箇所の要素のマージンが狭く、その修正が必要かどうかの相談になります！
修正箇所
修正前
修正後
赤枠の部分のアイコン表示と「日報はまだありません。」の表示の上下の隙間があまりないので、そこをこのままでいくか、修正をするかのご相談です。
同じ要素の別タブ「ユーザーメモ」の同じ箇所は上下の隙間が程よく入っているようです。
対象のファイル
app/javascript/components/Reports.jsx
こちらのReactファイルの以下のクラスを追加しました。
↓今回、このクラスを追加したのは、これが日報があった場合に適用されるクラスで、日報がある場合には正しく「枠線」「バックグラウンドカラー」が表示されていたので、日報がない場合にも、同様に適用した修正になります。
日報が1以上存在する場合は表示されている
「日報がある場合」と「日報がない場合」と同じCSSで良いか、またその場合のデザイン的に問題ないかの確認になります🙏
@machida
お疲れ様です！1週間が経過したため、念のためのご連絡をさせていただきます🙏 私の手順の不手際で通知が届いていない可能性も考慮し、このメッセージを送ります🙇♂️
もしすでにご存じで、お忙しい状況でしたら、急ぎの件ではありませんので、お手すきの際に対応いただけますと幸いです。引き続きどうぞよろしくお願いいたします🙏
@taco-nantai
お疲れ様です🙏 お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか🙆？ 1ポイントのIssueで内容も軽いものなのでそんなにお手間かからない内容かとおもいます🙏
急ぎではありませんので、無理ないタイミングで全く問題ありません！(1~2週間後とかでもOKです)
ご検討のほど、よろしくお願いいたします🙇♂️
@taco-nantai
お忙しいところ確認いただき、ありがとうございました🙏 引き続き、よろしくお願いします！
@komagata
お疲れ様です！メンバーによるレビューが終わりました。確認をお願いいたします🙏