Skip to content

Face Library: Only handle click on card when selection mode is ongoing #17493

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

Merged
merged 1 commit into from
Apr 1, 2025

Conversation

NickM-27
Copy link
Collaborator

@NickM-27 NickM-27 commented Apr 1, 2025

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code
  • Documentation Update

Checklist

  • The code change is tested and works locally.
  • Local tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • UI changes including text have used i18n keys and have been added to the en locale.
  • The code has been formatted using Ruff (ruff format frigate)

Copy link

netlify bot commented Apr 1, 2025

Deploy Preview for frigate-docs ready!

Name Link
🔨 Latest commit db1b632
🔍 Latest deploy log https://app.netlify.com/sites/frigate-docs/deploys/67ec62a410314e000887bdf2
😎 Deploy Preview https://deploy-preview-17493--frigate-docs.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.

@hawkeye217 hawkeye217 merged commit 907283b into dev Apr 1, 2025
12 checks passed
@hawkeye217 hawkeye217 deleted the face-ui-fixes branch April 1, 2025 22:23
@pyrodex
Copy link

pyrodex commented Apr 2, 2025

Thank you for this commit! This did fix the box popping up on but it may have regressed the details page functionality. Right now clicking on a face doesn't even show the details box anymore and I've tried this on other browsers and even in incognito mode to ensure no caching is causing it.

@NickM-27
Copy link
Collaborator Author

NickM-27 commented Apr 2, 2025

I am not able to reproduce this, are you clicking on the image? keep in mind this is only for the train tab not the individual persons tab

@pyrodex
Copy link

pyrodex commented Apr 2, 2025

I am not able to reproduce this, are you clicking on the image? keep in mind this is only for the train tab not the individual persons tab

So it's even weirder! Some events show the details and some don't!

I had a couple of events generated after the patch applied and some before the patch. There is a mixture of ones that work or don't work regardless of patch application. When I look at dev tools I don't see any errors EXCEPT when clicking on one that works and clicking off the details page to go back to the face page. Other than that DEV console is clean.

thumbnail.jpg:1 
            
            
           GET https://frigate.domain.com/api/events/undefined/thumbnail.jpg 404 (Not Found)
Image
ae @ i18n-D_62mOUA.js:33
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
hr @ i18n-D_62mOUA.js:40
hf @ i18n-D_62mOUA.js:40
wg @ i18n-D_62mOUA.js:40
Gs @ i18n-D_62mOUA.js:40
Nh @ i18n-D_62mOUA.js:40
is @ i18n-D_62mOUA.js:38
Zs @ i18n-D_62mOUA.js:40
Ir.flushSync @ i18n-D_62mOUA.js:40
bT @ i18n-D_62mOUA.js:79
w0 @ main-CifDm8Gd.js:4
s @ main-CifDm8Gd.js:4
i @ main-CifDm8Gd.js:4

@NickM-27
Copy link
Collaborator Author

NickM-27 commented Apr 2, 2025

perhaps a face was detected but an event wasn't saved

@pyrodex
Copy link

pyrodex commented Apr 2, 2025

perhaps a face was detected but an event wasn't saved

Even further narrowing down has shown that if the Face is "UNKNOWN" none of the detail boxes work but if the face is known and grouped together all those pictures show the dialog box.

So in this example:

image

All the ones grouped together with the known person name all works. However the ones in the group of Unknown none of them work even the ones where the name guess is good and even the unknown one.

@NickM-27
Copy link
Collaborator Author

NickM-27 commented Apr 2, 2025

unknown works for me as well, if I had to guess this just means the second one does not have an event saved

@pyrodex
Copy link

pyrodex commented Apr 2, 2025

Ok, thanks for the feedback. I can monitor it.

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.

3 participants