Skip to content

new email #2770

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

Merged
merged 7 commits into from
Jul 16, 2025
Merged

new email #2770

merged 7 commits into from
Jul 16, 2025

Conversation

r10s
Copy link
Member

@r10s r10s commented Jul 12, 2025

this PR adds an top-level option to create a new email with subject - this is the first time, we officially have that :)

the PR requires a core with chatmail/core#6932

  • add 'new email' option to 'create chat' dialog
  • adapt 'new group' dialog to email wording (members -> recipients, group name -> subject, create -> continue, title)
  • list address contacts as recipients
  • call new api
  • test 

   

the last screenshot falls back to "group" wording, however, that seems okay for now. in a subsequent PR, if we see all that more in practise, we can consider to switch to "recipients" and "e-mail" also there (that would affect also incoming/existing chats, however)

the old "add contact manually" is removed on top-level by this PR - it is not a goal on its own, but part of a task the user wants to do. it is still possible to do "new contact / add contact manually", "scan qr code / add contact manually" and "new email / add recipients / add contact manually".
by the removal we have a cleaner, less cluttered UI and also in many cases, the user wants to set a subject, so "new email" is superior here

once this is merged, file issues for android/desktop

@r10s r10s changed the base branch from main to r10s/channels July 12, 2025 16:29
Base automatically changed from r10s/channels to main July 13, 2025 10:08
@r10s r10s force-pushed the r10s/new-email branch 3 times, most recently from 02392bf to c6f645b Compare July 13, 2025 12:09
@r10s r10s added wait-for-core PR/issue is waiting for core release enhancement actually in development, user visible enhancement labels Jul 13, 2025
@r10s r10s requested review from Amzd and hpk42 July 13, 2025 12:20
@adbenitez
Copy link
Member

"new email"

it is not really an email but just creating an email thread

@r10s r10s requested a review from adbenitez July 13, 2025 12:21
@r10s
Copy link
Member Author

r10s commented Jul 13, 2025

it is not really an email but just creating an email thread

sure, this is what it is internally.

so you suggest to reword to "New E-Mail Thread" in the UI? not sure if that is more or less helpful :)

eg. when thinking about german translation, it probably raises more questions than it solves

@adbenitez
Copy link
Member

adbenitez commented Jul 13, 2025

sure, this is what it is internally.

not internally but visible, you go through that steps as an unsuspecting user and didn't manage to create an email but a thread/group

so you suggest to reword to "New E-Mail Thread" in the UI?

yes that is what the action is actually doing, creating a group with classic email users which in the email world is called thread,

I think it is important to notice that this feature is not for the 99% of the users of Delta Chat but for the classic email users and will only appear to them, and they probably know what an email thread is

@link2xt
Copy link
Contributor

link2xt commented Jul 13, 2025

The option should be hidden for chatmail profiles. Don't see the code checking it now.

@r10s
Copy link
Member Author

r10s commented Jul 13, 2025

and they probably know what an email thread is

i doubt, most typical email users know. at least ppl around me, in german language, do not know :) 🪡🧵 ...

this is the typical thing of ppl answering "Re: Merry Christmas" months later without context :)

what ppl actually want to do, is to "write an e-mail" - not to start an "e-mail thread" - one could also argue that even on classic email apps, "new e-mail" is not 100% correct, as that starts a new thread as well (in contrast to "reply"). so, i think, it is of some worth to pick up the same wording as in classic email apps. that all sounds a bit nitpicky to me :)

but i would not die on that hill, i see the point, and maybe our email users are different :) let's see if more voices come up, the string anyways needs to be changed in android repo

EDIT: in some 1:1 discussions, a plain "New E-Mail" was regarded to be superior, i'd stay with that for now

@r10s
Copy link
Member Author

r10s commented Jul 13, 2025

The option should be hidden for chatmail profiles. Don't see the code checking it now.

it is hidden for chatmail.

see NewChatViewController.swift, line 76:

        if self.dcContext.isChatmail == false {
            newOptions.append(.newEmail)
        }

@r10s r10s marked this pull request as ready for review July 15, 2025 11:52
@r10s r10s removed the wait-for-core PR/issue is waiting for core release label Jul 15, 2025
@r10s
Copy link
Member Author

r10s commented Jul 15, 2025

k, core has been updated, this is read to review now

@r10s r10s force-pushed the r10s/new-email branch from c6f645b to 17d4cab Compare July 15, 2025 12:31
@@ -12,6 +12,7 @@ class NewGroupController: UITableViewController, MediaPickerDelegate {
private var deleteGroupImage: Bool = false

let createBroadcast: Bool
let createEmail: Bool
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This createEmail and createBroadcast could probably be consolidated into an enum? Or am I misunderstanding and is it possible to have both creatBroadcast: true and createEmail: true?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i still think, that is a minor, but i played around to see how it goes and added a commit :)

so, we have a createMode now

@r10s r10s merged commit b7e8b94 into main Jul 16, 2025
2 checks passed
@r10s r10s deleted the r10s/new-email branch July 16, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement actually in development, user visible enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants