bootcamp icon indicating copy to clipboard operation
bootcamp copied to clipboard

practice_memo.vueをhtmlとjsでの実装に変更

Open taco-nantai opened this issue 11 months ago • 16 comments

Issue

  • #7329

概要

practice_memo.vueを削除し、htmlとjsでの実装に変更しました。 また、プラクティスメモの余白が正しく表示されるようになりました。

変更確認方法

  1. feature/convert-practice-memo-vue-to-slimをローカルに取り込む
  2. ローカル環境を起動し、管理者ユーザーでログインする
  3. 任意の提出物のページに移動し、「プラクティスメモ」タブ内の機能が正常に動作していることを、mainブランチと比較して確認する

「プラクティスメモ」タブ確認項目:

  1. 「プラクティスメモ」タブを選択すると、保存されているプラクティスメモの内容がmarkdownとしてレンダリングされた状態で表示され、「編集」ボタンが表示される
  2. 「編集」ボタンをクリックすると、「メモ」タブが選択された状態で、テキストエリア内に保存されているプラクティスメモの内容が表示されており、編集可能な状態になっている
  3. 「プレビュー」タブを選択すると、一時的に編集が不可能になり、プラクティスメモの内容がmarkdownとしてレンダリングされた状態で表示される
  4. 「メモ」タブ内でプラクティスメモを編集してから「キャンセル」ボタンをクリックすると、プラクティスメモの内容が編集前の状態に戻り、画面表示が1.の状態に戻る
  5. 「メモ」タブ内でプラクティスメモを編集してから「保存」ボタンをクリックすると、「保存しました!」とトーストが表示され、画面表示が1.の状態に戻り、プラクティスメモの内容は編集後のものになっている
  6. プラクティスメモの内容が空の状態で保存すると、画面表示が1.の状態になるが、画像と一緒に「プラクティスメモはまだありません。」との表示が出る
  7. ページを再読み込みすると、「プラクティスメモ」タブ内では、最後に保存した状態のプラクティスメモが表示される

 

Screenshot

変更前

提出物のページで「プラクティスメモ」タブを選択

image

「編集」ボタンをクリックし、「メモ」タブが選択された状態

image

「プレビュー」タブが選択された状態

image

プラクティスメモを空にして保存した直後の状態

image

変更後

提出物のページで「プラクティスメモ」タブを選択 image

「編集」ボタンをクリックし、「メモ」タブが選択された状態 image

「プレビュー」タブが選択された状態 image

プラクティスメモを空にして保存した直後の状態 image

taco-nantai avatar Feb 26 '24 15:02 taco-nantai

@reckyy お疲れ様です。 こちらのPRのレビューをお願いしてもよろしいでしょうか🙏 お手すきの際で構いませんので、よろしくお願いいたします🙇‍♂️

ghost avatar Feb 27 '24 01:02 ghost

@taco-nantai レビュー依頼承知しました! CIが失敗しているようなので、そちらご確認いただければと思います。👀 パスを確認後レビューいたしますので、再度コメントいただけますと幸いです。 🙇

reckyy avatar Feb 27 '24 02:02 reckyy

@reckyy お疲れ様です。 CIが通過したので、レビューをお願いします🙇‍♂️

ghost avatar Feb 28 '24 05:02 ghost

@reckyy お疲れ様です。 レビューして頂きありがとうございました🙇‍♂️🙇‍♂️

ghost avatar Mar 02 '24 01:03 ghost

@komagata お疲れ様です。 PRにapproveをいただいたので、レビューをお願いします🙇‍♂️

3/6 追記 machidaさんにUIの修正をお願いしたので、そちらの修正が完了した後にレビューをお願いします🙇‍♂️

ghost avatar Mar 02 '24 01:03 ghost

@machida お疲れ様です。 本日のチーム開発ミーティングの中で、プラクティスメモのタブの、上方向のpaddingが表示されていない件について指摘をいただいたので、デザインの修正をお願いします🙇‍♂️

ghost avatar Mar 06 '24 10:03 ghost

@taco-nantai 他の似た部分のデザイン(HTML+CSS)を参考にすると治りそうな気がするのでちょっとみてみていただければありがたいです〜

無理でしたら再度 @machida さんに頼んでいただければと思います。

komagata avatar Mar 09 '24 17:03 komagata

@komagata @machida 改めて該当箇所のHTMLとCSSを確認してみたのですが、メンターメモのような他の似た部分と比べても問題点が見つけられず、独力では原因の解明が難しいと感じたので、大変申し訳ないのですが、 @machida さんに確認をお願いしたいです。 お手数おかけしてしまい、申し訳ありません🙇‍♂️

ghost avatar Mar 11 '24 05:03 ghost

@taco-nantai 了解ですー僕の方で見てみます🙆‍♂️

machida avatar Mar 11 '24 06:03 machida

@taco-nantai お待たせしました!こちら対応しましたー

machida avatar Mar 12 '24 05:03 machida

machidaさん確認と対応ありがとうございます🙇‍♂️

ghost avatar Mar 12 '24 06:03 ghost

@komagata お疲れ様です。 machidaさんにUIを修正していただいたので、レビューをお願いします🙇‍♂️

ghost avatar Mar 12 '24 06:03 ghost

@komagata レビューありがとうございます! ご指摘いただいた箇所を修正したので、確認をお願いします🙇‍♂️

ghost avatar Mar 20 '24 06:03 ghost

@taco-nantai テストが落ちているようです~。

komagata avatar Mar 26 '24 09:03 komagata

@taco-nantai テストが落ちているようです~。

@komagata 失礼しました! テストが通っていることを確認したので、再度確認をお願いします🙇‍♂️

ghost avatar Mar 26 '24 14:03 ghost

rebaseの操作の際にミスをしており、関係のないコミットを取り込んだ状態になっていたので修正しました。

ghost avatar Mar 29 '24 22:03 ghost