Bug 510421

Summary: [Session Verification] Unclear dialog: success or failure?
Product: [Applications] NeoChat Reporter: David REVOY <info>
Component: GeneralAssignee: Tobias Fella <fella>
Status: RESOLVED FIXED    
Severity: normal CC: carl, josh
Priority: NOR    
Version First Reported In: 25.08.1   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 24.08.3
Sentry Crash Report:
Attachments: ^ screenshot of the dialog

Description David REVOY 2025-10-09 09:58:49 UTC
Created attachment 185622 [details]
^ screenshot of the dialog

Hi,

## ISSUE:
After opening NeoChat, I got a session verification dialog windows (screenshot in attachment) but it left me confused: the text felt positive "The session was accepted on a different device", while the icon is a big red shield centered with a X in white on the middle. Text and icon don't convey the same feeling, and I felt like "Is it ok? Is it wrong?" but I had no guidance to know what to do.

## EXPECTED:
- If it is intended to be positive: replace the icon with a green shield.
- If something went wrong and the icon is the intended one: get a text that guide me on how to fix the issue (and feels less positive).
If the icon is just a placeholder or a "missing icon filler" (like for 404 broken images link): embed the missing the icon.

 ## INFO
KDE Plasma Debian 12, 5.27.5, Qt 5.15.8
NeoChat latest via Flatpak
Comment 1 Bug Janitor Service 2025-10-27 22:55:49 UTC
A possibly relevant merge request was started @ https://invent.kde.org/network/neochat/-/merge_requests/2519
Comment 2 Joshua Goins 2025-10-28 13:16:25 UTC
Git commit be92e56c3a1b70e75ae78f078519a800e2af7730 by Joshua Goins.
Committed on 28/10/2025 at 13:16.
Pushed by redstrate into branch 'master'.

Show less scary icon for neutral reasons when cancelling verification

For example, accepting a verification on another device shows a giant
red security icon which isn't really suitable. I chucked the
dialog-information icon in for some of the neutral-sounding messages so
this dialog can be a little less intimidating.
FIXED-IN: 24.08.3

M  +13   -1    src/app/qml/VerificationCanceled.qml

https://invent.kde.org/network/neochat/-/commit/be92e56c3a1b70e75ae78f078519a800e2af7730
Comment 3 Joshua Goins 2025-10-28 13:48:36 UTC
Git commit 0b9295e67e53da4a8f00bda934dbfe7163be41f2 by Joshua Goins.
Committed on 28/10/2025 at 13:16.
Pushed by redstrate into branch 'release/25.08'.

Show less scary icon for neutral reasons when cancelling verification

For example, accepting a verification on another device shows a giant
red security icon which isn't really suitable. I chucked the
dialog-information icon in for some of the neutral-sounding messages so
this dialog can be a little less intimidating.
FIXED-IN: 24.08.3


(cherry picked from commit be92e56c3a1b70e75ae78f078519a800e2af7730)

Co-authored-by: Joshua Goins <josh@redstrate.com>

M  +13   -1    src/app/qml/VerificationCanceled.qml

https://invent.kde.org/network/neochat/-/commit/0b9295e67e53da4a8f00bda934dbfe7163be41f2
Comment 4 David REVOY 2025-10-28 13:51:33 UTC
Thank you very much Redstrate for fixing it!