-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fix tab inserting the toString of objects instead of autocompletion when sharing folder #1316
Fix tab inserting the toString of objects instead of autocompletion when sharing folder #1316
Conversation
18c6f5c
to
84cc0ae
Compare
app/src/main/java/com/infomaniak/drive/ui/fileList/fileShare/AvailableShareableItemsAdapter.kt
Outdated
Show resolved
Hide resolved
84cc0ae
to
2eca9e0
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.
This creates a bug when you select a user then press enter
This has the same behavior as using tab. |
03a1ff0
to
c7ed9d4
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.
The bug with the physical keyboard's enter key already exists on master, and is a direct cause of Material, so we'll ignore this for now as fixing it would require a big refactor
7bf4904
to
50d2ee6
Compare
50d2ee6
to
35f63f2
Compare
45f79a5
to
9ea55cc
Compare
9ea55cc
to
83d3961
Compare
Quality Gate passedIssues Measures |
The problem is mostly solved, pressing tab will not insert the object toString anymore, but it will insert the current text, thus not changing anything.
A consequence of this is that the UI is refreshed and the list of users that was displayed gets closed.
This is the best solution found that does not require a refactor of the MaterialAutoCompleteTextView.