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

refs #23 検索仮実装 #67

Merged
merged 8 commits into from
Dec 21, 2021

Conversation

hmatsu47
Copy link
Collaborator

@hmatsu47 hmatsu47 commented Dec 20, 2021

👏 解決する issue / Resolved Issues

📝 関連する issue / Related Issues

⛏ 変更内容 / Details of Changes

  • 検索一覧画面に該当する Q&A をすべて一覧表示する(そこからカテゴリ別のページには遷移しないで、その場で対象の Q&A を表示する)形の検索を実装しました
  • 検索窓およびハッシュフラグメント経由の検索は、こちらの実装を利用させていただきました(検索本体は自前)

📸 スクリーンショット / Screenshots

  • トップページ

image

  • 検索結果(兼検索)ページ
    • 最初から A を展開しておく方法がわからず…(展開しても鬱陶しいだけかもしれない)

image

  • 検索ページの下に前後の遷移が出てしまうのがイケてないですが…。

image

@hmatsu47
Copy link
Collaborator Author

トップページの検索窓のデザインを元に戻してみました。
(ちょっとウザい?)
image

スマホ
image

検索結果のページは変えられていません…(小さい枠のままでいいと思うけどルーペのアイコン(ti-search)ぐらいは出したかった。CSS ナニモワカラナイ)。
image

同じ方法でカテゴリ別ページへの実装もできるはずです。

なお、今のところ複数キーワードの検索には対応していません(空白でスプリットして配列化したキーワードを AND または OR 検索すれば良いと思うけれど、あえてやっていない)。

Copy link
Member

@ww2or3ww ww2or3ww left a comment

Choose a reason for hiding this comment

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

@hmatsu47 ありがとうございます!
1点コメントさせていただきました。ご確認おねがいします。

あと、検索ボックスの下に「送信」という文字が出てしまっている?のが気になりました。
image

</form>

<script>
const SEARCH_URL = '/vaccinecert-faq/search/';
Copy link
Member

Choose a reason for hiding this comment

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

/vaccinecert-faq 要らないのでは?

@hmatsu47
Copy link
Collaborator Author

@ww2or3ww
ありがとうございます!

/vaccinecert-faq 要らないのでは?

うちの環境の基準ディレクトリがずれているようです。修正しました。

あと、検索ボックスの下に「送信」という文字が出てしまっている?のが気になりました。

こちらの環境では(Chrome・Safari とも)表示されていないので確認できませんでしたが、文字色も透明にしてみました。
消えたでしょうか?
(一応 value に「送信」を設定して消えるところまでは確認しました。このボタン、Chrome は非表示にするだけで良いのですが、Safari で非表示にしてしまうと Submit 時の処理が走らなくなってしまうので、透明で表示しています)

@@ -0,0 +1,36 @@
---
title: "全てのカテゴリから検索"
Copy link
Member

Choose a reason for hiding this comment

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

ページタイトルですが、

ワクチン接種証明書アプリFAQ - 全てのカテゴリから検索

のほうがいいかな?と思いましたがどうでしょうか。

Choose a reason for hiding this comment

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

LGMT

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

どうでしょうね?
ヘッダに書かれている分冗長になるかな、とも思ったのですが。

Copy link
Member

Choose a reason for hiding this comment

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

https://test.d2qv7xiwv4i3gl.amplifyapp.com/faq1/
この辺のタイトルもアプリ名称ないので、まぁ、いいか。

もしやるにしても、別チケットにしましょう!

@ww2or3ww ww2or3ww self-requested a review December 21, 2021 12:39
Copy link
Member

@ww2or3ww ww2or3ww left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます。
検索できること、送信という文字が見えなくなっていること、を確認できました!

@ww2or3ww ww2or3ww requested a review from poba1031 December 21, 2021 12:41
Copy link
Collaborator

@euledge euledge left a comment

Choose a reason for hiding this comment

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

LGTM

@euledge euledge merged commit 0abd704 into code-for-hamamatsu:main Dec 21, 2021
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.

4 participants