diff --git a/briar-android/src/main/java/org/briarproject/briar/android/blog/BlogPostViewHolder.java b/briar-android/src/main/java/org/briarproject/briar/android/blog/BlogPostViewHolder.java index 5fd3c8d4f..c16ca3f29 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/blog/BlogPostViewHolder.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/blog/BlogPostViewHolder.java @@ -171,9 +171,10 @@ class BlogPostViewHolder extends RecyclerView.ViewHolder { // TODO make author clickable #624 text.setText(c.getComment()); + Linkify.addLinks(text, Linkify.WEB_URLS); + text.setMovementMethod(null); if (fullText) { text.setTextIsSelectable(true); - Linkify.addLinks(text, Linkify.WEB_URLS); makeLinksClickable(text, listener::onLinkClick); } diff --git a/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationItemViewHolder.java b/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationItemViewHolder.java index 9a433b16a..d28ff2a12 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationItemViewHolder.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationItemViewHolder.java @@ -1,6 +1,7 @@ package org.briarproject.briar.android.conversation; import android.content.Context; +import android.text.util.Linkify; import android.view.View; import android.widget.ImageView; import android.widget.TextView; @@ -59,6 +60,7 @@ abstract class ConversationItemViewHolder extends ViewHolder { if (item.getText() != null) { text.setText(trim(item.getText())); + Linkify.addLinks(text, Linkify.WEB_URLS); makeLinksClickable(text, listener::onLinkClick); } diff --git a/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationNoticeViewHolder.java b/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationNoticeViewHolder.java index 7e5b5b94d..98abdcab1 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationNoticeViewHolder.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/conversation/ConversationNoticeViewHolder.java @@ -1,5 +1,6 @@ package org.briarproject.briar.android.conversation; +import android.text.util.Linkify; import android.view.View; import android.widget.TextView; @@ -41,6 +42,7 @@ class ConversationNoticeViewHolder extends ConversationItemViewHolder { } else { msgText.setVisibility(VISIBLE); msgText.setText(trim(text)); + Linkify.addLinks(msgText, Linkify.WEB_URLS); makeLinksClickable(msgText, listener::onLinkClick); layout.setBackgroundResource(isIncoming() ? R.drawable.notice_in_bottom : R.drawable.notice_out_bottom); diff --git a/briar-android/src/main/java/org/briarproject/briar/android/threaded/BaseThreadItemViewHolder.java b/briar-android/src/main/java/org/briarproject/briar/android/threaded/BaseThreadItemViewHolder.java index 9545ead34..fbd784528 100644 --- a/briar-android/src/main/java/org/briarproject/briar/android/threaded/BaseThreadItemViewHolder.java +++ b/briar-android/src/main/java/org/briarproject/briar/android/threaded/BaseThreadItemViewHolder.java @@ -4,6 +4,7 @@ import android.animation.Animator; import android.animation.ArgbEvaluator; import android.animation.ValueAnimator; import android.content.Context; +import android.text.util.Linkify; import android.view.View; import android.view.ViewGroup; import android.view.animation.AccelerateInterpolator; @@ -44,6 +45,7 @@ public abstract class BaseThreadItemViewHolder @CallSuper public void bind(I item, ThreadItemListener listener) { textView.setText(StringUtils.trim(item.getText())); + Linkify.addLinks(textView, Linkify.WEB_URLS); makeLinksClickable(textView, listener::onLinkClick); author.setAuthor(item.getAuthor(), item.getAuthorInfo()); diff --git a/briar-android/src/main/res/layout/list_item_conversation_msg_in_content.xml b/briar-android/src/main/res/layout/list_item_conversation_msg_in_content.xml index f0f226729..6c9dba640 100644 --- a/briar-android/src/main/res/layout/list_item_conversation_msg_in_content.xml +++ b/briar-android/src/main/res/layout/list_item_conversation_msg_in_content.xml @@ -42,7 +42,6 @@ android:layout_marginBottom="@dimen/message_bubble_padding_bottom_inner" android:textColor="?android:attr/textColorPrimary" android:textColorLink="@color/briar_text_link" - android:autoLink="web" app:layout_constrainedWidth="true" app:layout_constraintBottom_toTopOf="@+id/statusLayout" app:layout_constraintEnd_toEndOf="parent" diff --git a/briar-android/src/main/res/layout/list_item_conversation_msg_out.xml b/briar-android/src/main/res/layout/list_item_conversation_msg_out.xml index 144dfc1a9..82df8b666 100644 --- a/briar-android/src/main/res/layout/list_item_conversation_msg_out.xml +++ b/briar-android/src/main/res/layout/list_item_conversation_msg_out.xml @@ -50,7 +50,6 @@ android:layout_marginBottom="@dimen/message_bubble_padding_bottom_inner" android:textColor="@color/briar_text_primary_inverse" android:textColorLink="@color/briar_text_link_inverse" - android:autoLink="web" app:layout_constrainedWidth="true" app:layout_constraintBottom_toTopOf="@+id/statusLayout" app:layout_constraintEnd_toEndOf="parent" diff --git a/briar-android/src/main/res/layout/list_item_conversation_notice_in.xml b/briar-android/src/main/res/layout/list_item_conversation_notice_in.xml index 9fa638980..11fbe8f7c 100644 --- a/briar-android/src/main/res/layout/list_item_conversation_notice_in.xml +++ b/briar-android/src/main/res/layout/list_item_conversation_notice_in.xml @@ -22,7 +22,6 @@ android:background="@drawable/msg_in_top" android:elevation="@dimen/message_bubble_elevation" android:textColorLink="@color/briar_text_link" - android:autoLink="web" tools:text="Short message" tools:visibility="visible" /> diff --git a/briar-android/src/main/res/layout/list_item_conversation_notice_out.xml b/briar-android/src/main/res/layout/list_item_conversation_notice_out.xml index cb50d217b..70c7574fa 100644 --- a/briar-android/src/main/res/layout/list_item_conversation_notice_out.xml +++ b/briar-android/src/main/res/layout/list_item_conversation_notice_out.xml @@ -24,7 +24,6 @@ android:elevation="@dimen/message_bubble_elevation" android:textColor="@color/briar_text_primary_inverse" android:textColorLink="@color/briar_text_link_inverse" - android:autoLink="web" tools:text="This is a long long long message that spans over several lines.\n\nIt ends here." tools:visibility="visible" /> diff --git a/briar-android/src/main/res/layout/list_item_conversation_request.xml b/briar-android/src/main/res/layout/list_item_conversation_request.xml index 053c2fe40..56468fda0 100644 --- a/briar-android/src/main/res/layout/list_item_conversation_request.xml +++ b/briar-android/src/main/res/layout/list_item_conversation_request.xml @@ -23,7 +23,6 @@ android:elevation="@dimen/message_bubble_elevation" android:textColor="?android:attr/textColorPrimary" android:textColorLink="@color/briar_text_link" - android:autoLink="web" tools:text="Short message" tools:visibility="visible" /> diff --git a/briar-android/src/main/res/layout/list_item_thread.xml b/briar-android/src/main/res/layout/list_item_thread.xml index 65f3e6084..8c625d60a 100644 --- a/briar-android/src/main/res/layout/list_item_thread.xml +++ b/briar-android/src/main/res/layout/list_item_thread.xml @@ -115,7 +115,6 @@ android:textIsSelectable="true" android:textSize="@dimen/text_size_medium" android:textColorLink="@color/briar_text_link" - android:autoLink="web" app:layout_constraintBottom_toBottomOf="parent" app:layout_constraintEnd_toEndOf="parent" app:layout_constraintStart_toStartOf="parent"