From 603c761ebe5b1a61436ad557d169697b9704c996 Mon Sep 17 00:00:00 2001
From: Ezerous <ezerous@gmail.com>
Date: Thu, 29 Dec 2016 22:20:07 +0200
Subject: [PATCH] Recyclerview layout changes

---
 .../activities/main/forum/ForumAdapter.java   | 18 -----------
 .../activities/main/forum/ForumFragment.java  |  7 ++++-
 .../res/layout/fragment_forum_board_row.xml   | 16 ++++++----
 .../layout/fragment_forum_category_row.xml    | 30 +++++++------------
 .../main/res/layout/fragment_recent_row.xml   | 15 ++--------
 app/src/main/res/values/colors.xml            |  3 +-
 6 files changed, 31 insertions(+), 58 deletions(-)

diff --git a/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumAdapter.java b/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumAdapter.java
index e6842165..894f4bdf 100644
--- a/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumAdapter.java
+++ b/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumAdapter.java
@@ -69,34 +69,16 @@ class ForumAdapter extends ExpandableRecyclerAdapter<Category, Board, ForumAdapt
     class CategoryViewHolder extends ParentViewHolder {
 
         private TextView categoryTextview;
-        private ImageView arrowImageView;
 
         CategoryViewHolder(View itemView) {
             super(itemView);
             categoryTextview = (TextView) itemView.findViewById(R.id.category);
-
-            arrowImageView = (ImageView) itemView.findViewById(R.id.arrow); //todo animated arrow up/down
-            arrowImageView.setOnClickListener(new View.OnClickListener() {
-                @Override
-                public void onClick(View v) {
-                    if (isExpanded()) {
-                        collapseView();
-                    } else {
-                        expandView();
-                    }
-                }
-            });
         }
 
         void bind(Category category) {
             categoryTextview.setText(category.getName());
         }
 
-        @Override
-        public boolean shouldItemViewClickToggleExpansion() {
-            return false;
-        }
-
     }
 
     class BoardViewHolder extends ChildViewHolder {
diff --git a/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumFragment.java b/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumFragment.java
index 9b69b700..fba7fdb0 100644
--- a/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumFragment.java
+++ b/app/src/main/java/gr/thmmy/mthmmy/activities/main/forum/ForumFragment.java
@@ -2,6 +2,7 @@ package gr.thmmy.mthmmy.activities.main.forum;
 
 import android.os.AsyncTask;
 import android.os.Bundle;
+import android.support.v7.widget.DividerItemDecoration;
 import android.support.v7.widget.LinearLayoutManager;
 import android.support.v7.widget.RecyclerView;
 import android.view.LayoutInflater;
@@ -129,7 +130,11 @@ public class ForumFragment extends BaseFragment
             });
 
             RecyclerView recyclerView = (RecyclerView) rootView.findViewById(R.id.list);
-            recyclerView.setLayoutManager(new LinearLayoutManager(rootView.findViewById(R.id.list).getContext()));
+            LinearLayoutManager linearLayoutManager = new LinearLayoutManager(rootView.findViewById(R.id.list).getContext());
+            recyclerView.setLayoutManager(linearLayoutManager);
+            DividerItemDecoration dividerItemDecoration = new DividerItemDecoration(recyclerView.getContext(),
+                    linearLayoutManager.getOrientation());
+            recyclerView.addItemDecoration(dividerItemDecoration);
             recyclerView.setAdapter(forumAdapter);
 
         }
diff --git a/app/src/main/res/layout/fragment_forum_board_row.xml b/app/src/main/res/layout/fragment_forum_board_row.xml
index b87fd1c4..4fdace98 100644
--- a/app/src/main/res/layout/fragment_forum_board_row.xml
+++ b/app/src/main/res/layout/fragment_forum_board_row.xml
@@ -1,16 +1,20 @@
 <?xml version="1.0" encoding="utf-8"?>
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-    android:layout_width="wrap_content"
-              android:layout_height="wrap_content"
-              android:orientation="vertical"
-              android:paddingEnd="4dp"
-              android:paddingStart="4dp">
+    android:layout_width="match_parent"
+    android:layout_height="wrap_content"
+    android:orientation="vertical"
+    android:background="@color/primary_lighter">
 
     <TextView
         android:id="@+id/board"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:textAppearance="?attr/textAppearanceListItem"
-        android:textColor="@color/primary_text"/>
+        android:textColor="@color/accent"
+        android:textSize="16sp"
+        android:background="?attr/selectableItemBackground"
+        android:clickable="true"
+        android:gravity="center_horizontal"
+        android:padding="8dp"/>
 
 </LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/fragment_forum_category_row.xml b/app/src/main/res/layout/fragment_forum_category_row.xml
index 4be9c8a1..e235e98e 100644
--- a/app/src/main/res/layout/fragment_forum_category_row.xml
+++ b/app/src/main/res/layout/fragment_forum_category_row.xml
@@ -1,27 +1,19 @@
 <?xml version="1.0" encoding="utf-8"?>
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    xmlns:tools="http://schemas.android.com/tools"
-    android:layout_width="wrap_content"
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    android:paddingEnd="4dp"
-    android:paddingStart="4dp">
+    android:orientation="vertical"
+    android:background="@color/primary_light">
 
     <TextView
         android:id="@+id/category"
-        android:layout_width="wrap_content"
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:layout_alignParentStart="true"
         android:textAppearance="?attr/textAppearanceListItem"
-        android:textColor="@color/accent"/>
+        android:textColor="@color/primary_text"
+        android:textSize="20sp"
+        android:background="?attr/selectableItemBackground"
+        android:gravity="center_horizontal"
+        android:padding="8dp"/>
 
-    <ImageView
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_alignParentEnd="true"
-        app:srcCompat="@android:drawable/arrow_down_float"
-        android:id="@+id/arrow"
-        android:contentDescription="arrow" />
-
-
-</RelativeLayout>
\ No newline at end of file
+</LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/fragment_recent_row.xml b/app/src/main/res/layout/fragment_recent_row.xml
index cb4d23ca..930b3dd4 100644
--- a/app/src/main/res/layout/fragment_recent_row.xml
+++ b/app/src/main/res/layout/fragment_recent_row.xml
@@ -5,21 +5,10 @@
               android:orientation="vertical"
               >
 
-    <android.support.v7.widget.CardView
-        xmlns:card_view="http://schemas.android.com/apk/res-auto"
-        android:id="@+id/card_view"
-        android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_gravity="center"
-        android:foreground="?android:attr/selectableItemBackground"
-        card_view:cardCornerRadius="0dp"
-        card_view:cardElevation="0dp"
-        card_view:cardBackgroundColor="?attr/cardBackgroundColor">
-
         <RelativeLayout
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
-            android:layout_alignParentTop="true"
+            android:background="@color/primary_light"
             android:foreground="?android:attr/selectableItemBackground"
             android:paddingBottom="6dp"
             android:paddingLeft="10dp"
@@ -61,5 +50,5 @@
                 android:textColor="@color/secondary_text"
                 android:textStyle="italic"/>
         </RelativeLayout>
-    </android.support.v7.widget.CardView>
+
 </LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index 15e3ba63..8fa836d0 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -26,7 +26,8 @@
     <!--Dark-->
     <color name="primary">#2B2B2B</color>
     <color name="primary_dark">#333333</color>
-    <color name="primary_light">#3C3F41</color>
+    <color name="primary_light">#3C3C3C</color>
+    <color name="primary_lighter">#454545</color>
     <color name="accent">#26A69A</color>
     <color name="primary_text">#E7E7E7</color>
     <color name="secondary_text">#757575</color>