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

Open
bodhish opened this issue Oct 2, 2024 · 8 comments · May be fixed by #8834
Open

Add a better search UI for patients index page #8691

bodhish opened this issue Oct 2, 2024 · 8 comments · May be fixed by #8834

Comments

@bodhish
Copy link
Member

bodhish commented Oct 2, 2024

In the current patient page, we have 4 independet search on patient, at any point in time you can only search by one practically.
Here is an how the current UI looks.
image

Let upgrade it with a smart search compoent where the user can select what to search for.
image

It should also support keyboard short cuts.
image

The valid options would be Name | Phone Number | OP/IP number | Emergency Contact Number.

Also reduce the height of the count box to match the component.

@Jash2606
Copy link
Contributor

Jash2606 commented Oct 3, 2024

Hey @bodhish @rithviknishad ,
I'd like to contribute to this issue for Hacktoberfest . Could you please assign it to me?

@Jash2606
Copy link
Contributor

Jash2606 commented Oct 5, 2024

Hey @rithviknishad ,
I had created a Updated UI for search bar , Could you review it and Please let me know if any further changes required !

@i0am0arunava
Copy link

@bodhish @rithviknishad, I'd like to contribute to this issue. Could you please assign this to me?

@i0am0arunava
Copy link

i0am0arunava commented Oct 17, 2024

I have started to work on it.

@i0am0arunava
Copy link

@bodhish @rithviknishad @Jacobjeevan
,I have completed the work. If there are any issues , please provide feedback. Pressing Ctrl + '/' will open a palette, and users can navigate through the options using the arrow keys on the keyboard. Additionally, also I have reduce the height of the count box to match the component. If everything looks good, please let me know before I make the pull request. Is there any other process I need to complete?
here is my work Image

@bodhish
Copy link
Member Author

bodhish commented Oct 17, 2024

@i0am0arunava without seeing the code its hard to give feedback; In general try to make it in sync with design

@i0am0arunava
Copy link

i0am0arunava commented Oct 17, 2024

@bodhish @rithviknishad @Jacobjeevan,I have raised a pull request; please review it and provide feedback.

@i0am0arunava
Copy link

@bodhish @rithviknishad @Jacobjeevan,I accidentally deleted the forked repo. I have cleaned up the unrelated changes and raised a new pr please review it .#8834

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
4 participants