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

検索機能を拡充:自分のみのチェックボックスを追加 #8277

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

Conversation

kitarou888
Copy link

@kitarou888 kitarou888 commented Jan 15, 2025

Issue

概要

ヘッダーにある検索フォームに「自分のみ」のチェックボックスを追加し、自分が書いた日報その他のみをフィルタリングできるようにしました。

変更確認方法

  1. ブランチfeature/searchable-filtering-login-userをローカルに取り込む
  2. ローカルでアプリを起動
    1. bin/setup
    2. foreman start -f Procfile.dev
  3. ログイン
    ユーザーmachida パスワードtesttestでログイン
  4. ヘッダーの検索フォームにある「自分のみ」のチェックボックスにチェックを入れた状態で検索ボタンをクリック
  5. 自分の日報その他のみが表示されることを確認

Screenshot

変更前

変更後

@machida
Copy link
Member

machida commented Feb 3, 2025

@kitarou888

お待たせしました🙇‍♂️
検索のUIをPCのときもモーダルに変更しました。

image image

これでお願いしますー

@kitarou888 kitarou888 self-assigned this Feb 12, 2025
@kitarou888 kitarou888 force-pushed the feature/searchable-filtering-login-user branch from 359ec49 to 0c5c13b Compare February 14, 2025 13:32
@kitarou888
Copy link
Author

kitarou888 commented Feb 15, 2025

@machida
検索結果ページのタイトル表示について、私の方で案を考えてみましたので、ご意見を伺えたらと思います🙏

■ これまで

(検索ワードあり)
image

(検索ワードなし)
image

■ 改善案

(検索ワードあり、自分の投稿のみにチェックあり)
image

(検索ワードあり、自分の投稿のみにチェックなし)
image

(検索ワードなし、自分の投稿のみにチェックあり)
image

(検索ワードなし、自分の投稿のみにチェックなし)
image

@machida
Copy link
Member

machida commented Feb 15, 2025

@kitarou888 一旦それでリリースにしようと思います!別PRで人やカテゴリーを線で囲って他の文字と区別するなどの見た目のデザインをしたいと思いますー

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.

2 participants