search

cinnyapp / cinny

Yet another matrix client
https://cinny.in
GNU Affero General Public License v3.0
1.87k stars 241 forks source link

Encryption isn't indicated #1128

Open notramo opened 1 year ago

notramo commented 1 year ago

Describe the bug

UX should indicate encryption better. It's difficult to check encryption status for a room.

Reproduction

No response

Expected behavior

No response

Platform and versions

Platform independent.

Additional context

No response

LinuxinaBit commented 1 year ago

Would something similar to FluffyChat’s lock icon work well?

kfiven commented 1 year ago

There is a shield icon into message input if room has encryption enabled.

LinuxinaBit commented 1 year ago

Should there be an indicator if the room isn’t encrypted?

kfiven commented 1 year ago

If there's no shield icon, isn't that obvious?

LinuxinaBit commented 1 year ago

Yes, I guess that works… It just seems better to have a warning that the room isn’t encrypted, so privacy, in a way, becomes the default.

tezlm commented 1 year ago

It would be nice to have the input placeholder text change from "Send a message" to "Send an encrypted message" and add a tooltip to the "encrypted room" icon. Also, using a lock for both encryption and room visibility might be confusing, but there isn't a better way I can think of....

LinuxinaBit commented 1 year ago

A shield icon?

jasikpark commented 1 year ago

https://github.com/cinnyapp/cinny/blob/da92ce3a46ab4f702f518c728437b1fb574ef0fc/src/app/organisms/room/RoomViewInput.jsx#L369

I wonder if a tooltip for the icon could be an improvement? not a solution for this discussion, but I think it'd be helpful?

K4LCIFER commented 1 year ago

Perhaps it would also be a good idea to modify the text box's placeholder text to say "Send a secure message..." instead of "Send a message...".