From 685a39ba58fab82cc0486e49394a0f398bba8254 Mon Sep 17 00:00:00 2001 From: Alex Ning Date: Sat, 18 Jul 2020 21:56:39 +0800 Subject: [PATCH] Fix UI issues in post and comment item. Fix Reddit gallery post cannot be opened in PostCompactLayout when clicking the imageview. --- .gitignore | 1 + .../Adapter/PostRecyclerViewAdapter.java | 37 ++++++++++++++----- app/src/main/res/layout/item_comment.xml | 8 ++-- app/src/main/res/layout/item_post_compact.xml | 7 ++-- .../item_post_compact_right_thumbnail.xml | 7 ++-- .../res/layout/item_post_detail_gallery.xml | 23 ++++++------ ...tem_post_detail_image_and_gif_autoplay.xml | 23 ++++++------ .../main/res/layout/item_post_detail_link.xml | 23 ++++++------ .../item_post_detail_no_preview_link.xml | 23 ++++++------ .../main/res/layout/item_post_detail_text.xml | 23 ++++++------ ...item_post_detail_video_and_gif_preview.xml | 23 ++++++------ .../item_post_detail_video_autoplay.xml | 23 ++++++------ app/src/main/res/layout/item_post_gallery.xml | 18 +++++---- .../item_post_image_and_gif_autoplay.xml | 18 +++++---- app/src/main/res/layout/item_post_link.xml | 18 +++++---- .../res/layout/item_post_no_preview_link.xml | 18 +++++---- app/src/main/res/layout/item_post_text.xml | 18 +++++---- .../item_post_video_and_gif_preview.xml | 18 +++++---- .../layout/item_post_video_type_autoplay.xml | 18 +++++---- 19 files changed, 187 insertions(+), 160 deletions(-) diff --git a/.gitignore b/.gitignore index 7bc61541..5614bb96 100644 --- a/.gitignore +++ b/.gitignore @@ -20,6 +20,7 @@ bin/ gen/ out/ /app/release +/app/minifiedRelease # Gradle files .gradle/ diff --git a/app/src/main/java/ml/docilealligator/infinityforreddit/Adapter/PostRecyclerViewAdapter.java b/app/src/main/java/ml/docilealligator/infinityforreddit/Adapter/PostRecyclerViewAdapter.java index 45e6cc1d..b558db87 100644 --- a/app/src/main/java/ml/docilealligator/infinityforreddit/Adapter/PostRecyclerViewAdapter.java +++ b/app/src/main/java/ml/docilealligator/infinityforreddit/Adapter/PostRecyclerViewAdapter.java @@ -795,7 +795,11 @@ public class PostRecyclerViewAdapter extends PagedListAdapter + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true"/> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> @@ -22,51 +23,49 @@ + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0"/> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" /> + app:layout_constraintTop_toTopOf="parent" + app:layout_constrainedWidth="true" + app:layout_constraintHorizontal_bias="0" /> + app:layout_constraintHorizontal_bias="0" + app:layout_constrainedWidth="true" /> + app:layout_constraintTop_toTopOf="parent" + tools:visibility="visible" />