fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
286 stars 71 forks source link

answer.vue, answers.vueを非vue化する #5116

Open komagata opened 2 years ago

komagata commented 2 years ago

vueを使わないようにするためにタイトルになっているファイルを使っている箇所を普通のHTMLでの実装に変える。

元々Reactに変更する予定でしたが、最終的にはHotwireにする予定なので、VueもReactも使わない実装にしたいです。(普通にviewに表示する方式)

応相談

最終的にはrailsを7系にアップデートしてhotwireにする予定。 その前段階として、SPAではなく、極力JS無しの紙芝居構成にしたい。(hotwireを使うとその紙芝居構成を画面遷移無しで切り替えられる)

現状はGitHub風のコメント欄になっているが、一時的にもっと簡単な形式にしてもいいかもと思っているが @machida さんと相談して進めていただきたいです。 (上記の部分もcommentsのReact化を大変にしている要因の一つかもと思うので)

github-actions[bot] commented 2 years ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 2 years ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 1 year ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 1 year ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 1 year ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 1 year ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 1 year ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 11 months ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 9 months ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 7 months ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

github-actions[bot] commented 5 months ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

reckyy commented 3 months ago

@machida お疲れ様です。 こちらのIssueの実装方針についてご相談させていただきたいです。

現状はGitHub風のコメント欄になっているが、一時的にもっと簡単な形式にしてもいいかもと思っているが @machida さんと相談して進めていただきたいです。 (上記の部分もcommentsのReact化を大変にしている要因の一つかもと思うので)

ベストアンサーの表示を簡略化する、リアクションをGitHubの賛成票のみにする、といった提案を私からするのではなく、machidaさんから簡略化したデザインをいただき、それを基に実装を進めるという方向が良いでしょうか。

ご存知かもしれませんが、GitHubの賛成票については以下の内容です。

スクリーンショット 2024-08-19 11 52 44
machida commented 3 months ago

@reckyy 機能の簡略化は無しでいきましょう。というのも、今より機能を減らして喜ぶのは開発者だけですが、機能やサービスはユーザーのためにあるからです。今とそっくりそのまま実装してください。難しいところが出てきたら、都度相談してください。

Anawerの一覧に関しては、現在は数件ごとに読み込みをさせる方式ですが、そこはpagerでもいいと考えています。

reckyy commented 3 months ago

@machida 承知しました! 引き続きよろしくお願いいたします。 🙇

reckyy commented 3 months ago

📝メモ

answers.vueを内部で使用している、question-answers.vue 同じくquestion-answers.vueの内部で使用されている commentPlaceholder.vue ai_answer.vue これらも同様に非Vue化する。

reckyy commented 2 months ago

@machida

お疲れ様です! 一点確認させていただきたいことがあります。

Anawerの一覧に関しては、現在は数件ごとに読み込みをさせる方式ですが、そこはpagerでもいいと考えています

この数件ごとに読み込みをさせる方式というのは、提出物のコメントで見られる以下の部分のことでしょうか。

スクリーンショット 2024-08-25 12 39 27

もし上記の認識が正しければ、今回非Vue化するファイルにはこの方式が適用されていなかったため、回答を全てそのまま羅列する形式でも問題ないでしょうか。(現在の仕様と同様、ひたすら下にスクロールする形) 画像の青の部分です。 image https://github.com/fjordllc/bootcamp/pull/7439 より引用

よろしくお願いいたします。

machida commented 2 months ago

その認識で正しいです! 読み込み方式を使わずにanswerを羅列する場合、answerが多いとスマホのSafariでエラーが出るので、読み込み方式でなく、ページャーでもいいので、一度に表示する件数の上限を15くらいに制限を入れていただきたいです。

machida commented 2 months ago

IMG_2862 こういうエラーが出ない様にしたいです。

reckyy commented 2 months ago

@machida ご返信ありがとうございます!

新たにpagerの処理も追加する旨、承知しました 💪 以下、新規回答フォームに関してです。もし変更した方がよければ、教えていただけますと幸いです。

もし回答数が15件以上あり、回答のページが複数あった場合、新規回答フォームは最後のページにのみ表示されるようにしようと思っています。

スクリーンショット_2024-08-26_16_14_27 スクリーンショット_2024-08-26_16_14_32
reckyy commented 2 months ago

@machida

もし回答数が15件以上あり、回答のページが複数あった場合、新規回答フォームは最後のページにのみ表示されるようにしようと思っています。

と書きましたが、やっぱり各ページに新規回答フォームは配置しようと思います。

machida commented 2 months ago

@reckyy 了解です!

やっぱり各ページに新規回答フォームは配置しようと思います。

そのようにお願いしますー

github-actions[bot] commented 3 weeks ago

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。