Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] feat!: Show a ~ in front of contact names if the name wasn't explicitly set #6411

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Hocuri
Copy link
Collaborator

@Hocuri Hocuri commented Jan 6, 2025

Problem

Eve can just change her displayname to "Bob", and Alice's device will show the messages as coming from "Bob".

Or, Eve can just change her displayname to "Saved Messages", and then her chat will be shown as if it was Saved Messages, so that people may send private information to her instead of saving it to the Saved Messages.

Solution

There is a ~ in front of the name if it's just the name chosen by the sender, rather than the name with which you saved them into your contact book. This is the same behavior as WhatsApp has, so people know it already.

@hpk42 and me think that this small change will make the problem stated above a lot smaller already, although of course the ~ can always be overlooked.

This PR doesn't add any migration for 1:1 chat names. This means that, as of now, chat names will only get the ~ the next time the name changes. Not sure whether we should change this.

Future work

UIs should make it easy to set a display name, e.g. by automatically opening the contact-edit screen after scanning a QR code and by requiring not more than 2 clicks to get to the contact-edit screen from the chat (there could e.g. be a prominent button "Set a name for this contact"/"Name this contact" in the profile or in the chat. And when clicking the contact name in the profile, the edit-contact activity should also open.

BREAKING CHANGE: UIs need to stop adding a ~ to the OverrideSendername, see deltachat/deltachat-android#3520

@Hocuri Hocuri changed the title [WIP] Show a ~ in front of contact names if the name wasn't explicitly set [WIP] feat!: Show a ~ in front of contact names if the name wasn't explicitly set Jan 6, 2025
Hocuri added a commit to deltachat/deltachat-android that referenced this pull request Jan 6, 2025
Companion for
deltachat/deltachat-core-rust#6411. This PR is not
meant to change user-visible behavior, we just shifted the
responsibility to add the "~" to core. (without this PR, the name would
be "~~Alice" sometimes)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant