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

Search is broken after message edits #27386

Closed
rda0 opened this issue Apr 24, 2024 · 3 comments
Closed

Search is broken after message edits #27386

rda0 opened this issue Apr 24, 2024 · 3 comments
Labels
A-Message-Editing A-New-Search-Experience The new search dialog available in Labs S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect

Comments

@rda0
Copy link
Contributor

rda0 commented Apr 24, 2024

Steps to reproduce

  1. Create a new room (encrypted or unencrypted)
  2. Send message: foo
  3. Send message: bar
  4. Search for foo or bar works ✔️
  5. Edit the bar message: bar -> foo

Outcome

What did you expect?

  1. Search for foo shows 2 foo results
  2. Search for bar shows nothing (I think I would not expect to find the original message before the edit)

What happened instead?

  1. Search for foo: shows nothing (No more results)
  2. Search for bar: shows 2(1) foo result (I think it only finds 1 message and the other is shown as surrounding text)

Operating system

Linux

Browser information

Firefox 125.0.2 (64-bit)

URL for webapp

app.element.io

Application version

Element version: 1.11.65

Homeserver

matrix.org, phys.ethz.ch

Will you send logs?

Yes

@rda0 rda0 added the T-Defect label Apr 24, 2024
@dosubot dosubot bot added A-Message-Editing A-New-Search-Experience The new search dialog available in Labs S-Major Severely degrades major functionality or product features, with no satisfactory workaround labels Apr 24, 2024
@t3chguy
Copy link
Member

t3chguy commented Apr 24, 2024

The search is done by the Matrix Server, so you'd have to open your issue there. You haven't specified which server you are using so I cannot point you to the appropriate repository.

@t3chguy t3chguy closed this as not planned Won't fix, can't repro, duplicate, stale Apr 24, 2024
@rda0
Copy link
Contributor Author

rda0 commented Apr 24, 2024

@t3chguy Done.
update: homeserver info (matrix.org). It happens in encrypted and unencrypted rooms.

@t3chguy
Copy link
Member

t3chguy commented Apr 24, 2024

matrix.org uses https://github.com/element-hq/synapse/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Message-Editing A-New-Search-Experience The new search dialog available in Labs S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants