Bug 470142

Summary:	Repeated "Fingerprint not detected" text when fingerprint is not recognized on lock screen
Product:	[Plasma] plasmashell	Reporter:	Henning <boredsquirrel>
Component:	Theme - Breeze	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	minor	CC:	nate, postix, visual-bugs-null
Priority:	NOR
Version First Reported In:	5.27.5
Target Milestone:	1.0
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/plasma-workspace/-/commit/bfcc8b7f62310b06827c59dbbc0deb7f486a3833	Version Fixed/Implemented In:	6.0
Sentry Crash Report:

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