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