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

We seem to have broken pill highlights on search results #5007

Closed
ara4n opened this issue Sep 8, 2017 · 11 comments
Closed

We seem to have broken pill highlights on search results #5007

ara4n opened this issue Sep 8, 2017 · 11 comments
Labels
A-Timeline-Search Help Wanted Extra attention is needed O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Sep 8, 2017

When you search for something you're meant to see a pill around the matching term in the search results. It seems to have gone awol.

@lampholder
Copy link
Member

I can't reproduce this on /develop on Chrome on Ubuntu or Mac OSX

@ara4n are you able to share the search term or the matching text to help diagnose/triage?

@grahamperrin
Copy link

I get a pill on the search term but not on the authors:

2017-09-15 19 12 38 riot 2

@turt2live
Copy link
Member

This seems a bit temperamental about when to pill or not:
image

seems to work fine for code blocks:
image

(keyword was draft in #riot-dev btw)

@yannicka
Copy link

yannicka commented May 21, 2020

On Riot Desktop 1.6.0, some results are not highlighted (search for "call" on "#riot-web:matrix.org"):

Peek 21-05-2020 12-04

@t3chguy
Copy link
Member

t3chguy commented May 21, 2020

This could well be a Synapse bug as the Synapse FTS is what tells us what to highlight in each message

@poljar
Copy link
Contributor

poljar commented May 21, 2020

Isn't that a search across all rooms? If so results from encrypted search results won't have the highlight.

@t3chguy
Copy link
Member

t3chguy commented May 21, 2020

Shouldn't the seshat-backed search have the same API as the Matrix server backed one and thus do that too?

@t3chguy
Copy link
Member

t3chguy commented May 21, 2020

also looking at the gif, that one is This room only

@poljar
Copy link
Contributor

poljar commented May 21, 2020

Right, I think I forgot how the highlights work. Seems to be Seshat unrelated indeed.

@kittykat kittykat added O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience and removed P2 labels Sep 30, 2021
@kittykat kittykat added the Help Wanted Extra attention is needed label Mar 31, 2022
@kittykat
Copy link
Contributor

Pills seems to sometimes be working reliably and sometimes not be working reliably:

search-pills.mp4

@t3chguy
Copy link
Member

t3chguy commented Jul 17, 2023

image
image
image

Seems to work far more reliably now

@t3chguy t3chguy closed this as completed Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Timeline-Search Help Wanted Extra attention is needed O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Development

No branches or pull requests

9 participants