From 014b335835f200963059cb75b1f0c6c433eb4970 Mon Sep 17 00:00:00 2001
From: devKeeey <dev.keeey.ooody@gmail.com>
Date: Wed, 12 Feb 2020 19:31:41 +0900
Subject: [PATCH] fix datetext dismiss.

---
 .../ui/widget/SessionsItemDecoration.kt       | 94 ++++++++++++++-----
 1 file changed, 68 insertions(+), 26 deletions(-)

diff --git a/feature/session/src/main/java/io/github/droidkaigi/confsched2020/session/ui/widget/SessionsItemDecoration.kt b/feature/session/src/main/java/io/github/droidkaigi/confsched2020/session/ui/widget/SessionsItemDecoration.kt
index 2f765276d..679d281e1 100644
--- a/feature/session/src/main/java/io/github/droidkaigi/confsched2020/session/ui/widget/SessionsItemDecoration.kt
+++ b/feature/session/src/main/java/io/github/droidkaigi/confsched2020/session/ui/widget/SessionsItemDecoration.kt
@@ -59,21 +59,58 @@ class SessionsItemDecoration(
             val startDateTimeText =
                 calcDateTimeText(position, view, shouldShowDateText(position))
 
-            if (startDateTimeText.dateText != null) {
-                c.drawText(
-                    startDateTimeText.dateText.value,
-                    startDateTimeText.dateText.positionX,
-                    startDateTimeText.dateText.positionY,
-                    textPaint
-                )
+            when {
+                startDateTimeText.dateText.value.isNotEmpty() -> {
+                    c.drawText(
+                        startDateTimeText.dateText.value,
+                        startDateTimeText.dateText.positionX,
+                        startDateTimeText.dateText.positionY,
+                        textPaint
+                    )
+
+                    c.drawText(
+                        startDateTimeText.startTimeText.value,
+                        startDateTimeText.startTimeText.positionX,
+                        startDateTimeText.startTimeText.positionY,
+                        textPaint
+                    )
+                }
+                visibleSessionDate -> {
+                    when (index) {
+                        0 -> {
+                            c.drawText(
+                                sessionItem.startSessionDate(),
+                                startDateTimeText.dateText.positionX,
+                                startDateTimeText.dateText.positionY,
+                                textPaint
+                            )
+
+                            c.drawText(
+                                startDateTimeText.startTimeText.value,
+                                startDateTimeText.startTimeText.positionX,
+                                (startDateTimeText.startTimeText.positionY + sessionTimeTextHeightInPx),
+                                textPaint
+                            )
+                        }
+                        else -> {
+                            c.drawText(
+                                startDateTimeText.startTimeText.value,
+                                startDateTimeText.startTimeText.positionX,
+                                startDateTimeText.startTimeText.positionY,
+                                textPaint
+                            )
+                        }
+                    }
+                }
+                else -> {
+                    c.drawText(
+                        startDateTimeText.startTimeText.value,
+                        startDateTimeText.startTimeText.positionX,
+                        startDateTimeText.startTimeText.positionY,
+                        textPaint
+                    )
+                }
             }
-
-            c.drawText(
-                startDateTimeText.startTimeText.value,
-                startDateTimeText.startTimeText.positionX,
-                startDateTimeText.startTimeText.positionY,
-                textPaint
-            )
         }
     }
 
@@ -105,24 +142,29 @@ class SessionsItemDecoration(
         } else null
 
         // session date text
-        val dateText = if (shouldShowDateText) {
-
-            var sessionDateTextPositionY = view.top.coerceAtLeast(
-                sessionTimeTextMarginTopInPx.toInt()
-            ) + sessionTimeTextHeightInPx
+        var sessionDateTextPositionY = view.top.coerceAtLeast(
+            sessionTimeTextMarginTopInPx.toInt()
+        ) + sessionTimeTextHeightInPx
 
-            if (sessionItem.startSessionTime() != nextSessionItem?.startSessionTime()) {
-                sessionDateTextPositionY = sessionDateTextPositionY.coerceAtMost(
-                    view.bottom.toFloat() - sessionTimeTextHeightInPx
-                )
-            }
+        if (sessionItem.startSessionTime() != nextSessionItem?.startSessionTime()) {
+            sessionDateTextPositionY = sessionDateTextPositionY.coerceAtMost(
+                view.bottom.toFloat() - sessionTimeTextHeightInPx
+            )
+        }
 
+        val dateText = if (shouldShowDateText) {
             PositionalText(
                 value = sessionItem.startSessionDate(),
                 positionX = sessionTimeTextMarginStartInPx,
                 positionY = sessionDateTextPositionY
             )
-        } else null
+        } else {
+            PositionalText(
+                value = "",
+                positionX = sessionTimeTextMarginStartInPx,
+                positionY = sessionDateTextPositionY
+            )
+        }
 
         val sessionTimeTextMarginTop = if (shouldShowDateText) {
             sessionTimeTextMarginTopInPx + sessionTimeTextHeightInPx
@@ -149,7 +191,7 @@ class SessionsItemDecoration(
     }
 
     private data class StartDateTimeText(
-        val dateText: PositionalText?,
+        val dateText: PositionalText,
         val startTimeText: PositionalText
     )