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

feat: add fprint support #862

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TitouanReal
Copy link

@TitouanReal TitouanReal commented Jan 6, 2025

Adds basic fingerprint support:

Capture d’écran du 2025-01-08 21-53-52

Capture d’écran du 2025-01-08 21-52-06

I would like to finish this work and allow users to add and remove fingerprints. To continue, I would like official designs to follow.

Technical info useful for design:

  • There can be multiple fprint readers.
  • It is possible for zero or one of the readers to be marked as "Default".
  • Adding a fprint requires its name (right-thumb, left-ring-finger, ...)

For more info, see the doc.

@TitouanReal
Copy link
Author

I have seen you put DBus bindinds in here. I believe you would prefer to have a fprint crate in there and use the crate in cosmic-settings.

@mmstick
Copy link
Member

mmstick commented Jan 6, 2025

Needs input from @maria-komarova then

@mmstick mmstick requested review from a team January 6, 2025 15:18
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