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

Punctuation marks are prefixed with backslashes in message body #1260

Closed
rom4nik opened this issue Sep 8, 2023 · 2 comments · Fixed by #1289
Closed

Punctuation marks are prefixed with backslashes in message body #1260

rom4nik opened this issue Sep 8, 2023 · 2 comments · Fixed by #1289
Labels
A-Rich-Text-Editor O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Team: Element X Feature

Comments

@rom4nik
Copy link
Contributor

rom4nik commented Sep 8, 2023

Steps to reproduce

  1. Go to a room
  2. Type e.g. !ping or .,*"':;!? in message box
  3. Send message

Outcome

What did you expect?

content.body in message has !ping value.

What happened instead?

{
    "content": {
        "body": "\\!ping",
        "format": "org.matrix.custom.html",
        "formatted_body": "!ping",
        "msgtype": "m.text"
    },
    [...]
}

Another example:

{
    "content": {
        "body": "\\.\\,\\*\\\"\\'\\:\\;\\!\\?",
        "format": "org.matrix.custom.html",
        "formatted_body": ".,*\"':;!?",
        "msgtype": "m.text"
    },
    [...]
}

btw, single backslashes are visible in reply previews. Can't bisect right now, but it might have been caused by #1172, since a build a few commits before that PR was merged didn't have this issue.

Your phone model

Google Pixel 3

Operating system version

Android 12

Application version and app store

5a0e0a8 (current develop branch)

Homeserver

Synapse 1.91.1

Will you send logs?

No

Are you willing to provide a PR?

No

@rom4nik rom4nik added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Sep 8, 2023
@kittykat kittykat added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience Team: Element X Feature A-Rich-Text-Editor Z-Schedule labels Sep 8, 2023
@kittykat
Copy link
Contributor

kittykat commented Sep 8, 2023

Same as element-hq/element-x-ios#1629

@frebib
Copy link
Contributor

frebib commented Sep 12, 2023

#1291 should fix this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Rich-Text-Editor O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Team: Element X Feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants