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

Messages with formatted bodies don't get properly edited #7359

Closed
jmartinesp opened this issue Oct 13, 2022 · 1 comment
Closed

Messages with formatted bodies don't get properly edited #7359

jmartinesp opened this issue Oct 13, 2022 · 1 comment
Assignees
@jmartinesp
Labels
O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems

Comments

@jmartinesp
Copy link
Member

Steps to reproduce

  1. Enable the rich text editor in labs in the Android app and send a formatted message. Alternatively, write some formatted message in the web version.
  2. Disable the rich text editor, try to edit the formatted message.

Outcome

What did you expect?

The contents of the formatted message should be overwritten.

What happened instead?

Only the 'body' of the formatted message changes, while the 'formattedBody' stays the same.

Your phone model

Emulator

Operating system version

13

Application version and app store

1.5.4-dev [40105042] (G-b07f6f50) misc/apply-designs-to-wysiwyg-editor

Homeserver

matrix.org

Will you send logs?

No

Are you willing to provide a PR?

Yes
@jmartinesp jmartinesp added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Oct 13, 2022
@jmartinesp jmartinesp self-assigned this Oct 13, 2022
@jmartinesp jmartinesp added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Oct 13, 2022
@jmartinesp jmartinesp mentioned this issue Oct 13, 2022
Merged
15 tasks
@jmartinesp
Copy link
Member Author

Fixed in #7360 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmartinesp jmartinesp

Labels
O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jmartinesp