Skip to content

Conversation

@starypatyk
Copy link
Contributor

Should resolve #411

Added support for images received from keyboard as described here:
https://developer.android.com/guide/topics/text/image-keyboard

When a keyboard (like Google Keyboard) sends an image - usually a GIF 😜 - to the text entry field, the code uploads the image to the current chat.

Implementation is a bit tricky, since it "injects" a new class ImageEmojiEditText into a part of the layout managed by external ChatKit component. However since the ImageEmojiEditText inherits from EmojiEditText used originally, ChatKit seems to work just fine. And we can attach the required callback in the ChatController class.

@starypatyk starypatyk added 3. to review Waiting for reviews enhancement New feature or request feature: chat 🗨️ labels Dec 22, 2021
Receive images (usually GIFs) from keyboard and upload them to the current chat.

Signed-off-by: Dariusz Olszewski <[email protected]>
Signed-off-by: Dariusz Olszewski <[email protected]>
@mahibi mahibi force-pushed the feature/411/gif_keyboard branch from 306e640 to 996c4c5 Compare January 4, 2022 14:38
@mahibi
Copy link
Collaborator

mahibi commented Jan 4, 2022

works fine, thank you very much @starypatyk 👍

@nextcloud-android-bot
Copy link
Collaborator

Lint

TypemasterPR
Warnings222222
Errors11

SpotBugs (new)

Warning Type Number
Bad practice Warnings 12
Correctness Warnings 121
Experimental Warnings 2
Internationalization Warnings 9
Malicious code vulnerability Warnings 168
Performance Warnings 29
Security Warnings 4
Dodgy code Warnings 214
Total 559

SpotBugs (master)

Warning Type Number
Bad practice Warnings 12
Correctness Warnings 121
Experimental Warnings 2
Internationalization Warnings 9
Malicious code vulnerability Warnings 168
Performance Warnings 29
Security Warnings 4
Dodgy code Warnings 214
Total 559

@github-actions
Copy link
Contributor

github-actions bot commented Jan 4, 2022

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/1749-talk.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud Talk app.

@mahibi mahibi added this to the 13.1.0 milestone Jan 4, 2022
@mahibi mahibi merged commit 58b714c into master Jan 4, 2022
@delete-merged-branch delete-merged-branch bot deleted the feature/411/gif_keyboard branch January 4, 2022 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature or request feature: chat 🗨️

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add GIF (Giphy) keyboard

4 participants