-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat: typing indicator in conversation view (WPB-4706) #2292
Conversation
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1103 succeeded. The build produced the following APK's: |
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.
Looking good 🚀 I would just not use this state with others because it will change a lot
observeAudioPlayerState() | ||
} | ||
|
||
private fun observeUsersTypingState() { |
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.
I would prefer to have small view model for this state. This state is independent from others 🤔
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.
Done!
app/src/main/kotlin/com/wire/android/ui/home/conversations/UsersTypingIndicator.kt
Outdated
Show resolved
Hide resolved
APKs built during tests are available here. Scroll down to Artifacts! |
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1113 succeeded. The build produced the following APK's: |
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1121 succeeded. The build produced the following APK's: |
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.
Wow that was fast 🐎
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1127 succeeded. The build produced the following APK's: |
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
We need to add the capability to add the typing indicator for conversations.
Causes (Optional)
Whenever another user is typing, we are going to show the writing pen indication.
Depends on
Testing
Test Coverage (Optional)
Attachments (Optional)
Screen.Recording.2023-10-02.at.11.37.16.mov
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764
.