From ec27951850616f416ce7101b25c55a1836c46f52 Mon Sep 17 00:00:00 2001
From: ganfra <francois.ganard@gmail.com>
Date: Tue, 30 Oct 2018 11:47:05 +0100
Subject: [PATCH] Add fake filter view + refine UI

---
 ...raintLayout.kt => CheckableFrameLayout.kt} |  4 +-
 .../home/room/detail/RoomDetailFragment.kt    |  4 +-
 .../home/room/list/RoomSummaryItem.kt         |  6 +-
 app/src/main/res/drawable/bg_room_item.xml    | 11 ++--
 .../main/res/drawable/bg_search_edit_text.xml |  5 ++
 .../main/res/layout/fragment_room_detail.xml  |  4 +-
 .../main/res/layout/fragment_room_list.xml    | 48 ++++++++++++---
 app/src/main/res/layout/item_room.xml         | 61 +++++++++++--------
 .../main/res/layout/item_room_category.xml    | 14 +++--
 9 files changed, 104 insertions(+), 53 deletions(-)
 rename app/src/main/java/im/vector/riotredesign/core/platform/{CheckableConstraintLayout.kt => CheckableFrameLayout.kt} (90%)
 create mode 100644 app/src/main/res/drawable/bg_search_edit_text.xml

diff --git a/app/src/main/java/im/vector/riotredesign/core/platform/CheckableConstraintLayout.kt b/app/src/main/java/im/vector/riotredesign/core/platform/CheckableFrameLayout.kt
similarity index 90%
rename from app/src/main/java/im/vector/riotredesign/core/platform/CheckableConstraintLayout.kt
rename to app/src/main/java/im/vector/riotredesign/core/platform/CheckableFrameLayout.kt
index 967c0a8c..28aadd58 100644
--- a/app/src/main/java/im/vector/riotredesign/core/platform/CheckableConstraintLayout.kt
+++ b/app/src/main/java/im/vector/riotredesign/core/platform/CheckableFrameLayout.kt
@@ -1,12 +1,12 @@
 package im.vector.riotredesign.core.platform
 
 import android.content.Context
-import android.support.constraint.ConstraintLayout
 import android.util.AttributeSet
 import android.view.View
 import android.widget.Checkable
+import android.widget.FrameLayout
 
-class CheckableConstraintLayout : ConstraintLayout, Checkable {
+class CheckableFrameLayout : FrameLayout, Checkable {
 
     private var mChecked = false
 
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/RoomDetailFragment.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/RoomDetailFragment.kt
index ccb88653..efe9fcad 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/detail/RoomDetailFragment.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/detail/RoomDetailFragment.kt
@@ -64,10 +64,10 @@ class RoomDetailFragment : RiotFragment(), TimelineEventAdapter.Callback {
         layoutManager.stackFromEnd = true
         timelineAdapter.registerAdapterDataObserver(object : RecyclerView.AdapterDataObserver() {
             override fun onItemRangeInserted(positionStart: Int, itemCount: Int) {
-                /*if (layoutManager.findFirstVisibleItemPosition() == 0) {
+                if (layoutManager.findFirstVisibleItemPosition() == 0) {
                     layoutManager.scrollToPosition(0)
                 }
-                */
+
             }
         })
         recyclerView.layoutManager = layoutManager
diff --git a/app/src/main/java/im/vector/riotredesign/features/home/room/list/RoomSummaryItem.kt b/app/src/main/java/im/vector/riotredesign/features/home/room/list/RoomSummaryItem.kt
index c4b10606..c9f943c0 100644
--- a/app/src/main/java/im/vector/riotredesign/features/home/room/list/RoomSummaryItem.kt
+++ b/app/src/main/java/im/vector/riotredesign/features/home/room/list/RoomSummaryItem.kt
@@ -5,7 +5,7 @@ import android.widget.ImageView
 import android.widget.TextView
 import im.vector.riotredesign.R
 import im.vector.riotredesign.core.epoxy.KotlinModel
-import im.vector.riotredesign.core.platform.CheckableConstraintLayout
+import im.vector.riotredesign.core.platform.CheckableFrameLayout
 
 
 data class RoomSummaryItem(
@@ -17,11 +17,11 @@ data class RoomSummaryItem(
 
     private val titleView by bind<TextView>(R.id.titleView)
     private val avatarImageView by bind<ImageView>(R.id.avatarImageView)
-    private val rootView by bind<CheckableConstraintLayout>(R.id.itemRoomLayout)
+    private val rootView by bind<CheckableFrameLayout>(R.id.itemRoomLayout)
 
     override fun bind() {
         rootView.isChecked = isSelected
-        titleView.setOnClickListener { listener?.invoke() }
+        rootView.setOnClickListener { listener?.invoke() }
         titleView.text = title
         avatarImageView.setImageDrawable(avatarDrawable)
     }
diff --git a/app/src/main/res/drawable/bg_room_item.xml b/app/src/main/res/drawable/bg_room_item.xml
index 19bc072b..7a8d5873 100644
--- a/app/src/main/res/drawable/bg_room_item.xml
+++ b/app/src/main/res/drawable/bg_room_item.xml
@@ -1,18 +1,15 @@
 <?xml version="1.0" encoding="utf-8"?>
 <selector xmlns:android="http://schemas.android.com/apk/res/android">
-    <item android:state_pressed="true">
-        <shape>
-            <solid android:color="@android:color/white"/>
-        </shape>
-    </item>
+
     <item android:state_checked="true">
         <shape>
-            <solid android:color="@android:color/white"/>
+            <solid android:color="@android:color/white" />
+            <corners android:radius="4dp" />
         </shape>
     </item>
     <item>
         <shape>
-            <solid android:color="@android:color/transparent"/>
+            <solid android:color="@android:color/transparent" />
         </shape>
     </item>
 
diff --git a/app/src/main/res/drawable/bg_search_edit_text.xml b/app/src/main/res/drawable/bg_search_edit_text.xml
new file mode 100644
index 00000000..bba3fa26
--- /dev/null
+++ b/app/src/main/res/drawable/bg_search_edit_text.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android">
+    <solid android:color="@color/light_blue_grey" />
+    <corners android:radius="4dp" />
+</shape>
\ No newline at end of file
diff --git a/app/src/main/res/layout/fragment_room_detail.xml b/app/src/main/res/layout/fragment_room_detail.xml
index 3dff21eb..0f22a314 100644
--- a/app/src/main/res/layout/fragment_room_detail.xml
+++ b/app/src/main/res/layout/fragment_room_detail.xml
@@ -34,7 +34,7 @@
 
             <TextView
                 android:id="@+id/toolbarTitleView"
-                android:layout_width="wrap_content"
+                android:layout_width="0dp"
                 android:layout_height="wrap_content"
                 android:layout_marginEnd="8dp"
                 android:layout_marginStart="8dp"
@@ -50,7 +50,7 @@
 
             <TextView
                 android:id="@+id/toolbarSubtitleView"
-                android:layout_width="wrap_content"
+                android:layout_width="0dp"
                 android:layout_height="wrap_content"
                 android:layout_marginBottom="8dp"
                 android:layout_marginEnd="8dp"
diff --git a/app/src/main/res/layout/fragment_room_list.xml b/app/src/main/res/layout/fragment_room_list.xml
index e2dd58a9..0df248b8 100644
--- a/app/src/main/res/layout/fragment_room_list.xml
+++ b/app/src/main/res/layout/fragment_room_list.xml
@@ -1,13 +1,47 @@
 <?xml version="1.0" encoding="utf-8"?>
-<im.vector.riotredesign.core.platform.StateView xmlns:android="http://schemas.android.com/apk/res/android"
-    android:id="@+id/stateView"
+
+
+<android.support.constraint.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
     android:background="@color/pale_grey">
 
-    <com.airbnb.epoxy.EpoxyRecyclerView
-        android:id="@+id/epoxyRecyclerView"
-        android:layout_width="match_parent"
-        android:layout_height="match_parent" />
+    <EditText
+        android:id="@+id/filterRoomView"
+        android:layout_width="0dp"
+        android:layout_height="32dp"
+        android:layout_marginBottom="8dp"
+        android:layout_marginEnd="16dp"
+        android:layout_marginStart="8dp"
+        android:layout_marginTop="8dp"
+        android:background="@drawable/bg_search_edit_text"
+        android:drawableLeft="@drawable/ic_search_white"
+        android:drawablePadding="8dp"
+        android:drawableTint="#9fa9ba"
+        android:hint="Filter by name..."
+        android:paddingLeft="8dp"
+        app:layout_constraintBottom_toTopOf="@+id/stateView"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
 
-</im.vector.riotredesign.core.platform.StateView>
\ No newline at end of file
+    <im.vector.riotredesign.core.platform.StateView
+        android:id="@+id/stateView"
+        android:layout_width="0dp"
+        android:layout_height="0dp"
+        android:layout_marginBottom="0dp"
+        android:layout_marginEnd="0dp"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toBottomOf="@id/filterRoomView">
+
+        <com.airbnb.epoxy.EpoxyRecyclerView
+            android:id="@+id/epoxyRecyclerView"
+            android:layout_width="match_parent"
+            android:layout_height="match_parent" />
+
+    </im.vector.riotredesign.core.platform.StateView>
+
+</android.support.constraint.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/item_room.xml b/app/src/main/res/layout/item_room.xml
index c19e5a54..3f81df9e 100644
--- a/app/src/main/res/layout/item_room.xml
+++ b/app/src/main/res/layout/item_room.xml
@@ -1,36 +1,49 @@
 <?xml version="1.0" encoding="utf-8"?>
 
-<im.vector.riotredesign.core.platform.CheckableConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
+<im.vector.riotredesign.core.platform.CheckableFrameLayout
+    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:id="@+id/itemRoomLayout"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
+    android:paddingLeft="8dp"
+    android:paddingRight="16dp"
+    android:foreground="?attr/selectableItemBackground"
     android:background="@drawable/bg_room_item"
-    android:minHeight="48dp">
+    android:clickable="true"
+    android:focusable="true">
 
-    <ImageView
-        android:id="@+id/avatarImageView"
-        android:layout_width="32dp"
-        android:layout_height="32dp"
-        android:layout_marginStart="16dp"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent"
-        tools:src="@tools:sample/avatars" />
-
-    <TextView
-        android:id="@+id/titleView"
-        android:layout_width="0dp"
+    <android.support.constraint.ConstraintLayout
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:layout_marginStart="16dp"
         android:duplicateParentState="true"
-        android:textColor="@color/color_room_title"
-        android:textSize="14sp"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintEnd_toEndOf="parent"
-        app:layout_constraintStart_toEndOf="@id/avatarImageView"
-        app:layout_constraintTop_toTopOf="parent"
-        tools:text="@tools:sample/full_names" />
+        android:minHeight="48dp">
 
-</im.vector.riotredesign.core.platform.CheckableConstraintLayout>
\ No newline at end of file
+        <ImageView
+            android:id="@+id/avatarImageView"
+            android:layout_width="32dp"
+            android:layout_height="32dp"
+            android:layout_marginStart="16dp"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintTop_toTopOf="parent"
+            tools:src="@tools:sample/avatars" />
+
+        <TextView
+            android:id="@+id/titleView"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:layout_marginStart="16dp"
+            android:duplicateParentState="true"
+            android:textColor="@color/color_room_title"
+            android:textSize="14sp"
+            app:layout_constraintBottom_toBottomOf="parent"
+            app:layout_constraintEnd_toEndOf="parent"
+            app:layout_constraintStart_toEndOf="@id/avatarImageView"
+            app:layout_constraintTop_toTopOf="parent"
+            tools:text="@tools:sample/full_names" />
+
+    </android.support.constraint.ConstraintLayout>
+
+</im.vector.riotredesign.core.platform.CheckableFrameLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/item_room_category.xml b/app/src/main/res/layout/item_room_category.xml
index c0115b5a..480b7837 100644
--- a/app/src/main/res/layout/item_room_category.xml
+++ b/app/src/main/res/layout/item_room_category.xml
@@ -4,20 +4,22 @@
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
+    android:clickable="true"
+    android:focusable="true"
     android:gravity="center_vertical"
     android:minHeight="24dp"
     android:padding="16dp"
+    android:background="?attr/selectableItemBackground"
     tools:background="@color/pale_grey">
 
     <TextView
         android:id="@+id/roomCategoryTitleView"
         android:layout_width="0dp"
-        android:layout_height="wrap_content"
+        android:layout_height="0dp"
         android:layout_marginEnd="8dp"
-        android:gravity="center_vertical"
         android:drawableLeft="@drawable/ic_expand_more_white"
         android:drawableTint="@color/bluey_grey_two"
-        android:foregroundTint="@color/bluey_grey_two"
+        android:gravity="center_vertical"
         android:text="DIRECT MESSAGES"
         android:textColor="@color/bluey_grey_two"
         app:layout_constraintBottom_toBottomOf="parent"
@@ -28,9 +30,9 @@
 
     <ImageView
         android:id="@+id/roomCategoryAddButton"
-        android:layout_width="24dp"
-        android:layout_height="24dp"
-        android:layout_marginEnd="8dp"
+        android:layout_width="32dp"
+        android:layout_height="32dp"
+
         android:scaleType="centerInside"
         android:src="@drawable/ic_add_circle_white"
         android:tint="@color/bluey_grey_two"