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

TW-1923: Improve search screen #1961

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

hieutbui
Copy link
Member

@hieutbui hieutbui commented Jul 22, 2024

Ticket

Root cause

  • Currently, when search message is empty, search controller doesn't check for keyword is Matrix ID

Solution

  • When search message is empty, check if keyword is Matrix ID then hide empty widget.

Test recommendations

  • Search keywords that is a valid Matrix ID
  • Search keywords that is a valid Matrix ID but not complete (Ex: @test:domain.co)
  • Send message with a Matrix ID inside a room then search for this Matrix ID

Resolved

  • Web:
1923.web.mp4
  • Android:
  • IOS:

Copy link

This PR has been deployed to https://linagora.github.io/twake-on-matrix/1961

@hieutbui hieutbui force-pushed the TW-1923-Improve-search-screen branch from e12cfd6 to 7b370f6 Compare July 23, 2024 03:12
@hieutbui hieutbui changed the title [WIP] TW-1923: Improve search screen TW-1923: Improve search screen Jul 23, 2024
@hieutbui hieutbui force-pushed the TW-1923-Improve-search-screen branch from 98c94cb to db457c5 Compare July 23, 2024 06:30
@sherlockvn
Copy link
Contributor

not in the DOD, but i think this case, we should remove the Messages section too
image

@KhaledNjim
Copy link
Contributor

not in the DOD, but i think this case, we should remove the Messages section too image

Updated

image

@nqhhdev
Copy link
Member

nqhhdev commented Nov 25, 2024

I tested it on mobile, web works well

@nqhhdev
Copy link
Member

nqhhdev commented Nov 25, 2024

I have one concern about padding in chat search on web. WDYT @hoangdat @KhaledNjim

  • Currently
Screenshot 2024-11-25 at 10 23 57
  • After updated
Screenshot 2024-11-25 at 10 22 32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants