From 247e81ba00ba411f484b26d228d5fb0e0008383c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rafa=C5=82=20Borcz?= <RafalBO99@outlook.com>
Date: Sat, 8 Dec 2018 12:27:00 +0100
Subject: [PATCH] Show full subject name on expand (#193)

---
 .../grade/details/GradeDetailsHeader.kt       | 16 +++++++---
 .../main/res/layout/header_grade_details.xml  | 32 +++----------------
 2 files changed, 15 insertions(+), 33 deletions(-)

diff --git a/app/src/main/java/io/github/wulkanowy/ui/modules/grade/details/GradeDetailsHeader.kt b/app/src/main/java/io/github/wulkanowy/ui/modules/grade/details/GradeDetailsHeader.kt
index 8d8cac8f..5dec00a9 100644
--- a/app/src/main/java/io/github/wulkanowy/ui/modules/grade/details/GradeDetailsHeader.kt
+++ b/app/src/main/java/io/github/wulkanowy/ui/modules/grade/details/GradeDetailsHeader.kt
@@ -31,11 +31,12 @@ class GradeDetailsHeader(
 
     override fun bindViewHolder(adapter: FlexibleAdapter<IFlexible<*>>?, holder: ViewHolder, position: Int, payloads: MutableList<Any>?) {
         holder.run {
-            gradeHeaderSubject.text = subject
+            gradeHeaderSubject.apply {
+                text = subject
+                maxLines = if (isExpanded) 2 else 1
+            }
             gradeHeaderAverage.text = average
             gradeHeaderNumber.text = number
-            gradeHeaderPredicted.visibility = GONE
-            gradeHeaderFinal.visibility = GONE
             gradeHeaderNote.visibility = if (newGrades > 0) VISIBLE else GONE
 
             isViewExpandable = isExpandable
@@ -71,11 +72,16 @@ class GradeDetailsHeader(
             contentView.setOnClickListener(this)
         }
 
+        override val containerView: View
+            get() = contentView
+
         override fun isViewCollapsibleOnClick() = isViewExpandable
 
         override fun isViewExpandableOnClick() = isViewExpandable
 
-        override val containerView: View
-            get() = contentView
+        override fun onClick(view: View?) {
+            super.onClick(view)
+            mAdapter.getItem(adapterPosition)?.let { mAdapter.updateItem(it) }
+        }
     }
 }
diff --git a/app/src/main/res/layout/header_grade_details.xml b/app/src/main/res/layout/header_grade_details.xml
index 77e5a783..f92aec62 100644
--- a/app/src/main/res/layout/header_grade_details.xml
+++ b/app/src/main/res/layout/header_grade_details.xml
@@ -5,10 +5,10 @@
     android:layout_height="wrap_content"
     android:background="@drawable/ic_all_divider"
     android:foreground="?attr/selectableItemBackgroundBorderless"
-    android:paddingBottom="10dp"
     android:paddingLeft="16dp"
+    android:paddingTop="10dp"
     android:paddingRight="16dp"
-    android:paddingTop="10dp">
+    android:paddingBottom="10dp">
 
     <TextView
         android:id="@+id/gradeHeaderSubject"
@@ -16,8 +16,8 @@
         android:layout_height="wrap_content"
         android:layout_marginEnd="20dp"
         android:layout_marginRight="20dp"
-        android:layout_toLeftOf="@id/gradeHeaderNote"
         android:layout_toStartOf="@id/gradeHeaderNote"
+        android:layout_toLeftOf="@id/gradeHeaderNote"
         android:ellipsize="end"
         android:maxLines="1"
         android:text="@string/app_name"
@@ -38,8 +38,8 @@
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_below="@id/gradeHeaderSubject"
-        android:layout_marginLeft="10dp"
         android:layout_marginStart="10dp"
+        android:layout_marginLeft="10dp"
         android:layout_marginTop="5dp"
         android:layout_toEndOf="@+id/gradeHeaderAverage"
         android:layout_toRightOf="@+id/gradeHeaderAverage"
@@ -47,30 +47,6 @@
         android:textColor="?android:attr/android:textColorSecondary"
         android:textSize="12sp" />
 
-    <TextView
-        android:id="@+id/gradeHeaderPredicted"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_below="@id/gradeHeaderAverage"
-        android:layout_marginEnd="10dp"
-        android:layout_marginRight="10dp"
-        android:layout_marginTop="5dp"
-        android:text="@string/grade_predicted"
-        android:textColor="?android:attr/android:textColorSecondary"
-        android:textSize="12sp" />
-
-    <TextView
-        android:id="@+id/gradeHeaderFinal"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:layout_below="@id/gradeHeaderAverage"
-        android:layout_marginTop="5dp"
-        android:layout_toEndOf="@+id/gradeHeaderPredicted"
-        android:layout_toRightOf="@+id/gradeHeaderPredicted"
-        android:text="@string/grade_final"
-        android:textColor="?android:attr/android:textColorSecondary"
-        android:textSize="12sp" />
-
     <ImageView
         android:id="@+id/gradeHeaderNote"
         android:layout_width="wrap_content"