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.

Sign up for GitHub

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

Jump to bottom

Editing a /me action ruins the action by converting it to a text message #19515

Closed
dkasak opened this issue Oct 26, 2021 · 3 comments
Closed

Editing a /me action ruins the action by converting it to a text message #19515

dkasak opened this issue Oct 26, 2021 · 3 comments
Labels
A-Message-Editing O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter

Comments

@dkasak
Copy link
Member

dkasak commented Oct 26, 2021

Steps to reproduce

  1. Type in /me is te-sting.
  2. Press up arrow, intending to edit the action.
  3. You're offered is te-sting in a text field.
  4. Correct to is testing.

Outcome

What did you expect?

That the result will remain an action, now displaying the corrected text * dkasak is testing.

What happened instead?

The action was converted to an ordinary message with the content is testing.

Operating system

No response

Browser information

No response

URL for webapp

No response

Application version

No response

Homeserver

No response

Will you send logs?

No
@dkasak dkasak added T-Defect A-Message-Editing S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely labels Oct 26, 2021
@t3chguy
Copy link
Member

t3chguy commented Oct 26, 2021

@dkasak can you please specify the application version? This should have been fixed recently. matrix-org/matrix-react-sdk#6931

@t3chguy t3chguy added the X-Needs-Info This issue is blocked awaiting information from the reporter label Oct 26, 2021
@kittykat
Copy link
Contributor

@dkasak We can't reproduce the issue so I'm going to close this issue for now. Please re-open after updating the original issue with URL and application version if you are still seeing the problem.

@dkasak
Copy link
Member Author

dkasak commented Oct 29, 2021 •
edited
Loading

It does work in Element Desktop 1.9.3, so all good. I must had been on an earlier version when I reproduced. Sorry for omitting the version, I was in a hurry and must have forgotten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Message-Editing O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@kittykat @dkasak @t3chguy