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

Option to cancel sending of message mid-flight #1345

Open
turt2live opened this issue Jun 12, 2017 · 8 comments
Open

Option to cancel sending of message mid-flight #1345

turt2live opened this issue Jun 12, 2017 · 8 comments

Comments

@turt2live
Copy link
Member

turt2live commented Jun 12, 2017

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

  • Platform: web (in-browser)
  • Browser: Chrome 58
  • OS: Windows 10
  • URL: riot.im/develop
@t3chguy

This comment has been minimized.

@t3chguy

This comment has been minimized.

@turt2live turt2live changed the title 'Cancel sending' option disappeared from message context menu Option to cancel sending of message mid-flight Jun 12, 2017
@t3chguy

This comment has been minimized.

@turt2live
Copy link
Member Author

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

@ara4n
Copy link
Member

ara4n commented Jun 13, 2017

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
Copy link
Member

t3chguy commented Jun 13, 2017

@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
Copy link

Is there a parity with Riot iOS label? 😄

@t3chguy
Copy link
Member

t3chguy commented Mar 31, 2023

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

@t3chguy t3chguy transferred this issue from element-hq/element-web Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants