Consider treating an edit of deleting all text a redaction #11024
Comments
|
Related: element-hq/element-meta#1335 |
|
I'd prefer if we changed the presentation of a message that's been edited to be blank. A redaction is intended to be a very specific thing where the user actively needs to purge the message from as many places as possible, eg. because it leaks sensitive content or contains something illegal, although we don't really present it like this in the UI at the moment or offer an alternative. I think we should be trying to move towards redactions being a special and unusual case rather than doing redactions where the user didn't ask to. |
|
To me it seems like a user editing their message to blank means they didn't want to send that message and just didn't know that they were supposed to click the Remove button. It seems like the only reason a user would do that is because they intend for their message to not be seen anymore (which btw isn't what happens even with my PR due to https://github.com/vector-im/riot-web/issues/11978 still existing). Imo it is a fairly big privacy issue. Maybe there are examples of a user editing their message to blank and not intending for their message to be deleted but I can't think of any. |
|
I think the app should be as clear as possible about what operation has been performed, so the user can understand what has happened and use the app to achieve what they intend, rather than second-guessing the user's intentions. If a user does edit their message to be blank, I think it's better if they can then see that the history is still visible, so if they really want to try to remove the old message, they can then still choose to redact. |
|
+1, basically echoed by my comment (with suggested comp) here: |
|
Is a "redaction" different than "Deleting a message" In my case, I typed a message that was part of a longer message, and accidentally hit the enter key. So it's very much incomplete. I could just edit the message and have it updated, but it's still sitting there, partially completed, on everyone's screen. I'd rather drop it, and start my message over again. I don't see the same "redacted" semantics as described by @dbkr . To me, it's just a message that I do want deleted, not for any of the mentioned reasons, but I want it gone. To me, it doesn't matter if it's "Redacted" or "Deleted" it doesn't contribute to the conversation, and is erroneous. |
|
Redaction is the Matrix term for it, in the UI it is exposed as |
|
For some context, Discord and Telegram have this behaviour, so some users could/would be expecting this, coming from other platforms. They each do a normal popup of "do you want to delete this message?" as a blocking popup, and which'd take a second enter press/click to accept/decline. Edit: Platforms with this behaviour;
|
aaronraimist
removed
the
Z-Community-PR
|
Just copy/pasting an excerpt from a duplicate issue I made.
Editing a message with empty content will prompt for the deletion of the message as if the delete button was pushed. (Usually with an "Are you sure?" on in the case of Element, probably asking for the reason. |
|
@SethFalco added that to my comment |
SimonBrandner
added
O-Occasional
|
You can add Mattermost to this list, the prompt to delete a message is also confirm-able with the return key (same as Discord which is also important IMO, haven't tested others). |
|
This seems to be implemented now on app.element.io, though i don't know where, or by whom 👀 @SimonBrandner could this be closed? |
* OIDC: add delegatedauthentication to validated server config ([\element-hq#11053](matrix-org/matrix-react-sdk#11053)). Contributed by @kerryarchibald. * Allow image pasting in plain mode in RTE ([\element-hq#11056](matrix-org/matrix-react-sdk#11056)). Contributed by @alunturner. * Show room options menu if "UIComponent.roomOptionsMenu" is enabled ([\element-hq#10365](matrix-org/matrix-react-sdk#10365)). Contributed by @maheichyk. * Allow image pasting in rich text mode in RTE ([\element-hq#11049](matrix-org/matrix-react-sdk#11049)). Contributed by @alunturner. * Update voice broadcast redaction to use MSC3912 `with_rel_type` instead of `with_relations` ([\element-hq#11014](matrix-org/matrix-react-sdk#11014)). Fixes element-hq#25471. * Add config to skip widget_build_url for DM rooms ([\element-hq#11044](matrix-org/matrix-react-sdk#11044)). Fixes vector-im/customer-retainer#74. * Inhibit interactions on forward dialog message previews ([\element-hq#11025](matrix-org/matrix-react-sdk#11025)). Fixes element-hq#23459. * Removed `DecryptionFailureBar.tsx` ([\element-hq#11027](matrix-org/matrix-react-sdk#11027)). Fixes element-hq/element-meta#1358. Contributed by @florianduros. * Fix translucent `TextualEvent` on search results panel ([\element-hq#10810](matrix-org/matrix-react-sdk#10810)). Fixes element-hq#25292. Contributed by @luixxiul. * Matrix matrix scheme permalink constructor not stripping query params ([\element-hq#11060](matrix-org/matrix-react-sdk#11060)). Fixes element-hq#25535. * Fix: "manually verify by text" does nothing ([\element-hq#11059](matrix-org/matrix-react-sdk#11059)). Fixes element-hq#25375. Contributed by @kerryarchibald. * Make group calls respect the ICE fallback setting ([\element-hq#11047](matrix-org/matrix-react-sdk#11047)). Fixes vector-im/voip-internal#65. * Align list items on the tooltip to the start ([\element-hq#11041](matrix-org/matrix-react-sdk#11041)). Fixes element-hq#25355. Contributed by @luixxiul. * Clear thread panel event permalink when changing rooms ([\element-hq#11024](matrix-org/matrix-react-sdk#11024)). Fixes element-hq#25484. * Fix spinner placement on pinned widgets being reloaded ([\element-hq#10970](matrix-org/matrix-react-sdk#10970)). Fixes element-hq#25431. Contributed by @luixxiul.
ie: send a message, edit it, select all, delete, send. Should probably be a redaction
The text was updated successfully, but these errors were encountered: