Skip to content

lock: enable parallel fprintd unlocks#3006

Merged
ItsLemmy merged 3 commits into
noctalia-dev:mainfrom
CLEMENTINATOR:fprintd-fix-issues
Jun 13, 2026
Merged

lock: enable parallel fprintd unlocks#3006
ItsLemmy merged 3 commits into
noctalia-dev:mainfrom
CLEMENTINATOR:fprintd-fix-issues

Conversation

@CLEMENTINATOR

Copy link
Copy Markdown
Contributor

Summary

lockscreen: parallel fingerprint and password input

Motivation

Discussion on discord. User (and I) had to press enter to unlock the session with a fingerprint reader.
Took example on hyprlock for the implementation.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Refactoring
  • Build / packaging

Related Issue

Testing

Manual Coverage

  • Tested on Niri
  • [] Tested on Hyprland
  • [] Tested on Sway
  • [] Tested on another compositor:
  • [] Tested with different bar positions and density settings
  • [] Tested at different interface scaling values
  • Tested with multiple monitors

Screenshots / Videos

Checklist

  • This PR is ready for review, or it is marked as Draft.
  • I read and followed the relevant guidance in CONTRIBUTING.md.
  • I ran just format with clang-format v22+ installed, or this PR has no code changes.
  • I ran the relevant build or test commands, or explained why they were not run.
  • I self-reviewed the changes.
  • I checked for new warnings or errors.
  • I will update end-user documentation after merge, or this PR does not change user-facing configuration or behavior.
  • I added or updated assets/translations/en.json, or this PR adds no new user-facing strings.
  • I did not edit non-English translation files unless this PR is explicitly for translation tooling, an import/export sync, or a maintainer-requested locale change.
  • I used the existing canonical names for config keys, IPC names, paths, and identifiers.

Additional Notes

Just need to test it on a non fingerprint system to make sure it doesn't break anything.

@ItsLemmy

Copy link
Copy Markdown
Collaborator

Test without finger print and it works, merging and hoping for the best.

@ItsLemmy ItsLemmy merged commit 3618cc2 into noctalia-dev:main Jun 13, 2026
2 checks passed
@ItsLemmy

Copy link
Copy Markdown
Collaborator

made a tiny change over it, let me know if that still works for you.

@CLEMENTINATOR

Copy link
Copy Markdown
Contributor Author

made a tiny change over it, let me know if that still works for you.

lkooks still good !

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