-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
fix(web): search people #8955
fix(web): search people #8955
Conversation
Deploying immich with
|
Latest commit: |
e16bfcd
|
Status: | ✅ Deploy successful! |
Preview URL: | https://89565f1f.immich.pages.dev |
Branch Preview URL: | https://fix-search-people.immich.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haven't tested it out yet but the code looks good so far
9f71ed4
to
4b46873
Compare
Any idea why I'm losing focus whenever I type a single character? Screen.Recording.2024-04-22.at.20.44.23.mov |
It was fixed on |
Co-authored-by: Daniel Dietzler <36593685+danieldietzler@users.noreply.github.com>
4b46873
to
e16bfcd
Compare
That was it, thank you Alex! However it still loses focus when the search field becomes empty. Is that intentional? |
Probably not, I think it is a "bug" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Potential fix for #8951.
Show people searched with the
searchPerson
API instead of relying on people returned by thegetAllPeople
API.Screenshots