-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
Add clear icon for search view in open conversations #4497
Conversation
a83de06
to
6c2a8d3
Compare
6c2a8d3
to
c07b72c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you have a closer look how much work it is to use a SearchView (https://developer.android.com/reference/com/google/android/material/search/SearchView) here? I think it would make more sense as it's intended to use with the appbar.
If SearchView is not possible for whatever reason, then you should wrap the
TextInputEditText inside a TextInputLayout which will also give you the possibility to use built in cancel button etc.
9bc5681
to
59673f4
Compare
Signed-off-by: sowjanyakch <[email protected]>
Signed-off-by: sowjanyakch <[email protected]>
59673f4
to
bda0531
Compare
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/4497-talk.apk |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when typing, the result list should also be updated if there searched string doesn't match anymore.
For example there are 3 rooms called "Talk".
When i search for "Tal", all 3 rooms are shown.
But when i add a letter that doesn't match, the results should disappear. So when i type "Talx", the old results are still shown.
I will implement the search bug in a different PR. |
Resolve #4490
🚧 TODO
🏁 Checklist
/backport to stable-xx.x