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

検索中のページタイトル・ヘッダーに検索情報を付加した #815

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

yonta
Copy link
Collaborator

@yonta yonta commented Dec 24, 2024

close #766

やったこと

  • 酒indexで検索時のページタイトルとヘッダに検索情報を付加
  • TurboFrameの範囲を拡大
    • 上記検索情報があるページタイトルもTurboFrameに含めるようにした
  • テストを追加・更新

変更後スクリーンショット

image

変更前スクリーンショット

image

Copy link

github-actions bot commented Dec 24, 2024

📦 esbuild Bundle Analysis for sakazuki

This analysis was generated by esbuild-bundle-analyzer. 🤖
This PR introduced no changes to the esbuild bundle! 🙌

今までは検索情報をページタイトルやヘッダにいれていなかった。
検索しているかしないかに関わらず、常に酒量を表示していた。
そのため、ブラウザバックリストが分かりづらかった。
また何を検索しているかや何件ヒットしたかの情報がなく、検索フィードバックがなかった。

このコミットによりヘッダ情報に検索単語を付加した。
ブラウザヒストリに検索単語が表示されるようになる。
ただしTurboの制限により、リロードをしたときのみである。 Issue #629

また、ページタイトルに検索ワードとヒット件数を入れるようにした。
@yonta yonta force-pushed the pr/issue766/search-title branch from e1dea17 to a93d8f0 Compare December 26, 2024 06:41
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.

検索結果のフィードバックが弱い
1 participant