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

Add a better search UI for patients index page(#8691) #8818

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

i0am0arunava
Copy link

@i0am0arunava i0am0arunava commented Oct 17, 2024


Pull Request: Smart Search Component Implementation

Description

This PR introduces a smart search component to enhance user experience on the patient page. It consolidates multiple search options into a single, user-friendly interface, adds keyboard shortcuts for improved accessibility, and adjusts the count box height for visual consistency.

Changes Made

Smart Search Functionality:

  • Implemented a unified search component allowing users to select their desired search criteria from the following options:

    • Name
    • Phone Number
    • UHID Number
  • The user can easily switch between search options, streamlining the search process.

Keyboard Shortcuts:

  • Added a shortcut: Ctrl + '/' to open a search palette.
  • Users can navigate through the options using the arrow keys (up and down).
  • Pressing Enter allows users to select the highlighted search option, making the search process more efficient.

Count Box Height Adjustment:

  • Reduced the height of the count box to match the new smart search component, ensuring a cohesive look and feel throughout the UI.

Screenshot 2024-10-17 230010

@i0am0arunava i0am0arunava requested a review from a team as a code owner October 17, 2024 17:31
Copy link

netlify bot commented Oct 17, 2024

Deploy Preview for care-ohc ready!

Name Link
🔨 Latest commit 3a26f5b
🔍 Latest deploy log https://app.netlify.com/sites/care-ohc/deploys/671274a4e4626f000812005f
😎 Deploy Preview https://deploy-preview-8818--care-ohc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the Deploy-Failed Deplyment is not showing preview label Oct 17, 2024
@rithviknishad rithviknishad removed the Deploy-Failed Deplyment is not showing preview label Oct 18, 2024
@Jacobjeevan
Copy link
Contributor

@i0am0arunava make sure to follow the PR format we have - proposed changes that mentions the issue # and merge checklist :)

@i0am0arunava
Copy link
Author

@i0am0arunava make sure to follow the PR format we have - proposed changes that mentions the issue # and merge checklist :)

okay I am doing

@bodhish
Copy link
Member

bodhish commented Oct 18, 2024

This is not what we want, please check the comments in #8725 (review)

@i0am0arunava
Copy link
Author

i0am0arunava commented Oct 18, 2024

This is not what we want, please check the comments in #8725 (review)

okay,I am making the changes as required.

This merge brings in the latest changes from the upstream develop branch, ensuring
that my feature branch is up to date with the latest updates and fixes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants