Skip to content

Image keyboard support in chat - #1749

Merged
mahibi merged 2 commits into
masterfrom
feature/411/gif_keyboard
Jan 4, 2022
Merged

Image keyboard support in chat#1749
mahibi merged 2 commits into
masterfrom
feature/411/gif_keyboard

Conversation

@starypatyk

Copy link
Copy Markdown
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 <starypatyk@users.noreply.github.com>
Signed-off-by: Dariusz Olszewski <starypatyk@users.noreply.github.com>
@mahibi
mahibi force-pushed the feature/411/gif_keyboard branch from 306e640 to 996c4c5 Compare January 4, 2022 14:38
@mahibi

mahibi commented Jan 4, 2022

Copy link
Copy Markdown
Collaborator

works fine, thank you very much @starypatyk 👍

@nextcloud-android-bot

Copy link
Copy Markdown
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

github-actions Bot commented Jan 4, 2022

Copy link
Copy Markdown
Contributor

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

3 participants