element-hq / element-meta

Shared/meta documentation and project artefacts for Element clients
75 stars 12 forks source link

Option to cancel sending of message mid-flight #1345

Open turt2live opened 7 years ago

turt2live commented 7 years ago

Description

For latency-ridden connections it may be ideal to have the option to cancel sending mid-flight. At least as a last ditch effort to hide your spelling mistakes.

Version information

t3chguy commented 7 years ago

errrr https://github.com/vector-im/riot-web/blame/162f3872e5d2182b30346e6f2903b30691d40cf6/src/components/views/context_menus/MessageContextMenu.js#L172 should be === "sending" rather than "not_sent"

t3chguy commented 7 years ago

errrrrm, upon further review this may be the issue of it being in the sending state and not being able to cancel it from that state.

This is neither a regression nor an issue :/

t3chguy commented 7 years ago

I'll let the pros triage the new being that this issue is.

turt2live commented 7 years ago

updated op - not an issue. Rephrased to be a feature request.

ara4n commented 7 years ago

hang on, i thought there was a 'cancel' option in the context menu for inflight msgs? or is this only after they've failed?

t3chguy commented 7 years ago

@ara4n only for queued or failed see https://github.com/vector-im/riot-web/blame/162f3872e5d2182b30346e6f2903b30691d40cf6/src/components/views/context_menus/MessageContextMenu.js#L172 and the cancel method it calls, just removes it from the queue

aaronraimist commented 5 years ago

Is there a parity with Riot iOS label? 😄

t3chguy commented 1 year ago

What is the expectation here if it is in-flight? To fast-follow a redaction?