diff --git a/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicIncomingTextMessageViewHolder.kt b/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicIncomingTextMessageViewHolder.kt index f8015062d..5eaf656cb 100644 --- a/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicIncomingTextMessageViewHolder.kt +++ b/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicIncomingTextMessageViewHolder.kt @@ -78,10 +78,6 @@ class MagicIncomingTextMessageViewHolder(incomingView: View) : MessageHolders @BindView(R.id.quotedChatMessageView) var quotedChatMessageView: RelativeLayout? = null - @JvmField - @BindView(R.id.quotedUserAvatar) - var quotedUserAvatar: ImageView? = null - @JvmField @BindView(R.id.quotedMessageAuthor) var quotedUserName: EmojiTextView? = null @@ -94,10 +90,6 @@ class MagicIncomingTextMessageViewHolder(incomingView: View) : MessageHolders @BindView(R.id.quotedMessage) var quotedMessage: EmojiTextView? = null - @JvmField - @BindView(R.id.quotedMessageTime) - var quotedMessageTime: TextView? = null - @JvmField @BindView(R.id.quoteColoredView) var quoteColoredView: View? = null @@ -236,10 +228,6 @@ class MagicIncomingTextMessageViewHolder(incomingView: View) : MessageHolders if (!message.isDeleted && message.parentMessage != null) { var parentChatMessage = message.parentMessage parentChatMessage.activeUser = message.activeUser - quotedUserAvatar?.load(parentChatMessage.user.avatar) { - addHeader("Authorization", ApiUtils.getCredentials(message.activeUser.username, message.activeUser.token)) - transformations(CircleCropTransformation()) - } parentChatMessage.imageUrl?.let { quotedMessagePreview?.visibility = View.VISIBLE quotedMessagePreview?.load(it) { @@ -254,9 +242,12 @@ class MagicIncomingTextMessageViewHolder(incomingView: View) : MessageHolders quotedUserName?.setTextColor(context!!.resources.getColor(R.color.textColorMaxContrast)) - quotedMessageTime?.text = DateFormatter.format(parentChatMessage.createdAt, DateFormatter.Template.TIME) - quotedMessageTime?.setTextColor(context!!.resources.getColor(R.color.warm_grey_four)) - quoteColoredView?.setBackgroundResource(R.color.textColorMaxContrast) + if(parentChatMessage.actorId?.equals(message.activeUser.userId) == true) { + quoteColoredView?.setBackgroundResource(R.color.colorPrimary) + } else { + quoteColoredView?.setBackgroundResource(R.color.textColorMaxContrast) + } + quotedChatMessageView?.visibility = View.VISIBLE } else { quotedChatMessageView?.visibility = View.GONE diff --git a/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicOutcomingTextMessageViewHolder.kt b/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicOutcomingTextMessageViewHolder.kt index 7d565edc2..c53cac64d 100644 --- a/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicOutcomingTextMessageViewHolder.kt +++ b/app/src/main/java/com/nextcloud/talk/adapters/messages/MagicOutcomingTextMessageViewHolder.kt @@ -65,10 +65,6 @@ class MagicOutcomingTextMessageViewHolder(itemView: View) : OutcomingTextMessage @BindView(R.id.quotedChatMessageView) var quotedChatMessageView: RelativeLayout? = null - @JvmField - @BindView(R.id.quotedUserAvatar) - var quotedUserAvatar: ImageView? = null - @JvmField @BindView(R.id.quotedMessageAuthor) var quotedUserName: EmojiTextView? = null @@ -81,10 +77,6 @@ class MagicOutcomingTextMessageViewHolder(itemView: View) : OutcomingTextMessage @BindView(R.id.quotedMessage) var quotedMessage: EmojiTextView? = null - @JvmField - @BindView(R.id.quotedMessageTime) - var quotedMessageTime: TextView? = null - @JvmField @BindView(R.id.quoteColoredView) var quoteColoredView: View? = null @@ -165,10 +157,6 @@ class MagicOutcomingTextMessageViewHolder(itemView: View) : OutcomingTextMessage if (!message.isDeleted && message.parentMessage != null) { var parentChatMessage = message.parentMessage parentChatMessage.activeUser = message.activeUser - quotedUserAvatar?.load(parentChatMessage.user.avatar) { - transformations(CircleCropTransformation()) - addHeader("Authorization", ApiUtils.getCredentials(message.activeUser.username, message.activeUser.token)) - } parentChatMessage.imageUrl?.let { quotedMessagePreview?.visibility = View.VISIBLE quotedMessagePreview?.load(it) { @@ -183,8 +171,6 @@ class MagicOutcomingTextMessageViewHolder(itemView: View) : OutcomingTextMessage quotedMessage?.setTextColor(context!!.resources.getColor(R.color.nc_outcoming_text_default)) quotedUserName?.setTextColor(context!!.resources.getColor(R.color.nc_grey)) - quotedMessageTime?.text = DateFormatter.format(parentChatMessage.createdAt, DateFormatter.Template.TIME) - quotedMessageTime?.setTextColor(context!!.resources.getColor(R.color.white60)) quoteColoredView?.setBackgroundResource(R.color.white) quotedChatMessageView?.visibility = View.VISIBLE diff --git a/app/src/main/java/com/nextcloud/talk/controllers/ChatController.kt b/app/src/main/java/com/nextcloud/talk/controllers/ChatController.kt index b76f7eeee..07ea1c994 100644 --- a/app/src/main/java/com/nextcloud/talk/controllers/ChatController.kt +++ b/app/src/main/java/com/nextcloud/talk/controllers/ChatController.kt @@ -1459,17 +1459,11 @@ class ChatController(args: Bundle) : BaseController(args), MessagesListAdapter messageInputView?.findViewById(R.id.quotedMessage)?.maxLines = 2 messageInputView?.findViewById(R.id.quotedMessage)?.ellipsize = TextUtils.TruncateAt.END messageInputView?.findViewById(R.id.quotedMessage)?.text = it.text - messageInputView?.findViewById(R.id.quotedMessageTime)?.text = DateFormatter.format(it.createdAt, DateFormatter.Template.TIME) messageInputView?.findViewById(R.id.quotedMessageAuthor)?.text = it.actorDisplayName ?: context!!.getText(R.string.nc_nick_guest) conversationUser?.let { currentUser -> - messageInputView?.findViewById(R.id.quotedUserAvatar)?.load(it.user.avatar) { - addHeader("Authorization", credentials!!) - transformations(CircleCropTransformation()) - } - chatMessage.imageUrl?.let { previewImageUrl -> messageInputView?.findViewById(R.id.quotedMessageImage)?.visibility = View.VISIBLE diff --git a/app/src/main/res/layout/controller_chat.xml b/app/src/main/res/layout/controller_chat.xml index 3aa0e8875..c60d594a8 100644 --- a/app/src/main/res/layout/controller_chat.xml +++ b/app/src/main/res/layout/controller_chat.xml @@ -66,8 +66,6 @@ android:layout_width="match_parent" android:layout_height="1dp" android:layout_above="@+id/messageInputView" - android:layout_marginLeft="16dp" - android:layout_marginRight="16dp" android:background="@color/controller_chat_separator" /> diff --git a/app/src/main/res/layout/item_message_quote.xml b/app/src/main/res/layout/item_message_quote.xml index 6fff959dd..12a538d68 100644 --- a/app/src/main/res/layout/item_message_quote.xml +++ b/app/src/main/res/layout/item_message_quote.xml @@ -1,4 +1,25 @@ - + + - - + android:background="@color/colorPrimary" /> + tools:text="Jane Doe" /> - - + android:contentDescription="@string/nc_message_quote_cancel_reply" + android:visibility="gone" + tools:visibility="visible" /> - \ No newline at end of file + diff --git a/app/src/main/res/layout/rv_item_browser_file.xml b/app/src/main/res/layout/rv_item_browser_file.xml index 32ad6be52..b8828500c 100644 --- a/app/src/main/res/layout/rv_item_browser_file.xml +++ b/app/src/main/res/layout/rv_item_browser_file.xml @@ -19,9 +19,10 @@ --> @@ -38,17 +39,18 @@ + app:tint="@color/favorite_icon_tint" /> @@ -66,7 +68,9 @@ android:marqueeRepeatLimit="1" android:paddingBottom="6dp" android:singleLine="true" - android:textSize="12sp" /> + android:textColor="@color/textColorMaxContrast" + android:textSize="12sp" + tools:text="3 minutes ago" /> + android:textColor="@color/conversation_item_header" + android:textSize="@dimen/two_line_primary_text_size" + tools:text="filename.md" /> - @@ -55,7 +54,7 @@ diff --git a/app/src/main/res/layout/rv_item_conversation_info_participant.xml b/app/src/main/res/layout/rv_item_conversation_info_participant.xml index b492e70c3..c98f43a7e 100644 --- a/app/src/main/res/layout/rv_item_conversation_info_participant.xml +++ b/app/src/main/res/layout/rv_item_conversation_info_participant.xml @@ -38,7 +38,8 @@ android:layout_height="12dp" android:layout_gravity="bottom|end" android:background="@drawable/shape_lock_bubble" - android:visibility="gone" /> + android:visibility="gone" + tools:visibility="visible" /> + android:visibility="gone" + tools:visibility="visible" /> + android:background="@drawable/shape_lock_bubble" + android:contentDescription="@string/password_protected" /> + app:tintMode="src_in" /> diff --git a/app/src/main/res/values-night/colors.xml b/app/src/main/res/values-night/colors.xml index 2d71a2a7e..71662a257 100644 --- a/app/src/main/res/values-night/colors.xml +++ b/app/src/main/res/values-night/colors.xml @@ -48,6 +48,8 @@ #373737 #D8D8D8 + #484848 + #D8D8D8 @android:color/holo_purple diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 380d4fd11..c6b24ee83 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -271,6 +271,7 @@ You sent a video. You sent an image. %1$s: %2$s + Cancel reply You: %1$s @@ -360,7 +361,10 @@ No phone number integration due to missing permissions Chat via %s Account not found - + + Favorite + Password protected + Avatar Account icon No personal info set diff --git a/scripts/analysis/lint-results.txt b/scripts/analysis/lint-results.txt index 80ceeef7d..c205f5007 100644 --- a/scripts/analysis/lint-results.txt +++ b/scripts/analysis/lint-results.txt @@ -1,2 +1,2 @@ DO NOT TOUCH; GENERATED BY DRONE - Lint Report: 3 errors and 126 warnings + Lint Report: 3 errors and 117 warnings