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.

Sign up for GitHub

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

Jump to bottom

Refactor MessageInput logic into it's own fragment #3784

Open
rapterjet2004 opened this issue Mar 29, 2024 · 0 comments · May be fixed by #3792
Open

Refactor MessageInput logic into it's own fragment #3784

rapterjet2004 opened this issue Mar 29, 2024 · 0 comments · May be fixed by #3792
Assignees
@rapterjet2004
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) feature: 🗨️ chat technical debt Technical issue

Comments

@rapterjet2004
Copy link
Contributor

MessageInput has grown in complexity. It's unreasonable to have it in ChatActivity. It should be moved into it's own fragment to simplify the logic, and prevent future bugs from occurring.
@rapterjet2004 rapterjet2004 added 1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) technical debt Technical issue feature: 🗨️ chat labels Mar 29, 2024
@rapterjet2004 rapterjet2004 linked a pull request Apr 1, 2024 that will close this issue
Open
41 tasks
@rapterjet2004 rapterjet2004 self-assigned this Apr 11, 2024
@rapterjet2004 rapterjet2004 mentioned this issue May 17, 2024
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rapterjet2004 rapterjet2004

Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) feature: 🗨️ chat technical debt Technical issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactored Message input view into it's own Fragment nextcloud/talk-android
1 participant
@rapterjet2004