[Bug]: UX: bad mail notification trigger
- Dominant language
- C++
- Stars
- 3.9k
- Forks
- 1k
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 123
Description
### ⚠️ Before submitting, please verify the following: ⚠️
- [x] This is a **bug**, not a question or a configuration issue.
- [x] This issue is **not** already reported on Github (I've searched it).
- [x] Nextcloud Server and Desktop Client are **up to date**. See [Server Maintenance and Release Schedule](https://github.com/nextcloud/server/wiki/Maintenance-and-Release-Schedule) and [Desktop Releases](https://nextcloud.com/install/#install-clients) for supported versions.
- [x] I agree to follow Nextcloud's [Code of Conduct](https://nextcloud.com/contribute/code-of-conduct/)
### Bug description
This describes problems in UX on that dialog box.
The many mail triggers cause problems, written below.
IMHO:
- User needs time to prepare, and no bogus mails should be sent.
- i plea for a "Confirm button" and remove the field-triggers (akin to web)
At the following point in UX, an Email address has been entered, and a mail has already been sent:

Sidenote: At the time of writing the server suffers from https://github.com/nextcloud/server/issues/48908, so for bug's will, that mail is not being sent.
### Steps to reproduce
With the following unwanted effects.
- After entering the address, mail goes out immediately (probably good? unlike web.)
- When "note to recipient" is checked, the mail goes out immediately, containing the wrong message "**Enter a note for the recipient**". The user will then enter a new message, **triggering yet another mail**.
- When "Password protect" is checked, the mail goes out immediately, **not waiting for the user to enter their own passwd**. Changing the password **will trigger yet another mail**.
### Expected behavior
I think mimicking the UX closely to web is ideal, namely:
- (maybe) auto-open the option dialog **right after** full email entering.
- this could be interesting behaviour because any defaults (like server-side expiration dates) are **at once perceived**, preventing further unwanted results.
- the option dialog should have ie. a "Confirm and send" button
- all inline box triggers should be removed in favour of that button.
- no unwanted mails go out.
So the user can prepare that communication package, and confirm.
### Which files are affected by this bug
none
### Operating system
macOS
### Which version of the operating system you are running.
Macos 13.6
### Package
Official macOS 12+ universal pkg
### Nextcloud Server version
30.0.1
### Nextcloud Desktop Client version
3.14.2
### Is this bug present after an update or on a fresh install?
Updated from a minor version (ex. 3.4.2 to 3.4.4)
### Are you using the Nextcloud Server Encryption module?
Encryption is Disabled
### Are you using an external user-backend?
- [x] Default internal user-backend
- [ ] LDAP/ Active Directory
- [ ] SSO - SAML
- [ ] Other
### Nextcloud Server logs
_No response_
### Additional info
_No response_
Contributor guide
Assessment
This issue has not been assessed yet.