Skip to content

Commit

Permalink
WIP file captions
Browse files Browse the repository at this point in the history 
Signed-off-by: Julius Linus <[email protected]>
  • Loading branch information
@rapterjet2004
rapterjet2004 committed Oct 31, 2023
1 parent f980861 commit 4326c7e
Show file tree
Hide file tree
Showing 11 changed files with 595 additions and 42 deletions.
48 changes: 47 additions & 1 deletion app/src/main/java/com/nextcloud/talk/adapters/messages/IncomingPreviewMessageViewHolder.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@

package com.nextcloud.talk.adapters.messages;

import android.text.Spanned;
import android.util.TypedValue;
import android.view.View;
import android.widget.ImageView;
import android.widget.ProgressBar;
Expand All @@ -33,6 +35,10 @@
import com.nextcloud.talk.databinding.ItemCustomIncomingPreviewMessageBinding;
import com.nextcloud.talk.databinding.ReactionsInsideMessageBinding;
import com.nextcloud.talk.models.json.chat.ChatMessage;
import com.nextcloud.talk.ui.ImageStackContainer;
import com.nextcloud.talk.utils.TextMatchers;

import java.util.HashMap;

import androidx.annotation.NonNull;
import androidx.core.content.ContextCompat;
Expand All @@ -49,7 +55,36 @@ public IncomingPreviewMessageViewHolder(View itemView, Object payload) {
@Override
public void onBind(@NonNull ChatMessage message) {
super.onBind(message);

if(!message.isVoiceMessage()) {
assert viewThemeUtils != null;
Spanned processedMessageText = messageUtils.enrichChatMessageText(
binding.messageCaption.getContext(),
message,
true,
viewThemeUtils);

assert processedMessageText != null;
processedMessageText = messageUtils.processMessageParameters(
binding.messageCaption.getContext(),
viewThemeUtils,
processedMessageText,
message,
itemView);

assert context != null;
var textSize = context.getResources().getDimension(R.dimen.chat_text_size);
HashMap<String, HashMap<String, String>> messageParameters = message.getMessageParameters();
if (
(messageParameters == null || messageParameters.size() <= 0) &&
TextMatchers.isMessageWithSingleEmoticonOnly(message.getText())
) {
textSize = (float) (textSize * IncomingTextMessageViewHolder.TEXT_SIZE_MULTIPLIER);
itemView.setSelected(true);
}
binding.messageCaption.setTextSize(TypedValue.COMPLEX_UNIT_PX, textSize);
binding.messageCaption.setText(processedMessageText);
viewThemeUtils.talk.themeIncomingMessageBubble(binding.messageCaption, true, false);
}
binding.messageAuthor.setText(message.getActorDisplayName());
binding.messageText.setTextColor(ContextCompat.getColor(binding.messageText.getContext(),
R.color.no_emphasis_text));
Expand All @@ -63,6 +98,12 @@ public EmojiTextView getMessageText() {
return binding.messageText;
}

@NonNull
@Override
public EmojiTextView getMessageCaption() {
return binding.messageCaption;
}

@Override
public ProgressBar getProgressBar() {
return binding.progressBar;
Expand Down Expand Up @@ -100,4 +141,9 @@ public ProgressBar getPreviewContactProgressBar() {
@Override
public ReactionsInsideMessageBinding getReactionsBinding(){ return binding.reactions; }

@NonNull
@Override
public ImageStackContainer getImageStack() {
return binding.imageStack;
}
}
48 changes: 48 additions & 0 deletions ...src/main/java/com/nextcloud/talk/adapters/messages/OutcomingPreviewMessageViewHolder.java
View file
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@

package com.nextcloud.talk.adapters.messages;

import android.text.Spanned;
import android.util.TypedValue;
import android.view.View;
import android.widget.ImageView;
import android.widget.ProgressBar;
Expand All @@ -31,7 +33,12 @@
import com.nextcloud.talk.databinding.ItemCustomOutcomingPreviewMessageBinding;
import com.nextcloud.talk.databinding.ReactionsInsideMessageBinding;
import com.nextcloud.talk.models.json.chat.ChatMessage;
import com.nextcloud.talk.ui.ImageStackContainer;
import com.nextcloud.talk.utils.TextMatchers;

import java.util.HashMap;

import androidx.annotation.NonNull;
import androidx.core.content.ContextCompat;
import androidx.emoji2.widget.EmojiTextView;

Expand All @@ -48,6 +55,38 @@ public OutcomingPreviewMessageViewHolder(View itemView) {
public void onBind(ChatMessage message) {
super.onBind(message);

if(!message.isVoiceMessage()) {
assert viewThemeUtils != null;
Spanned processedMessageText = messageUtils.enrichChatMessageText(
binding.messageCaption.getContext(),
message,
false,
viewThemeUtils);

assert processedMessageText != null;
processedMessageText = messageUtils.processMessageParameters(
binding.messageCaption.getContext(),
viewThemeUtils,
processedMessageText,
message,
itemView);

assert context != null;
var textSize = context.getResources().getDimension(R.dimen.chat_text_size);
HashMap<String, HashMap<String, String>> messageParameters = message.getMessageParameters();
if (
(messageParameters == null || messageParameters.size() <= 0) &&
TextMatchers.isMessageWithSingleEmoticonOnly(message.getText())
) {
textSize = (float)(textSize * IncomingTextMessageViewHolder.TEXT_SIZE_MULTIPLIER);
itemView.setSelected(true);
}
binding.messageCaption.setTextSize(TypedValue.COMPLEX_UNIT_PX, textSize);
binding.messageCaption.setText(processedMessageText);
viewThemeUtils.talk.themeOutgoingMessageBubble(binding.messageCaption, true, false);
}


binding.messageText.setTextColor(ContextCompat.getColor(binding.messageText.getContext(),
R.color.no_emphasis_text));
binding.messageTime.setTextColor(ContextCompat.getColor(binding.messageText.getContext(),
Expand Down Expand Up @@ -91,4 +130,13 @@ public ProgressBar getPreviewContactProgressBar() {

@Override
public ReactionsInsideMessageBinding getReactionsBinding() { return binding.reactions; }

@Override
public EmojiTextView getMessageCaption() { return binding.messageCaption; }

@NonNull
@Override
public ImageStackContainer getImageStack() {
return binding.imageStack;
}
}
25 changes: 24 additions & 1 deletion app/src/main/java/com/nextcloud/talk/adapters/messages/PreviewMessageViewHolder.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -50,12 +50,15 @@ import com.nextcloud.talk.data.user.model.User
import com.nextcloud.talk.databinding.ReactionsInsideMessageBinding
import com.nextcloud.talk.extensions.loadChangelogBotAvatar
import com.nextcloud.talk.models.json.chat.ChatMessage
import com.nextcloud.talk.ui.ImageStackContainer
import com.nextcloud.talk.ui.theme.ViewThemeUtils
import com.nextcloud.talk.users.UserManager
import com.nextcloud.talk.utils.DateUtils
import com.nextcloud.talk.utils.DisplayUtils
import com.nextcloud.talk.utils.DrawableUtils.getDrawableResourceIdForMimeType
import com.nextcloud.talk.utils.FileViewerUtils
import com.nextcloud.talk.utils.FileViewerUtils.ProgressUi
import com.nextcloud.talk.utils.message.MessageUtils
import com.stfalcon.chatkit.messages.MessageHolders.IncomingImageMessageViewHolder
import io.reactivex.Single
import io.reactivex.SingleObserver
Expand All @@ -80,6 +83,12 @@ abstract class PreviewMessageViewHolder(itemView: View?, payload: Any?) :
@Inject
lateinit var dateUtils: DateUtils

@Inject
lateinit var messageUtils: MessageUtils

@Inject
lateinit var userManager: UserManager

@JvmField
@Inject
var okHttpClient: OkHttpClient? = null
Expand Down Expand Up @@ -111,6 +120,13 @@ abstract class PreviewMessageViewHolder(itemView: View?, payload: Any?) :
if (message.getCalculateMessageType() === ChatMessage.MessageType.SINGLE_NC_ATTACHMENT_MESSAGE) {
fileViewerUtils = FileViewerUtils(context!!, message.activeUser!!)
val fileName = message.selectedIndividualHashMap!![KEY_NAME]

imageStack.resetFilePaths(message.filePathList)
messageCaption.visibility = if (message.message != "{file}") {
View.VISIBLE
} else {
View.GONE
}
messageText.text = fileName

if (message.activeUser != null &&
Expand Down Expand Up @@ -223,7 +239,12 @@ abstract class PreviewMessageViewHolder(itemView: View?, payload: Any?) :
image = previewContactPhoto
image.setImageDrawable(ContextCompat.getDrawable(context!!, R.drawable.ic_mimetype_text_vcard))
}
} else {
} else if (message.displayImageStack()) {
imageStack.visibility = View.VISIBLE
previewContainer.visibility = View.GONE
previewContactContainer.visibility = View.GONE
}
else {
previewContainer.visibility = View.VISIBLE
previewContactContainer.visibility = View.GONE
}
Expand Down Expand Up @@ -312,7 +333,9 @@ abstract class PreviewMessageViewHolder(itemView: View?, payload: Any?) :
}

abstract val messageText: EmojiTextView
abstract val messageCaption: EmojiTextView
abstract val previewContainer: View
abstract val imageStack: ImageStackContainer
abstract val previewContactContainer: MaterialCardView
abstract val previewContactPhoto: ImageView
abstract val previewContactName: EmojiTextView
Expand Down
93 changes: 54 additions & 39 deletions app/src/main/java/com/nextcloud/talk/chat/ChatActivity.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ import android.widget.RelativeLayout.LayoutParams
import android.widget.SeekBar
import android.widget.TextView
import androidx.activity.OnBackPressedCallback
import androidx.appcompat.app.AlertDialog
import androidx.appcompat.view.ContextThemeWrapper
import androidx.core.content.ContextCompat
import androidx.core.content.FileProvider
Expand Down Expand Up @@ -113,7 +112,6 @@ import coil.target.Target
import coil.transform.CircleCropTransformation
import com.google.android.flexbox.FlexboxLayout
import com.google.android.material.button.MaterialButton
import com.google.android.material.dialog.MaterialAlertDialogBuilder
import com.google.android.material.snackbar.Snackbar
import com.nextcloud.android.common.ui.theme.utils.ColorRole
import com.nextcloud.talk.BuildConfig
Expand Down Expand Up @@ -188,6 +186,7 @@ import com.nextcloud.talk.ui.StatusDrawable
import com.nextcloud.talk.ui.bottom.sheet.ProfileBottomSheet
import com.nextcloud.talk.ui.dialog.AttachmentDialog
import com.nextcloud.talk.ui.dialog.DateTimePickerFragment
import com.nextcloud.talk.ui.dialog.FileAttachmentPreviewFragment
import com.nextcloud.talk.ui.dialog.MessageActionsDialog
import com.nextcloud.talk.ui.dialog.ShowReactionsDialog
import com.nextcloud.talk.ui.recyclerview.MessageSwipeActions
Expand Down Expand Up @@ -2411,40 +2410,12 @@ class ChatActivity :
filenamesWithLineBreaks.append(filename).append("\n")
}

val confirmationQuestion = when (filesToUpload.size) {
1 -> context.resources?.getString(R.string.nc_upload_confirm_send_single)?.let {
String.format(it, title.trim())
}

else -> context.resources?.getString(R.string.nc_upload_confirm_send_multiple)?.let {
String.format(it, title.trim())
}
}

binding.messageInputView.context?.let {
val materialAlertDialogBuilder = MaterialAlertDialogBuilder(it)
.setTitle(confirmationQuestion)
.setMessage(filenamesWithLineBreaks.toString())
.setPositiveButton(R.string.nc_yes) { _, _ ->
if (permissionUtil.isFilesPermissionGranted()) {
uploadFiles(filesToUpload)
} else {
UploadAndShareFilesWorker.requestStoragePermission(this)
}
}
.setNegativeButton(R.string.nc_no) { _, _ ->
// unused atm
}

viewThemeUtils.dialog.colorMaterialAlertDialogBackground(it, materialAlertDialogBuilder)

val dialog = materialAlertDialogBuilder.show()

viewThemeUtils.platform.colorTextButtons(
dialog.getButton(AlertDialog.BUTTON_POSITIVE),
dialog.getButton(AlertDialog.BUTTON_NEGATIVE)
)
}
val newFragment: DialogFragment = FileAttachmentPreviewFragment.newInstance(
filenamesWithLineBreaks.toString(),
filesToUpload,
this::uploadFiles
)
newFragment.show(supportFragmentManager, FileAttachmentPreviewFragment.TAG)
} catch (e: IllegalStateException) {
context.resources?.getString(R.string.nc_upload_failed)?.let {
Snackbar.make(
Expand Down Expand Up @@ -2630,13 +2601,17 @@ class ChatActivity :
}
}

private fun uploadFiles(files: MutableList<String>) {
private fun uploadFiles(files: MutableList<String>, caption: String = "") {
// TODO for now all files are uploaded with a caption. In the future, set only one, and just don't render the
// rest, cuz it'll be a group message
val groupKey = SystemClock.elapsedRealtime().toString()
var newCaption = "$groupKey:" + "${files.size}:" + caption
for (file in files) {
uploadFile(file, false)
uploadFile(file, false, newCaption)
}
}

private fun uploadFile(fileUri: String, isVoiceMessage: Boolean) {
private fun uploadFile(fileUri: String, isVoiceMessage: Boolean, caption: String = "") {
var metaData = ""

if (!participantPermissions.hasChatPermission()) {
Expand All @@ -2648,6 +2623,10 @@ class ChatActivity :
metaData = VOICE_MESSAGE_META_DATA
}

if (caption != "") {
metaData = "{\"caption\":\"$caption\"}"
}

try {
require(fileUri.isNotEmpty())
UploadAndShareFilesWorker.upload(
Expand Down Expand Up @@ -3189,6 +3168,8 @@ class ChatActivity :

handleExpandableSystemMessages(chatMessageList)

processGroupedFiles(chatMessageList)

processHeaderChatLastGiven(response, lookIntoFuture)

if (chatMessageList.isNotEmpty() &&
Expand Down Expand Up @@ -3343,6 +3324,40 @@ class ChatActivity :
}
}

private fun processGroupedFiles(chatMessageList: List<ChatMessage>) {
val map = mutableMapOf<String, ChatMessage>()
for (message in chatMessageList) {
if (message.getCalculateMessageType() == ChatMessage.MessageType.SINGLE_NC_ATTACHMENT_MESSAGE &&
message.message!!.isNotBlank() && message.message != "{file}"
) {
val groupKey = message.message!!.substringBefore(':')
message.message = message.message!!.substringAfter(':')
val size = message.message!!.substringBefore(':').toInt()
message.message = message.message!!.substringAfter(':')
if (!map.contains(groupKey)) {
map.put(groupKey, message)
message.message = message.message!!.substringAfter(':')
val name = message.messageParameters?.get("file")?.get(PreviewMessageViewHolder.KEY_NAME)
if (name != null) {
val path = applicationContext.cacheDir.absolutePath + "/" + name
message.filePathList.add(path)
adapter?.notifyDataSetChanged()
}
} else {
message.displayedElseWhere = true
val name = message.messageParameters?.get("file")?.get(PreviewMessageViewHolder.KEY_NAME)
if (name != null) {
val path = applicationContext.cacheDir.absolutePath + "/" + name
map[groupKey]!!.filePathList.add(path)
adapter?.notifyDataSetChanged()
}
// TODO for now, both files will be displayed, but I just want to get this to work
// later on I can work on "hiding" the non rendered messages
}
}
}
}

private fun updateReadStatusOfAllMessages(xChatLastCommonRead: Int?) {
if (adapter != null) {
for (message in adapter!!.items) {
Expand Down
Loading

0 comments on commit 4326c7e

Please sign in to comment.