Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs新規作成フォームのプラクティス選択をChoices.jsに置き換える #7508

Conversation

a-terumoto-gs
Copy link
Contributor

@a-terumoto-gs a-terumoto-gs commented Mar 8, 2024

Issue

概要

select2というjQueryライブラリで実装されていたDocs作成フォームのプラクティス選択部分のセレクトボックスを、
他ページでのプラクティス選択部分の実装に合わせ、Choices.jsというライブラリでの実装に変更しました。

関連issue

変更確認方法

  1. feature/replace-practice-selection-in-creation-new-docs-form-with-choices-jsをローカルに取り込む
  2. http://localhost:3000/pages にアクセス
  3. 右上の+Doc作成を押す
  4. 「関連プラクティス」欄について、仕様が以下のように変わっていることを確認する
  • デフォルトとして「関連プラクティスを指定しない」という文字列が「関連プラクティス」欄にグレーで表示されている
    image

  • プルダウンをクリックして、一覧の中でhoverしたプラクティスの欄がグレーになり、右側に「選択」とグレーで表示される

image

  • プラクティスを一つ選択すると「関連プラクティス」欄に黒字で表示される
    image

Screenshot

変更前

デフォルトの「関連プラクティスを指定しない」という文字列が黒
hoverするとプラクティスの欄が濃い紫に
選択済みのプラクティスは濃いグレーに
Image from Gyazo

変更後

デフォルトの「関連プラクティスを指定しない」という文字列がグレー
hoverするとプラクティスの欄がグレーに、かつ右側に「選択」と表示
Image from Gyazo

@a-terumoto-gs a-terumoto-gs self-assigned this Mar 8, 2024
@a-terumoto-gs
Copy link
Contributor Author

@machida
おつかれさまです!
こちらのissueのデザインについてなのですが、以下のissueで実装された他ページのものを流用して実装しています。
一応確認をお願いしてもよろしいでしょうか?

@machida
Copy link
Member

machida commented Mar 25, 2024

@a-terumoto-gs 確認しました!!このデザインでOKですー👍

@a-terumoto-gs a-terumoto-gs force-pushed the feature/replace-practice-selection-in-creation-new-docs-form-with-choices-js branch from be0ab58 to e1b862a Compare March 25, 2024 08:33
@a-terumoto-gs a-terumoto-gs marked this pull request as ready for review March 25, 2024 08:52
@a-terumoto-gs
Copy link
Contributor Author

a-terumoto-gs commented Mar 25, 2024

@kurumadaisuke
お疲れ様です!
お手すきの際にこちらのPRのレビューをお願いしてもよろしいでしょうか?
お忙しい等あれば遠慮なくその旨お伝えください。
よろしくお願いいたしますm(__)m

@kurumadaisuke
Copy link
Contributor

@a-terumoto-gs
すいません....
ちょっと仕事がバタバタしていてレビューまで、お時間空いてしまっても大丈夫であれば確認いたします🙇‍♂️
もし、急ぎ等あれば申し訳ないのですが違う方への依頼お願いできますでしょうか🙇‍♂️

@a-terumoto-gs a-terumoto-gs requested review from ham-cap and removed request for kurumadaisuke March 26, 2024 02:38
@a-terumoto-gs
Copy link
Contributor Author

@kurumadaisuke
コメントありがとうございます!
お忙しいところお手数おかけして申し訳ないです💦
今回は他の方にお願いさせてもらおうと思いますm(__)m

@a-terumoto-gs
Copy link
Contributor Author

@ham-cap
お疲れ様です!
お手すきの際にこちらのPRのレビューをお願いしてもよろしいでしょうか?
お忙しい等あれば遠慮なくその旨お伝えください。
よろしくお願いいたしますm(__)m

@ham-cap
Copy link
Contributor

ham-cap commented Mar 29, 2024

@a-terumoto-gs
返信が遅くなり申し訳ありません🙏
コンフリクトが起きているようですので、まずそちらを解消していただけますでしょうか👀
image
よろしくお願いいたします🙏

@a-terumoto-gs a-terumoto-gs force-pushed the feature/replace-practice-selection-in-creation-new-docs-form-with-choices-js branch from e1b862a to 8305434 Compare April 3, 2024 06:40
@a-terumoto-gs
Copy link
Contributor Author

@ham-cap
対応遅くなってしまい申し訳ございません。
コンフリクト解消いたしましたm(__)m
レビューお願いいたします!

@a-terumoto-gs
Copy link
Contributor Author

@ham-cap
お忙しいところ失礼しますm(__)m
こちらいかがでしょうか?

@a-terumoto-gs a-terumoto-gs force-pushed the feature/replace-practice-selection-in-creation-new-docs-form-with-choices-js branch from 8305434 to d8a1640 Compare April 18, 2024 06:38
@ham-cap
Copy link
Contributor

ham-cap commented Apr 18, 2024

@a-terumoto-gs
遅くなっており申し訳ありません🙇
明日中には対応させていただきますのでもう少しだけお時間いただければ幸いです🙏💦

Copy link
Contributor

@ham-cap ham-cap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@a-terumoto-gs
お疲れ様です!
ローカルでの動作確認とコードの確認をさせていただきました!
問題ないと思いますのでApproveさせていただきますー🙆‍♂️

@a-terumoto-gs
Copy link
Contributor Author

@ham-cap
ありがとうございました(*'▽')

@komagata
メンバーの方にApproveいただいたのでレビューをお願いいたしますm(__)m

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このIssueはとっても助かります~!
確認させていただきました。OKです~👌

@komagata komagata merged commit 212bb69 into main Apr 24, 2024
3 checks passed
@komagata komagata deleted the feature/replace-practice-selection-in-creation-new-docs-form-with-choices-js branch April 24, 2024 02:47
@komagata komagata mentioned this pull request Apr 24, 2024
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants