Bug 510421 - [Session Verification] Unclear dialog: success or failure?
Summary: [Session Verification] Unclear dialog: success or failure?
Status: RESOLVED FIXED
Alias: None
Product: NeoChat
Classification: Applications
Component: General (other bugs)
Version First Reported In: 25.08.1
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Tobias Fella
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-10-09 09:58 UTC by David REVOY
Modified: 2025-10-28 13:51 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 24.08.3
Sentry Crash Report:


Attachments
^ screenshot of the dialog (19.58 KB, image/jpeg)
2025-10-09 09:58 UTC, David REVOY
Details

Note You need to log in before you can comment on or make changes to this bug.
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!