Bug 470142 - Repeated "Fingerprint not detected" text when fingerprint is not recognized on lock screen
Summary: Repeated "Fingerprint not detected" text when fingerprint is not recognized o...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Theme - Breeze (other bugs)
Version First Reported In: 5.27.5
Platform: Fedora RPMs Linux
: NOR minor
Target Milestone: 1.0
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-22 19:03 UTC by Henning
Modified: 2023-09-20 16:36 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Henning 2023-05-22 19:03:14 UTC
SUMMARY
1. Locked screen. I swipe a bit on the touchpad, the password prompt opens.
2. I hold my finger on the fingerprint reader (working). 
3. The fingerprint is not detected, failing in a pretty high interval
4. The area below the password field fills up with "Could not detect Fingerprint" messages on top of each other

OBSERVED RESULT
cluttered area, overlaying important things.

EXPECTED RESULT
Only one error message should be shown.

0. Fingerprint not detected
1. Error message appears
2. Trying again, fingerprint not detected again
3. Previous error message is cleared
4. New error message appears

This could also be animated, but not important. Just clear the recent error messages to avoid this


Specified App:
kscreenlocker-5.27.5-1.fc38.x86_64

--- Software ---
OS: Fedora Linux 38.20230522.0 (Kinoite)
KDE Plasma: 5.27.5
KDE Frameworks: 5.106.0
Qt: 5.15.9
Kernel: 6.2.15-300.fc38.x86_64
Compositor: wayland
Comment 1 Bug Janitor Service 2023-09-03 17:00:36 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/3247
Comment 2 Henning 2023-09-03 17:43:40 UTC
awesome! funny how easy that fix was
Comment 3 Nate Graham 2023-09-20 16:36:08 UTC
Git commit bfcc8b7f62310b06827c59dbbc0deb7f486a3833 by Nate Graham.
Committed on 20/09/2023 at 18:27.
Pushed by ngraham into branch 'master'.

lockscreen: Bounce on repeated identical messages

Currently repeated messages from the backend stack up, which presents a
rather confusing and redundant UX.

Instead, detect when the existing message text already contains the
newly-sent text, and if it does, make it do a fancy little bounce
instead of appending the same text.
FIXED-IN: 6.0

M  +25   -0    lookandfeel/org.kde.breeze/contents/components/SessionManagementScreen.qml
M  +1    -0    lookandfeel/org.kde.breeze/contents/lockscreen/LockScreen.qml
M  +15   -13   lookandfeel/org.kde.breeze/contents/lockscreen/LockScreenUi.qml
M  +4    -0    lookandfeel/org.kde.breeze/contents/lockscreen/MainBlock.qml

https://invent.kde.org/plasma/plasma-workspace/-/commit/bfcc8b7f62310b06827c59dbbc0deb7f486a3833