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.

Sign up for GitHub

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

Jump to bottom

ユーザー編集画面の所属企業にインクリメンタルサーチを使えるようにする #4385

Merged
merged 4 commits into from
Mar 19, 2022
Merged

ユーザー編集画面の所属企業にインクリメンタルサーチを使えるようにする #4385

merged 4 commits into from
Mar 19, 2022

Conversation

garammasala29
Copy link
Contributor

@garammasala29 garammasala29 commented Mar 10, 2022 •
edited
Loading

Issue

概要

Choices-jsを使用し、ユーザー編集画面の所属企業をインクリメンタルサーチできるようにしました

変更確認方法

  1. ブランチ feature/changed-select-box-for-affiliated-companies-to-incremental-searchをローカルに取り込む
  2. bin/setupを実行する
  3. rails s でローカル環境を立ち上げ、管理者でログイン(komagataで確認しました)
  4. ユーザー個別ページ>管理者として情報変更、「企業」から今回の変更箇所が確認できます。

デモ

default.mov

備考

Choices.jsのセレクトボックスのオプションに関しましては、https://discord.com/channels/715806612824260640/809595476847493192/952741097531142174
で町田さんに確認しています。

@garammasala29 garammasala29 force-pushed the feature/changed-select-box-for-affiliated-companies-to-incremental-search branch 2 times, most recently from 3b6dc3f to a810eff Compare March 11, 2022 00:54
@garammasala29
Copy link
Contributor Author

@machida さん
お疲れ様です。お手隙の際に、こちらのデザインをお願いします🙏

@machida machida removed their assignment Mar 15, 2022
@machida
Copy link
Member

machida commented Mar 15, 2022

@garammasala29 デザイン調整をしましたー

@garammasala29
Copy link
Contributor Author

@machida さん
ありがとうございます！cssでの読み込む方法、勉強になりました🙏

@garammasala29 garammasala29 marked this pull request as ready for review March 16, 2022 02:06
@garammasala29 garammasala29 requested a review from saeyama March 16, 2022 02:14
@garammasala29
Copy link
Contributor Author

@saeyama さん
お疲れ様です。お手隙の際にこちらのレビューをお願いいたします🙏
新たなパッケージを追加しましたので、そちらの使い方を読み解くのにお時間を頂くかもしれません。
不明点等ありましたら気兼ねなく連絡いただけたらと思います🙇‍♂️

saeyama
saeyama reviewed Mar 17, 2022
View reviewed changes
Copy link
Contributor

@saeyama saeyama left a comment

Choose a reason for hiding this comment

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

@garammasala29 さん、レビュー依頼ありがとうございます！
返信遅くなって申し訳ありません🙇‍♀️
choices.js、CDNの読み込み方、とても勉強になります…！
管理者でログインしデモ通りの動作確認が出来ました！

気になった点が1点あります🙏
全角で該当しない文字を1文字入力すると一致する情報は見つかりませんと出るのですが、半角英数字を入力した場合、何文字か入力をしないと一致する情報は見つかりませんと表示されないのが気になりました。

const choicesのsearchResultLimit: 10を1に変えれば良いのかな〜と思ったのですが、下記と同じでした。
choices.js-npmのoptionで何かないかな〜と見たのですが、ちょっと見つけられなかったです。。

該当しない全角
(全角1文字目で一致する情報は見つかりません)
image

該当しない半角数字
(10文字)
image

(11文字で一致する情報は見つかりません)
image

該当しない半角英字
(aから始まる企業はない)
image

image

ただ、検索しなくても、該当企業があれば選択できるので、そこまで気にすることもないような気もします。
上記の点、駒形さん、町田さんに確認して、特に問題なければ、チームリーダーレビューへ進めて頂いて良いと思います…！

よろしくお願いします🙇‍♀️

@garammasala29
Copy link
Contributor Author

@saeyama さん
ご確認ありがとうございます。choices.jsをしっかり読み込んでいただいたようで、お時間いただき本当にありがとうございます🙏
半角英数字を入力した場合なんですが、
image
の画像でお伝えしますと、aが入っている企業が全て検索される仕様にchoices.jsがなっているようです😄
したがって、問題ないと判断してチームリーダーレビューへ進めさせていただきますね！
今後もどうぞよろしくお願いします🙇‍♂️

@garammasala29 garammasala29 requested a review from komagata March 17, 2022 21:36
@garammasala29
Copy link
Contributor Author

@komagata さん
お疲れ様です。お手隙の際にこちらのレビューをよろしくお願いします🙏

@saeyama saeyama self-requested a review March 18, 2022 11:57
saeyama
saeyama approved these changes Mar 18, 2022
View reviewed changes
Copy link
Contributor

@saeyama saeyama left a comment

Choose a reason for hiding this comment

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

@garammasala29 さん、了解しました！
もしかしたら不要だったかもしれませんが、Approveさせて頂きました！
こちらこそ今後ともよろしくお願いします🙏

komagata
komagata reviewed Mar 18, 2022
View reviewed changes
app/javascript/company-select.js Outdated
noResultsText: '一致する情報は見つかりません',
itemSelectText: '選択'
})
return choices
Copy link
Member

Choose a reason for hiding this comment

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

他に何もしないのであれば1行目でreturnでもいいかもです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

変数に代入して作業していた名残からでした💦
修正致しました→9e6511e

@garammasala29 garammasala29 force-pushed the feature/changed-select-box-for-affiliated-companies-to-incremental-search branch from a95a212 to 9e6511e Compare March 18, 2022 22:07
@garammasala29
Copy link
Contributor Author

@komagata さん
お疲れ様です。修正致しましたので、お手隙の際にご確認よろしくお願いします🙏

komagata
komagata approved these changes Mar 19, 2022
View reviewed changes
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.

確認しました、OKですー🙆‍♂️

@komagata komagata merged commit fd211e5 into main Mar 19, 2022
@komagata komagata deleted the feature/changed-select-box-for-affiliated-companies-to-incremental-search branch March 19, 2022 06:18
@github-actions github-actions bot mentioned this pull request Mar 19, 2022
Merged
17 tasks
This was referenced Mar 24, 2022
Merged
Closed
@saeyama saeyama mentioned this pull request Apr 11, 2022
Merged
@saeyama saeyama mentioned this pull request Apr 27, 2022
Merged
This was referenced May 2, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@saeyama saeyama saeyama approved these changes

@komagata komagata komagata approved these changes

Assignees

@garammasala29 garammasala29

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@garammasala29 @machida @komagata @saeyama