bootcamp
bootcamp copied to clipboard
practice_memo.vueをhtmlとjsでの実装に変更
Issue
- #7329
概要
practice_memo.vueを削除し、htmlとjsでの実装に変更しました。 また、プラクティスメモの余白が正しく表示されるようになりました。
変更確認方法
- feature/convert-practice-memo-vue-to-slimをローカルに取り込む
- ローカル環境を起動し、管理者ユーザーでログインする
- 任意の提出物のページに移動し、「プラクティスメモ」タブ内の機能が正常に動作していることを、mainブランチと比較して確認する
「プラクティスメモ」タブ確認項目:
- 「プラクティスメモ」タブを選択すると、保存されているプラクティスメモの内容がmarkdownとしてレンダリングされた状態で表示され、「編集」ボタンが表示される
- 「編集」ボタンをクリックすると、「メモ」タブが選択された状態で、テキストエリア内に保存されているプラクティスメモの内容が表示されており、編集可能な状態になっている
- 「プレビュー」タブを選択すると、一時的に編集が不可能になり、プラクティスメモの内容がmarkdownとしてレンダリングされた状態で表示される
- 「メモ」タブ内でプラクティスメモを編集してから「キャンセル」ボタンをクリックすると、プラクティスメモの内容が編集前の状態に戻り、画面表示が1.の状態に戻る
- 「メモ」タブ内でプラクティスメモを編集してから「保存」ボタンをクリックすると、「保存しました!」とトーストが表示され、画面表示が1.の状態に戻り、プラクティスメモの内容は編集後のものになっている
- プラクティスメモの内容が空の状態で保存すると、画面表示が1.の状態になるが、画像と一緒に「プラクティスメモはまだありません。」との表示が出る
- ページを再読み込みすると、「プラクティスメモ」タブ内では、最後に保存した状態のプラクティスメモが表示される
Screenshot
変更前
提出物のページで「プラクティスメモ」タブを選択
「編集」ボタンをクリックし、「メモ」タブが選択された状態
「プレビュー」タブが選択された状態
プラクティスメモを空にして保存した直後の状態
変更後
提出物のページで「プラクティスメモ」タブを選択
「編集」ボタンをクリックし、「メモ」タブが選択された状態
「プレビュー」タブが選択された状態
プラクティスメモを空にして保存した直後の状態
@reckyy お疲れ様です。 こちらのPRのレビューをお願いしてもよろしいでしょうか🙏 お手すきの際で構いませんので、よろしくお願いいたします🙇♂️
@taco-nantai レビュー依頼承知しました! CIが失敗しているようなので、そちらご確認いただければと思います。👀 パスを確認後レビューいたしますので、再度コメントいただけますと幸いです。 🙇
@reckyy お疲れ様です。 CIが通過したので、レビューをお願いします🙇♂️
@reckyy お疲れ様です。 レビューして頂きありがとうございました🙇♂️🙇♂️
@komagata お疲れ様です。 PRにapproveをいただいたので、レビューをお願いします🙇♂️
3/6 追記 machidaさんにUIの修正をお願いしたので、そちらの修正が完了した後にレビューをお願いします🙇♂️
@machida お疲れ様です。 本日のチーム開発ミーティングの中で、プラクティスメモのタブの、上方向のpaddingが表示されていない件について指摘をいただいたので、デザインの修正をお願いします🙇♂️
@taco-nantai 他の似た部分のデザイン(HTML+CSS)を参考にすると治りそうな気がするのでちょっとみてみていただければありがたいです〜
無理でしたら再度 @machida さんに頼んでいただければと思います。
@komagata @machida 改めて該当箇所のHTMLとCSSを確認してみたのですが、メンターメモのような他の似た部分と比べても問題点が見つけられず、独力では原因の解明が難しいと感じたので、大変申し訳ないのですが、 @machida さんに確認をお願いしたいです。 お手数おかけしてしまい、申し訳ありません🙇♂️
@taco-nantai 了解ですー僕の方で見てみます🙆♂️
@taco-nantai お待たせしました!こちら対応しましたー
machidaさん確認と対応ありがとうございます🙇♂️
@komagata お疲れ様です。 machidaさんにUIを修正していただいたので、レビューをお願いします🙇♂️
@komagata レビューありがとうございます! ご指摘いただいた箇所を修正したので、確認をお願いします🙇♂️
@taco-nantai テストが落ちているようです~。
@taco-nantai テストが落ちているようです~。
@komagata 失礼しました! テストが通っていることを確認したので、再度確認をお願いします🙇♂️
rebaseの操作の際にミスをしており、関係のないコミットを取り込んだ状態になっていたので修正しました。