From a1a4f323028a2947e84cec62192484ffe3458bb9 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 08:58:31 +0100
Subject: [PATCH 01/18] Fix LiveDataAdapter

---
 .../ui/adapter/components/LiveDataAdapter.kt  | 34 ++++++++++++++-----
 .../bottomsheet/BottomSheetListFragment.kt    |  3 +-
 .../BottomSheetListGameFragment.kt            |  3 +-
 3 files changed, 29 insertions(+), 11 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt b/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt
index 2528a469..5cb5cbfe 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt
@@ -9,6 +9,11 @@ import androidx.recyclerview.widget.RecyclerView
 import net.pokeranalytics.android.R
 import net.pokeranalytics.android.ui.view.RowViewType
 
+enum class LiveDataViewType {
+	DATA,
+	BOTTOM_SHEET_DATA
+}
+
 interface LiveDataDataSource {
     val title: String
     val primaryKey: String
@@ -20,17 +25,20 @@ interface LiveDataDelegate {
     fun size() : Int
 }
 
-class LiveDataAdapter(var adapterDelegate: LiveDataDelegate, var layout: Int? = null) : RecyclerView.Adapter<RecyclerView.ViewHolder>() {
+class LiveDataAdapter(var adapterDelegate: LiveDataDelegate, var liveDataViewType: LiveDataViewType? = LiveDataViewType.DATA) : RecyclerView.Adapter<RecyclerView.ViewHolder>() {
 
     inner class DataViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView) {
         fun bind(row: LiveDataDataSource, listener: View.OnClickListener) {
-	        try {
-		        itemView.findViewById<AppCompatTextView>(R.id.title).text = row.title
-		        itemView.findViewById<ConstraintLayout>(R.id.container).setOnClickListener(listener)
-	        } catch (e: Exception) {
-		        e.printStackTrace()
+	        when(liveDataViewType) {
+		        LiveDataViewType.DATA -> {
+			        itemView.findViewById<AppCompatTextView>(R.id.rowTitle_title).text = row.title
+			        itemView.findViewById<ConstraintLayout>(R.id.rowTitle_container).setOnClickListener(listener)
+		        }
+		        LiveDataViewType.BOTTOM_SHEET_DATA -> {
+			        itemView.findViewById<AppCompatTextView>(R.id.title).text = row.title
+			        itemView.findViewById<ConstraintLayout>(R.id.container).setOnClickListener(listener)
+		        }
 	        }
-
         }
     }
 
@@ -39,8 +47,16 @@ class LiveDataAdapter(var adapterDelegate: LiveDataDelegate, var layout: Int? =
     }
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RecyclerView.ViewHolder {
-	    val layoutToInflate = layout ?: R.layout.row_title
-        return DataViewHolder(LayoutInflater.from(parent.context).inflate(layoutToInflate, parent, false))
+	    val layoutToInflate = when(liveDataViewType) {
+		    LiveDataViewType.DATA -> {
+			    R.layout.row_title
+		    }
+		    LiveDataViewType.BOTTOM_SHEET_DATA -> {
+			    R.layout.row_bottom_sheet_title
+		    }
+		    else ->  R.layout.row_title
+	    }
+	    return DataViewHolder(LayoutInflater.from(parent.context).inflate(layoutToInflate, parent, false))
     }
 
     override fun getItemCount(): Int {
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt
index 7f083f62..2131c53e 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt
@@ -12,6 +12,7 @@ import net.pokeranalytics.android.model.realm.Game
 import net.pokeranalytics.android.ui.adapter.components.LiveDataAdapter
 import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
 import net.pokeranalytics.android.ui.adapter.components.LiveDataDelegate
+import net.pokeranalytics.android.ui.adapter.components.LiveDataViewType
 
 class BottomSheetListFragment : BottomSheetFragment(), LiveDataDelegate {
 
@@ -69,7 +70,7 @@ class BottomSheetListFragment : BottomSheetFragment(), LiveDataDelegate {
 		LayoutInflater.from(requireContext()).inflate(R.layout.bottom_sheet_list, view?.bottomSheetContainer, true)
 
 		val viewManager = LinearLayoutManager(requireContext())
-		dataAdapter = LiveDataAdapter(this, R.layout.row_bottom_sheet_title)
+		dataAdapter = LiveDataAdapter(this, LiveDataViewType.BOTTOM_SHEET_DATA)
 
 		reyclerView.apply {
 			setHasFixedSize(true)
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
index 08fcce2d..f97672fd 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
@@ -13,6 +13,7 @@ import net.pokeranalytics.android.ui.adapter.LimitTypesAdapter
 import net.pokeranalytics.android.ui.adapter.components.LiveDataAdapter
 import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
 import net.pokeranalytics.android.ui.adapter.components.LiveDataDelegate
+import net.pokeranalytics.android.ui.adapter.components.LiveDataViewType
 import timber.log.Timber
 
 
@@ -90,7 +91,7 @@ class BottomSheetListGameFragment : BottomSheetFragment(), LiveDataDelegate {
 
 
 		val viewManager2 = LinearLayoutManager(requireContext())
-		dataAdapter = LiveDataAdapter(this, R.layout.row_bottom_sheet_title)
+		dataAdapter = LiveDataAdapter(this, LiveDataViewType.BOTTOM_SHEET_DATA)
 
 		recyclerView2.apply {
 			setHasFixedSize(true)

From 1beed62fcf94d02ba8a682248b918dd032b5467c Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 09:41:14 +0100
Subject: [PATCH 02/18] Improve editable data UI

---
 .../ui/fragment/EditableDataFragment.kt       | 115 +++++++++++-------
 .../res/layout/fragment_editable_data.xml     |  87 ++++++-------
 app/src/main/res/menu/editable_data.xml       |  17 +++
 app/src/main/res/values/styles.xml            |  13 +-
 4 files changed, 138 insertions(+), 94 deletions(-)
 create mode 100644 app/src/main/res/menu/editable_data.xml

diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
index 62e25c8f..27c2dcc4 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
@@ -1,10 +1,7 @@
 package net.pokeranalytics.android.ui.fragment
 
-import android.content.DialogInterface
 import android.os.Bundle
-import android.view.LayoutInflater
-import android.view.View
-import android.view.ViewGroup
+import android.view.*
 import android.widget.Toast
 import androidx.appcompat.app.AlertDialog
 import androidx.recyclerview.widget.LinearLayoutManager
@@ -15,7 +12,10 @@ import net.pokeranalytics.android.R
 import net.pokeranalytics.android.model.LiveData
 import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.ui.activity.components.PokerAnalyticsActivity
-import net.pokeranalytics.android.ui.adapter.components.*
+import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
+import net.pokeranalytics.android.ui.adapter.components.RowRepresentableAdapter
+import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
+import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDelegate
 import net.pokeranalytics.android.ui.fragment.components.PokerAnalyticsFragment
 import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheetDelegate
 import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheetFragment
@@ -25,9 +25,10 @@ import timber.log.Timber
 
 class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, BottomSheetDelegate {
 
-    private lateinit var item: RealmObject
+	private lateinit var item: RealmObject
 	private lateinit var liveDataType: LiveData
 	private lateinit var rowRepresentableAdapter: RowRepresentableAdapter
+	private var editableMenu: Menu? = null
 
 	override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
 		return inflater.inflate(R.layout.fragment_editable_data, container, false)
@@ -39,6 +40,22 @@ class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate,
 		initUI()
 	}
 
+	override fun onCreateOptionsMenu(menu: Menu?, inflater: MenuInflater?) {
+		inflater?.inflate(R.menu.editable_data, menu)
+		this.editableMenu = menu
+		updateMenuUI()
+		super.onCreateOptionsMenu(menu, inflater)
+	}
+
+
+	override fun onOptionsItemSelected(item: MenuItem?): Boolean {
+		when (item!!.itemId) {
+			R.id.save -> saveData()
+			R.id.delete -> deleteData()
+		}
+		return true
+	}
+
 	override fun onRowSelected(row: RowRepresentable) {
 		BottomSheetFragment.create(fragmentManager, row, this, (this.item as RowEditable).getBottomSheetData(row))
 	}
@@ -74,6 +91,7 @@ class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate,
 		val activity = activity as PokerAnalyticsActivity
 		activity.setSupportActionBar(toolbar)
 		activity.supportActionBar?.setDisplayHomeAsUpEnabled(true)
+		setHasOptionsMenu(true)
 
 		val viewManager = LinearLayoutManager(requireContext())
 
@@ -81,49 +99,60 @@ class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate,
 			setHasFixedSize(true)
 			layoutManager = viewManager
 		}
+	}
+
+	/**
+	 * Update menu UI
+	 */
+	private fun updateMenuUI() {
+		editableMenu?.findItem(R.id.delete)?.isVisible = item.isManaged
+		editableMenu?.findItem(R.id.save)?.isVisible = true
+	}
+
+	/**
+	 * Save data
+	 */
+	private fun saveData() {
+		if ((this.item as ObjectSavable).isValidForSave()) {
+			this.getRealm().executeTransaction {
+				it.copyToRealmOrUpdate(this.item)
+			}
+			activity?.finish()
+		} else {
+			val builder = AlertDialog.Builder(requireContext())
+				.setMessage(R.string.empty_name_for_br_error)
+				.setNegativeButton(R.string.ok, null)
+			builder.show()
+		}
 
-		this.saveButton.text = this.saveButton.context.getString(R.string.save)
-		this.saveButton.setOnClickListener {
+	}
 
-			if ((this.item as ObjectSavable).isValidForSave()) {
-				this.getRealm().executeTransaction {
-					it.copyToRealmOrUpdate(this.item)
+	/**
+	 * Delete data
+	 */
+	private fun deleteData() {
+
+		val builder = AlertDialog.Builder(requireContext())
+		builder.setTitle(R.string.warning)
+			.setMessage(R.string.are_you_sure_you_want_to_do_that_)
+			.setNegativeButton(R.string.no, null)
+			.setPositiveButton(R.string.yes) { dialog, id ->
+				if (this.item.isManaged) {
+					Toast.makeText(requireContext(), "isManaged", Toast.LENGTH_SHORT).show()
+					Timber.d("is managed")
+					this.getRealm().executeTransaction {
+						this.liveDataType.deleteData(it, (this.item as LiveDataDataSource))
+					}
+				} else {
+					Toast.makeText(requireContext(), "isNotManaged", Toast.LENGTH_SHORT).show()
+					Timber.d("is not managed")
 				}
 				this.activity?.let {
 					it.finish()
 				}
-			} else {
-				val builder = AlertDialog.Builder(it.context)
-					.setMessage(R.string.empty_name_for_br_error)
-					.setNegativeButton(R.string.ok, null)
-				builder.show()
 			}
-		}
+		builder.show()
 
-		this.deleteButton.text = this.deleteButton.context.getString(R.string.delete)
-		this.deleteButton.setOnClickListener {
-			val builder = AlertDialog.Builder(it.context)
-			builder.setTitle(R.string.warning)
-				.setMessage(R.string.are_you_sure_you_want_to_do_that_)
-				.setNeutralButton(R.string.no, null)
-				.setNegativeButton(R.string.yes, DialogInterface.OnClickListener { dialog, id ->
-
-					if (this.item.isManaged) {
-						Toast.makeText(requireContext(), "isManaged", Toast.LENGTH_SHORT).show()
-						Timber.d("is managed")
-						this.getRealm().executeTransaction {
-							this.liveDataType.deleteData(it, (this.item as LiveDataDataSource))
-						}
- 					} else {
-						Toast.makeText(requireContext(), "isNotManaged", Toast.LENGTH_SHORT).show()
-						Timber.d("is not managed")
-					}
-					this.activity?.let {
-						it.finish()
-					}
-				})
-			builder.show()
-		}
 	}
 
 	/**
@@ -131,8 +160,8 @@ class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate,
 	 */
 	fun setData(dataType: Int, primaryKey: String?) {
 		this.liveDataType = LiveData.values()[dataType]
-		var proxyItem : RealmObject? = this.liveDataType.getData(this.getRealm(), primaryKey)
-        proxyItem?.let {
+		var proxyItem: RealmObject? = this.liveDataType.getData(this.getRealm(), primaryKey)
+		proxyItem?.let {
 			this.appBar.toolbar.title = "Update ${this.liveDataType.name.toLowerCase().capitalize()}"
 		} ?: run {
 			this.appBar.toolbar.title = "New ${this.liveDataType.name.toLowerCase().capitalize()}"
diff --git a/app/src/main/res/layout/fragment_editable_data.xml b/app/src/main/res/layout/fragment_editable_data.xml
index ed7b8bf4..bab049a9 100644
--- a/app/src/main/res/layout/fragment_editable_data.xml
+++ b/app/src/main/res/layout/fragment_editable_data.xml
@@ -1,74 +1,61 @@
 <?xml version="1.0" encoding="utf-8"?>
 <androidx.coordinatorlayout.widget.CoordinatorLayout 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="match_parent"
-													 android:layout_height="match_parent">
+	xmlns:app="http://schemas.android.com/apk/res-auto"
+	xmlns:tools="http://schemas.android.com/tools"
+	android:layout_width="match_parent"
+	android:layout_height="match_parent">
 
 	<androidx.core.widget.NestedScrollView
-			android:layout_width="match_parent"
-			android:layout_height="match_parent"
-			android:fillViewport="true"
-			app:layout_behavior="@string/appbar_scrolling_view_behavior" android:id="@+id/nestedScrollView">
+		android:id="@+id/nestedScrollView"
+		android:layout_width="match_parent"
+		android:layout_height="match_parent"
+		android:fillViewport="true"
+		app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
 		<androidx.constraintlayout.widget.ConstraintLayout
-				android:layout_width="match_parent"
-				android:layout_height="wrap_content">
+			android:layout_width="match_parent"
+			android:layout_height="wrap_content">
 
 			<androidx.recyclerview.widget.RecyclerView
-					android:id="@+id/recyclerView"
-					android:layout_width="0dp"
-					android:layout_height="0dp"
-					app:layout_constraintBottom_toBottomOf="parent"
-					app:layout_constraintEnd_toEndOf="parent"
-					app:layout_constraintStart_toStartOf="parent"
-					app:layout_constraintTop_toTopOf="parent"/>
+				android:id="@+id/recyclerView"
+				android:layout_width="0dp"
+				android:layout_height="0dp"
+				app:layout_constraintBottom_toBottomOf="parent"
+				app:layout_constraintEnd_toEndOf="parent"
+				app:layout_constraintStart_toStartOf="parent"
+				app:layout_constraintTop_toTopOf="parent" />
 
 		</androidx.constraintlayout.widget.ConstraintLayout>
 
 	</androidx.core.widget.NestedScrollView>
 
 	<com.google.android.material.appbar.AppBarLayout
-			android:id="@+id/appBar"
-			android:layout_width="match_parent"
-			android:layout_height="128dp"
-			android:theme="@style/ThemeOverlay.AppCompat.Dark.ActionBar">
+		android:id="@+id/appBar"
+		android:layout_width="match_parent"
+		android:layout_height="128dp"
+		android:theme="@style/ThemeOverlay.AppCompat.Dark.ActionBar">
 
 		<com.google.android.material.appbar.CollapsingToolbarLayout
-				android:id="@+id/collapsingToolbar"
-				android:layout_width="match_parent"
-				android:layout_height="match_parent"
-				app:contentScrim="?attr/colorPrimary"
-				app:expandedTitleGravity="bottom"
-				app:expandedTitleMarginStart="72dp"
-				app:expandedTitleTextAppearance="@style/PokerAnalyticsTheme.Toolbar.ExpandedTitleAppearance"
-				app:collapsedTitleTextAppearance="@style/PokerAnalyticsTheme.Toolbar.CollapsedTitleAppearance"
-				app:layout_scrollFlags="scroll|exitUntilCollapsed|snap">
+			android:id="@+id/collapsingToolbar"
+			android:layout_width="match_parent"
+			android:layout_height="match_parent"
+			app:collapsedTitleTextAppearance="@style/PokerAnalyticsTheme.Toolbar.CollapsedTitleAppearance"
+			app:contentScrim="?attr/colorPrimary"
+			app:expandedTitleGravity="bottom"
+			app:expandedTitleMarginStart="72dp"
+			app:expandedTitleTextAppearance="@style/PokerAnalyticsTheme.Toolbar.ExpandedTitleAppearance"
+			app:layout_scrollFlags="scroll|exitUntilCollapsed|snap">
 
 			<androidx.appcompat.widget.Toolbar
-					android:id="@+id/toolbar"
-					android:layout_width="match_parent"
-					android:layout_height="?attr/actionBarSize"
-					app:title="Poker Analytics"
-					app:titleTextColor="@color/white"
-					app:layout_collapseMode="pin"/>
+				android:id="@+id/toolbar"
+				android:layout_width="match_parent"
+				android:layout_height="?attr/actionBarSize"
+				app:layout_collapseMode="pin"
+				app:title="Poker Analytics"
+				app:titleTextColor="@color/white" />
 
 		</com.google.android.material.appbar.CollapsingToolbarLayout>
 
 	</com.google.android.material.appbar.AppBarLayout>
-	<Button
-			tools:text="Save"
-			android:layout_width="wrap_content"
-			android:layout_height="wrap_content"
-			android:id="@+id/saveButton"
-			app:layout_anchorGravity="right|top"
-			app:layout_anchor="@+id/appBar"/>
-	<Button
-			tools:text="Delete"
-			android:layout_width="wrap_content"
-			android:layout_height="wrap_content"
-			android:id="@+id/deleteButton"
-			app:layout_anchorGravity="bottom|right"
-			app:layout_anchor="@+id/nestedScrollView"/>
 
 </androidx.coordinatorlayout.widget.CoordinatorLayout>
\ No newline at end of file
diff --git a/app/src/main/res/menu/editable_data.xml b/app/src/main/res/menu/editable_data.xml
new file mode 100644
index 00000000..34f90d83
--- /dev/null
+++ b/app/src/main/res/menu/editable_data.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="utf-8"?>
+<menu xmlns:android="http://schemas.android.com/apk/res/android"
+	xmlns:app="http://schemas.android.com/apk/res-auto">
+
+	<item
+		android:id="@+id/delete"
+		android:icon="@drawable/ic_outline_delete"
+		android:title="@string/delete"
+		app:showAsAction="ifRoom" />
+
+	<item
+		android:id="@+id/save"
+		android:title="@string/save"
+		android:icon="@drawable/ic_check"
+		app:showAsAction="ifRoom" />
+
+</menu>
\ No newline at end of file
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 407bcfa9..f7336c04 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -14,7 +14,7 @@
 		<item name="bottomAppBarStyle">@style/PokerAnalyticsTheme.BottomAppBar</item>
 		<item name="editTextStyle">@style/PokerAnalyticsTheme.EditText</item>
 		<item name="android:textViewStyle">@style/PokerAnalyticsTheme.TextView</item>
-
+		<item name="alertDialogTheme">@style/PokerAnalyticsTheme.AlertDialog</item>
 
 	</style>
 
@@ -105,24 +105,29 @@
 		<item name="android:maxLines">1</item>
 		<item name="android:ellipsize">end</item>
 	</style>
+
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.Date">
 		<item name="android:textSize">12sp</item>
 		<item name="android:fontFamily">@font/roboto</item>r
 	</style>
+
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.DateNumber">
 		<item name="android:textSize">18sp</item>
 		<item name="android:fontFamily">@font/roboto_black</item>
 		<item name="android:letterSpacing">0.05</item>
 	</style>
+
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.Title">
 		<item name="android:textSize">16sp</item>
 		<item name="android:fontFamily">@font/roboto</item>
 	</style>
+
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.Subtitle">
 		<item name="android:textSize">12sp</item>
 		<item name="android:fontFamily">@font/roboto</item>
 		<item name="android:textColor">@color/kaki_lighter</item>
 	</style>
+
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.Result">
 		<item name="android:textSize">20sp</item>
 		<item name="android:fontFamily">@font/roboto_light</item>
@@ -130,6 +135,12 @@
 	</style>
 
 
+	<style name="PokerAnalyticsTheme.AlertDialog" parent="Theme.AppCompat.Light.Dialog.Alert">
+		<item name="colorAccent">@color/colorAccent</item>
+		<item name="android:textColorPrimary">@color/black</item>
+	</style>
+
+
 	<!-- EditText -->
 
 	<style name="PokerAnalyticsTheme.EditText" parent="Widget.AppCompat.EditText">

From b1b4db291cb2c1346e77e9fbf997cfdbfc079b5d Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 09:41:27 +0100
Subject: [PATCH 03/18] Update Style

---
 app/src/main/res/values/styles.xml | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index f7336c04..e3f57ac0 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -2,7 +2,7 @@
 
 	<!-- PokerAnalytics application theme -->
 	<style name="PokerAnalyticsTheme" parent="Theme.MaterialComponents.Light.NoActionBar">
-		<!-- Customize your theme here. -->
+
 		<item name="colorPrimary">@color/colorPrimary</item>
 		<item name="colorPrimaryDark">@color/colorPrimaryDark</item>
 		<item name="colorAccent">@color/colorAccent</item>
@@ -135,9 +135,16 @@
 	</style>
 
 
+
+	<style name="MyBorderlessButton">
+		<!-- Set background drawable and text size of the buttons here -->
+		<item name="android:background">@color/black</item>
+	</style>
+
 	<style name="PokerAnalyticsTheme.AlertDialog" parent="Theme.AppCompat.Light.Dialog.Alert">
 		<item name="colorAccent">@color/colorAccent</item>
 		<item name="android:textColorPrimary">@color/black</item>
+
 	</style>
 
 

From 0aa5d50f026597ae3004129b3d46774cbc32984e Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 10:01:24 +0100
Subject: [PATCH 04/18] Update alert dialog style

---
 app/src/main/res/values/styles.xml | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index e3f57ac0..b69a9439 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -135,16 +135,12 @@
 	</style>
 
 
+	<!-- Alert Dialog -->
 
-	<style name="MyBorderlessButton">
-		<!-- Set background drawable and text size of the buttons here -->
-		<item name="android:background">@color/black</item>
-	</style>
-
-	<style name="PokerAnalyticsTheme.AlertDialog" parent="Theme.AppCompat.Light.Dialog.Alert">
-		<item name="colorAccent">@color/colorAccent</item>
-		<item name="android:textColorPrimary">@color/black</item>
-
+	<style name="PokerAnalyticsTheme.AlertDialog" parent="Theme.MaterialComponents.Dialog">
+		<item name="colorAccent">@color/green</item>
+		<item name="android:textColorPrimary">@color/white</item>
+		<item name="android:background">@color/gray_dark</item>
 	</style>
 
 

From 2a1ac08f232daee4487727df7ac6a8c9a2f126dc Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 10:04:40 +0100
Subject: [PATCH 05/18] Clean code

---
 app/src/main/res/values/styles.xml | 2 --
 1 file changed, 2 deletions(-)

diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index b69a9439..d124ef6f 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -136,7 +136,6 @@
 
 
 	<!-- Alert Dialog -->
-
 	<style name="PokerAnalyticsTheme.AlertDialog" parent="Theme.MaterialComponents.Dialog">
 		<item name="colorAccent">@color/green</item>
 		<item name="android:textColorPrimary">@color/white</item>
@@ -145,7 +144,6 @@
 
 
 	<!-- EditText -->
-
 	<style name="PokerAnalyticsTheme.EditText" parent="Widget.AppCompat.EditText">
 		<item name="android:textColor">@color/white</item>
 		<item name="android:fontFamily">@font/roboto</item>

From 053d44479b8c1d9328657c8ca606265a41aacf99 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 12:22:13 +0100
Subject: [PATCH 06/18] Add Places dependence

---
 app/build.gradle                 | 7 +++++++
 app/src/main/AndroidManifest.xml | 3 +++
 2 files changed, 10 insertions(+)

diff --git a/app/build.gradle b/app/build.gradle
index 2b5195f7..16f3127c 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -34,6 +34,10 @@ android {
         }
     }
 
+    configurations {
+        all*.exclude group: 'com.google.guava', module: 'listenablefuture'
+    }
+
 }
 
 dependencies {
@@ -47,6 +51,9 @@ dependencies {
     implementation 'androidx.constraintlayout:constraintlayout:1.1.3'
     implementation 'androidx.lifecycle:lifecycle-extensions:2.0.0'
 
+    // Places
+    implementation 'com.google.android.libraries.places:places:1.0.0'
+
     // Firebase
     implementation 'com.google.firebase:firebase-core:16.0.7'
 
diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index 23186ad9..3354a3c9 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -2,6 +2,9 @@
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
           package="net.pokeranalytics.android">
 
+	<uses-permission android:name="android.permission.INTERNET"/>
+	<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION"/>
+
 	<application
 			android:allowBackup="true"
 			android:icon="@mipmap/ic_launcher"

From 3f8cc90a08a62486b1b675a696b231fe2f0948e5 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 12:22:20 +0100
Subject: [PATCH 07/18] Improve Session UI

---
 .../android/model/realm/Session.kt            | 36 ++++++--
 .../android/ui/fragment/SessionFragment.kt    | 18 ++++
 .../android/ui/view/SessionRowView.kt         |  2 +-
 .../pokeranalytics/android/util/Extensions.kt | 17 ++++
 app/src/main/res/layout/fragment_session.xml  | 91 ++++++++++++++++++-
 app/src/main/res/layout/row_session_view.xml  |  6 +-
 6 files changed, 157 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index bab4effd..e023d1e5 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -1,5 +1,6 @@
 package net.pokeranalytics.android.model.realm
 
+import android.content.Context
 import android.text.InputType
 import io.realm.*
 import io.realm.annotations.Ignore
@@ -16,6 +17,7 @@ import net.pokeranalytics.android.ui.view.RowEditable
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.SessionRow
 import net.pokeranalytics.android.util.data.sessionDao
+import net.pokeranalytics.android.util.getDuration
 import net.pokeranalytics.android.util.short
 import net.pokeranalytics.android.util.toCurrency
 import java.util.*
@@ -138,6 +140,15 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 		}
 	}
 
+	/**
+	 * Return the duration of the current session
+	 */
+	fun getDuration(context: Context): String {
+		val startDate = timeFrame?.startDate ?: Date()
+		val enDate = timeFrame?.endDate ?: Date()
+		return startDate.getDuration(context, enDate)
+	}
+
 	/**
 	 * Delete the object from realm
 	 * TODO: Cascade delete?
@@ -270,7 +281,6 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 	override val primaryKey: String get() = this.id
 
 
-
 	override fun getBottomSheetData(row: RowRepresentable): ArrayList<BottomSheetData> {
 
 		val data = ArrayList<BottomSheetData>()
@@ -282,8 +292,18 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 				data.add(BottomSheetData(100.0 * (cgBigBlind ?: 0.0)))
 				data.add(BottomSheetData(200.0 * (cgBigBlind ?: 0.0)))
 				data.add(BottomSheetData(buyin))
-				data.add(BottomSheetData("",inputType = InputType.TYPE_CLASS_NUMBER or InputType.TYPE_NUMBER_FLAG_DECIMAL))
-				data.add(BottomSheetData("",inputType = InputType.TYPE_CLASS_NUMBER or InputType.TYPE_NUMBER_FLAG_DECIMAL))
+				data.add(
+					BottomSheetData(
+						"",
+						inputType = InputType.TYPE_CLASS_NUMBER or InputType.TYPE_NUMBER_FLAG_DECIMAL
+					)
+				)
+				data.add(
+					BottomSheetData(
+						"",
+						inputType = InputType.TYPE_CLASS_NUMBER or InputType.TYPE_NUMBER_FLAG_DECIMAL
+					)
+				)
 			}
 			SessionRow.TIPS -> {
 				// Disable the buttons with value = 0, add current value & set the 2 edit texts
@@ -294,7 +314,9 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 				data.add(BottomSheetData("", inputType = InputType.TYPE_CLASS_NUMBER))
 				data.add(BottomSheetData("", inputType = InputType.TYPE_CLASS_NUMBER))
 			}
-			SessionRow.TABLE_SIZE -> {data.add(BottomSheetData(tableSize))}
+			SessionRow.TABLE_SIZE -> {
+				data.add(BottomSheetData(tableSize))
+			}
 			SessionRow.GAME -> {
 				// Add current game & games list
 				data.add(BottomSheetData(limit))
@@ -351,14 +373,16 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 				if (value == null) {
 					timeFrame = null
 				} else {
-					val timeFrameToUpdate = if (timeFrame != null) timeFrame as TimeFrame else realm.createObject(TimeFrame::class.java)
+					val timeFrameToUpdate =
+						if (timeFrame != null) timeFrame as TimeFrame else realm.createObject(TimeFrame::class.java)
 					timeFrameToUpdate.setDate(value, null)
 					timeFrame = timeFrameToUpdate
 				}
 			}
 			//TODO: Update
 			SessionRow.END_DATE -> if (value is Date?) {
-				val timeFrameToUpdate = if (timeFrame != null) timeFrame as TimeFrame else realm.createObject(TimeFrame::class.java)
+				val timeFrameToUpdate =
+					if (timeFrame != null) timeFrame as TimeFrame else realm.createObject(TimeFrame::class.java)
 				timeFrameToUpdate.setDate(null, value)
 				timeFrame = timeFrameToUpdate
 			}
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt
index 32481276..944d3a6d 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt
@@ -4,6 +4,7 @@ import android.os.Bundle
 import android.view.*
 import android.view.animation.OvershootInterpolator
 import android.widget.Toast
+import androidx.core.content.ContextCompat
 import androidx.interpolator.view.animation.FastOutSlowInInterpolator
 import io.realm.kotlin.where
 import kotlinx.android.synthetic.main.fragment_session.*
@@ -23,6 +24,7 @@ import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheet
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.SessionRow
 import net.pokeranalytics.android.ui.view.SmoothScrollLinearLayoutManager
+import net.pokeranalytics.android.util.px
 import net.pokeranalytics.android.util.toast
 import timber.log.Timber
 import java.util.*
@@ -156,6 +158,10 @@ class SessionFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Bott
 
 		when (currentSession.getState()) {
 			SessionState.PENDING -> {
+				state.text = "WAITING"
+				state.setTextColor(ContextCompat.getColor(requireContext(), R.color.white))
+				sessionData.animate().translationY(-(72f.px)).alpha(0f).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
+				recyclerView.animate().translationY(-(72f.px)).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
 				sessionMenu?.findItem(R.id.restart)?.isVisible = false
 				floatingActionButton.setImageResource(R.drawable.ic_outline_play)
 				bottomAppBar.menu.findItem(R.id.stop).isVisible = false
@@ -163,6 +169,10 @@ class SessionFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Bott
 					.setInterpolator(OvershootInterpolator()).start()
 			}
 			SessionState.STARTED -> {
+				state.text = "PLAYING"
+				state.setTextColor(ContextCompat.getColor(requireContext(), R.color.green))
+				sessionData.animate().translationY(0f).alpha(1f).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
+				recyclerView.animate().translationY(0f.px).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
 				sessionMenu?.findItem(R.id.restart)?.isVisible = true
 				floatingActionButton.setImageResource(R.drawable.ic_outline_pause)
 				bottomAppBar.menu.findItem(R.id.stop).isVisible = true
@@ -170,6 +180,10 @@ class SessionFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Bott
 					.setInterpolator(OvershootInterpolator()).start()
 			}
 			SessionState.PAUSED -> {
+				state.text = "BREAK"
+				state.setTextColor(ContextCompat.getColor(requireContext(), R.color.blue))
+				sessionData.animate().translationY(0f).alpha(1f).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
+				recyclerView.animate().translationY(0f.px).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
 				sessionMenu?.findItem(R.id.restart)?.isVisible = true
 				floatingActionButton.setImageResource(R.drawable.ic_outline_play)
 				bottomAppBar.menu.findItem(R.id.stop).isVisible = true
@@ -177,6 +191,10 @@ class SessionFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Bott
 					.setInterpolator(OvershootInterpolator()).start()
 			}
 			SessionState.FINISHED -> {
+				state.text = "FINISHED"
+				state.setTextColor(ContextCompat.getColor(requireContext(), R.color.white))
+				sessionData.animate().translationY(0f).alpha(1f).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
+				recyclerView.animate().translationY(0f.px).setInterpolator(FastOutSlowInInterpolator()).setDuration(500).start()
 				sessionMenu?.findItem(R.id.restart)?.isVisible = true
 				bottomAppBar.menu.findItem(R.id.stop).isVisible = false
 				floatingActionButton.animate().scaleX(0f).scaleY(0f).alpha(0f)
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt b/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt
index b1300f1c..0161b68c 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt
@@ -64,7 +64,7 @@ class SessionRowView : FrameLayout {
 
 		rowHistorySession.gameType.text = gameTitle
 
-		rowHistorySession.gameInfo.text = "Game info: duration, table size, ..."
+		rowHistorySession.gameInfo.text = session.getDuration(context)
 		rowHistorySession.gameResult.text = "$ 0"
 
 	}
diff --git a/app/src/main/java/net/pokeranalytics/android/util/Extensions.kt b/app/src/main/java/net/pokeranalytics/android/util/Extensions.kt
index 10fc83fe..1e8c5ab1 100644
--- a/app/src/main/java/net/pokeranalytics/android/util/Extensions.kt
+++ b/app/src/main/java/net/pokeranalytics/android/util/Extensions.kt
@@ -1,5 +1,6 @@
 package net.pokeranalytics.android.util
 
+import android.content.Context
 import android.content.res.Resources
 import android.widget.Toast
 import net.pokeranalytics.android.ui.activity.components.PokerAnalyticsActivity
@@ -9,6 +10,8 @@ import java.text.DecimalFormat
 import java.text.SimpleDateFormat
 import java.util.*
 
+
+
 // Sizes
 val Int.dp: Int
 	get() = (this / Resources.getSystem().displayMetrics.density).toInt()
@@ -55,6 +58,20 @@ fun Date.getDayNumber() : String {
 fun Date.getShortDayName() : String {
 	return SimpleDateFormat("EE", Locale.getDefault()).format(this)
 }
+// Return the duration between two dates
+fun Date.getDuration(context: Context, toDate: Date) : String {
+	val difference = (toDate.time - this.time).toInt()
+	val numOfDays = (difference / (1000 * 60 * 60 * 24))
+	val hours = (difference / (1000 * 60 * 60))
+	val minutes = (difference / (1000 * 60)) % 60
+	val seconds = (difference / 1000) % 60
+
+	val hoursStr = if (hours < 10) "0$hours" else "$hours"
+	val minutesStr = if (minutes < 10) "0$minutes" else "$minutes"
+	val secondsStr = if (seconds < 10) "0$seconds" else "$seconds"
+
+	return "$hoursStr:$minutesStr:$secondsStr"
+}
 
 
 // Toast
diff --git a/app/src/main/res/layout/fragment_session.xml b/app/src/main/res/layout/fragment_session.xml
index 95e68273..2ff2e6e1 100644
--- a/app/src/main/res/layout/fragment_session.xml
+++ b/app/src/main/res/layout/fragment_session.xml
@@ -15,16 +15,98 @@
 			android:layout_width="match_parent"
 			android:layout_height="wrap_content">
 
+			<androidx.constraintlayout.widget.ConstraintLayout
+				android:id="@+id/sessionData"
+				android:layout_width="match_parent"
+				android:layout_height="wrap_content"
+				android:background="@color/gray_darker"
+				android:elevation="4dp"
+				android:translationY="-72dp"
+				app:layout_constraintEnd_toEndOf="parent"
+				app:layout_constraintStart_toStartOf="parent"
+				app:layout_constraintTop_toTopOf="parent"
+				tools:translationY="0dp">
+
+				<androidx.appcompat.widget.AppCompatTextView
+					android:id="@+id/sinceTitle"
+					style="@style/PokerAnalyticsTheme.TextView.RowTitle"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:layout_marginStart="16dp"
+					android:layout_marginBottom="16dp"
+					android:text="Since"
+					android:textAllCaps="true"
+					android:textSize="12sp"
+					app:layout_constraintStart_toStartOf="parent"
+					app:layout_constraintTop_toTopOf="parent" />
+
+				<androidx.appcompat.widget.AppCompatTextView
+					android:id="@+id/sinceValue"
+					style="@style/PokerAnalyticsTheme.TextView.RowTitle"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:layout_marginStart="16dp"
+					android:layout_marginBottom="16dp"
+					android:text="2h33"
+					android:textSize="24sp"
+					app:fontFamily="@font/roboto_bold"
+					app:layout_constraintBottom_toBottomOf="parent"
+					app:layout_constraintStart_toStartOf="parent"
+					app:layout_constraintTop_toBottomOf="@+id/sinceTitle" />
+
+				<androidx.appcompat.widget.AppCompatTextView
+					android:id="@+id/resultTitle"
+					style="@style/PokerAnalyticsTheme.TextView.RowTitle"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:layout_marginStart="16dp"
+					android:layout_marginEnd="16dp"
+					android:layout_marginBottom="16dp"
+					android:text="Result"
+					android:textAllCaps="true"
+					android:textSize="12sp"
+					app:layout_constraintEnd_toEndOf="parent"
+					app:layout_constraintTop_toTopOf="parent" />
+
+				<androidx.appcompat.widget.AppCompatTextView
+					style="@style/PokerAnalyticsTheme.TextView.RowTitle"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:layout_marginEnd="16dp"
+					android:layout_marginBottom="16dp"
+					android:text="$ 325"
+					android:textColor="@color/green"
+					android:textSize="28sp"
+					app:layout_constraintBottom_toBottomOf="parent"
+					app:layout_constraintEnd_toEndOf="parent"
+					app:layout_constraintTop_toBottomOf="@+id/resultTitle" />
+
+				<androidx.appcompat.widget.AppCompatTextView
+					android:id="@+id/state"
+					style="@style/PokerAnalyticsTheme.TextView.RowTitle"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:text="PLAYING"
+					android:textColor="@color/colorAccent"
+					android:textSize="12sp"
+					app:layout_constraintEnd_toEndOf="parent"
+					app:layout_constraintStart_toStartOf="parent"
+					app:layout_constraintTop_toTopOf="parent" />
+
+			</androidx.constraintlayout.widget.ConstraintLayout>
+
 			<androidx.recyclerview.widget.RecyclerView
 				android:id="@+id/recyclerView"
 				android:layout_width="0dp"
 				android:layout_height="0dp"
-				android:paddingBottom="96dp"
 				android:clipToPadding="false"
+				android:paddingBottom="96dp"
+				android:translationY="-72dp"
 				app:layout_constraintBottom_toBottomOf="parent"
 				app:layout_constraintEnd_toEndOf="parent"
 				app:layout_constraintStart_toStartOf="parent"
-				app:layout_constraintTop_toTopOf="parent" />
+				app:layout_constraintTop_toBottomOf="@+id/sessionData"
+				tools:translationY="0dp" />
 
 		</androidx.constraintlayout.widget.ConstraintLayout>
 
@@ -55,6 +137,7 @@
 				app:titleTextColor="@color/white"
 				tools:title="Poker Analytics" />
 
+
 		</com.google.android.material.appbar.CollapsingToolbarLayout>
 
 	</com.google.android.material.appbar.AppBarLayout>
@@ -63,10 +146,10 @@
 		android:id="@+id/bottomAppBar"
 		android:layout_width="match_parent"
 		android:layout_height="wrap_content"
-		app:fabAlignmentMode="center"
+		android:layout_gravity="bottom"
 		android:background="@color/red"
 		android:theme="@style/PokerAnalyticsTheme.BottomAppBar"
-		android:layout_gravity="bottom" />
+		app:fabAlignmentMode="center" />
 
 	<com.google.android.material.floatingactionbutton.FloatingActionButton
 		android:id="@+id/floatingActionButton"
diff --git a/app/src/main/res/layout/row_session_view.xml b/app/src/main/res/layout/row_session_view.xml
index 9abbfcf9..9216e55c 100644
--- a/app/src/main/res/layout/row_session_view.xml
+++ b/app/src/main/res/layout/row_session_view.xml
@@ -9,9 +9,10 @@
 	<androidx.appcompat.widget.AppCompatTextView
 		android:id="@+id/dateDay"
 		style="@style/PokerAnalyticsTheme.TextView.SessionRow.Date"
-		android:layout_width="wrap_content"
+		android:layout_width="32dp"
 		android:layout_height="wrap_content"
 		android:layout_marginTop="8dp"
+		android:gravity="center"
 		android:textAllCaps="true"
 		app:layout_constraintBottom_toTopOf="@+id/dateNumber"
 		app:layout_constraintStart_toStartOf="@+id/guidelineStart"
@@ -22,9 +23,10 @@
 	<androidx.appcompat.widget.AppCompatTextView
 		android:id="@+id/dateNumber"
 		style="@style/PokerAnalyticsTheme.TextView.SessionRow.DateNumber"
-		android:layout_width="wrap_content"
+		android:layout_width="32dp"
 		android:layout_height="wrap_content"
 		android:layout_marginBottom="8dp"
+		android:gravity="center"
 		app:layout_constraintBottom_toBottomOf="parent"
 		app:layout_constraintEnd_toEndOf="@+id/dateDay"
 		app:layout_constraintHorizontal_bias="0.5"

From 33dc9437bca7e6558a85c8235cc130d79311f240 Mon Sep 17 00:00:00 2001
From: Razmig Sarkissian <razmig.sarkissian@staxriver.com>
Date: Fri, 22 Feb 2019 13:02:41 +0100
Subject: [PATCH 08/18] updated RowRepresentable

---
 .../pokeranalytics/android/model/LiveData.kt  |  2 +
 .../android/model/realm/Bankroll.kt           | 15 ++--
 .../android/model/realm/Game.kt               | 16 ++--
 .../android/model/realm/Location.kt           |  9 ++-
 .../android/model/realm/TournamentFeature.kt  |  9 ++-
 .../android/model/realm/TransactionType.kt    |  9 ++-
 .../components/RowRepresentableAdapter.kt     | 77 ++++++++++++-------
 .../android/ui/fragment/DataListFragment.kt   | 32 +++++---
 .../ui/fragment/EditableDataFragment.kt       |  9 +--
 .../android/ui/view/RowRepresentable.kt       |  8 +-
 .../android/ui/view/RowViewType.kt            | 35 ++++++++-
 11 files changed, 160 insertions(+), 61 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt b/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt
index 48212554..48a1065d 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt
@@ -16,6 +16,8 @@ interface ObjectSavable {
 	fun isValidForSave(): Boolean {
 		return true
 	}
+
+	fun uniqueIdentifier(): String
 }
 
 /**
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt
index 240ce6de..6ef2e2d3 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt
@@ -8,15 +8,12 @@ import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
 import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheetData
-import net.pokeranalytics.android.ui.view.BankrollRow
-import net.pokeranalytics.android.ui.view.RowEditable
-import net.pokeranalytics.android.ui.view.RowRepresentable
-import net.pokeranalytics.android.ui.view.SimpleRow
+import net.pokeranalytics.android.ui.view.*
 import java.util.*
 import kotlin.collections.ArrayList
 
 open class Bankroll(name: String = "") : RealmObject(), RowRepresentableDataSource, LiveDataDataSource,
-    RowEditable, ObjectSavable {
+    RowEditable, ObjectSavable, RowRepresentable {
 
     companion object {
         fun newInstance() : Bankroll {
@@ -42,6 +39,14 @@ open class Bankroll(name: String = "") : RealmObject(), RowRepresentableDataSour
 
     // @todo rate management
 
+    override fun getDisplayName(): String {
+        return this.name
+    }
+
+    override fun uniqueIdentifier(): String {
+        return this.id
+    }
+
     override val title: String get() = this.name
     override val primaryKey: String get() = this.id
 
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt
index 2171bfb4..186a9579 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt
@@ -4,16 +4,15 @@ import android.text.InputType
 import io.realm.RealmObject
 import io.realm.annotations.PrimaryKey
 import net.pokeranalytics.android.model.ObjectSavable
+import net.pokeranalytics.android.ui.adapter.components.DisplayableDelegate
 import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
 import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheetData
-import net.pokeranalytics.android.ui.view.GameRow
-import net.pokeranalytics.android.ui.view.RowEditable
-import net.pokeranalytics.android.ui.view.RowRepresentable
-import net.pokeranalytics.android.ui.view.SimpleRow
+import net.pokeranalytics.android.ui.view.*
 import java.util.*
 
-open class Game : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable {
+open class Game : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable,
+    RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -26,6 +25,13 @@ open class Game : RealmObject(), RowRepresentableDataSource, LiveDataDataSource,
 
     override val title: String get() = this.name
     override val primaryKey: String get() = this.id
+    override fun getDisplayName(): String {
+        return this.name
+    }
+
+    override fun uniqueIdentifier(): String {
+        return this.id
+    }
 
     override fun adapterRows(): ArrayList<RowRepresentable> {
         val rows = ArrayList<RowRepresentable>()
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt
index e5d24908..b6fa2e76 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt
@@ -13,7 +13,7 @@ import net.pokeranalytics.android.ui.view.SimpleRow
 import java.util.*
 
 
-open class Location : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable {
+open class Location : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable, RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -29,6 +29,13 @@ open class Location : RealmObject(), RowRepresentableDataSource, LiveDataDataSou
 
     override val title: String get() = this.name
     override val primaryKey: String get() = this.id
+    override fun getDisplayName(): String {
+        return this.name
+    }
+
+    override fun uniqueIdentifier(): String {
+        return this.id
+    }
 
     override fun adapterRows(): ArrayList<RowRepresentable> {
         val rows = ArrayList<RowRepresentable>()
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt
index b90c304b..edd3e5a8 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt
@@ -12,7 +12,7 @@ import net.pokeranalytics.android.ui.view.SimpleRow
 import net.pokeranalytics.android.ui.view.TournamentFeatureRow
 import java.util.*
 
-open class TournamentFeature : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable {
+open class TournamentFeature : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable, RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -22,6 +22,13 @@ open class TournamentFeature : RealmObject(), RowRepresentableDataSource, LiveDa
 
     override val title: String get() = this.name
     override val primaryKey: String get() = this.id
+    override fun getDisplayName(): String {
+        return this.name
+    }
+
+    override fun uniqueIdentifier(): String {
+        return this.id
+    }
 
     override fun adapterRows(): ArrayList<RowRepresentable> {
         val rows = ArrayList<RowRepresentable>()
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt
index f569440e..1f4dc94f 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt
@@ -13,7 +13,7 @@ import net.pokeranalytics.android.ui.view.TransactionTypeRow
 import java.util.*
 
 
-open class TransactionType : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable {
+open class TransactionType : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable, RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -32,6 +32,13 @@ open class TransactionType : RealmObject(), RowRepresentableDataSource, LiveData
 
     override val title: String get() = this.name
     override val primaryKey: String get() = this.id
+    override fun getDisplayName(): String {
+        return this.name
+    }
+
+    override fun uniqueIdentifier(): String {
+        return this.id
+    }
 
     override fun adapterRows(): ArrayList<RowRepresentable> {
         val rows = ArrayList<RowRepresentable>()
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/RowRepresentableAdapter.kt b/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/RowRepresentableAdapter.kt
index 9a1eb91c..b32ff748 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/RowRepresentableAdapter.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/RowRepresentableAdapter.kt
@@ -1,23 +1,49 @@
 package net.pokeranalytics.android.ui.adapter.components
 
+import android.inputmethodservice.Keyboard
 import android.view.View
 import android.view.ViewGroup
 import androidx.recyclerview.widget.DiffUtil
 import androidx.recyclerview.widget.RecyclerView
-import net.pokeranalytics.android.ui.view.BindableHolder
-import net.pokeranalytics.android.ui.view.RowRepresentable
-import net.pokeranalytics.android.ui.view.RowRepresentableDiffCallback
-import net.pokeranalytics.android.ui.view.RowViewType
+import net.pokeranalytics.android.ui.view.*
+
+interface RowRepresentableDataSource : DisplayableDataSource {
+	fun rowRepresentableForPosition(position:Int): RowRepresentable {
+		if (this.numberOfRows() > position) {
+			return this.adapterRows()[position]
+		} else {
+			throw IllegalStateException("Need to implement Data Source")
+		}
+	}
+
+	fun numberOfRows(): Int {
+		return this.adapterRows().size
+	}
+
+	fun viewTypeForPosition(position:Int): Int {
+		return this.rowRepresentableForPosition(position).viewType
+	}
+
+	fun indexForRow(row:RowRepresentable): Int {
+		return this.adapterRows().indexOf(row)
+	}
+}
+
+interface RowRepresentableDelegate : DisplayableDelegate {
+	fun onIndexSelected(position: Int) {}
+}
 
 /**
  * An interface used to provide RowRepresentableAdapter content and value in the form of rows
  */
-interface RowRepresentableDataSource {
+interface DisplayableDataSource {
 
 	/**
 	 * Returns a list of rows
 	 */
-	fun adapterRows(): ArrayList<RowRepresentable>
+	fun adapterRows(): ArrayList<RowRepresentable> {
+		return ArrayList()
+	}
 
 	/**
 	 * Returns a boolean for a specific row
@@ -48,39 +74,29 @@ interface RowRepresentableDataSource {
 	 * - switch (bool)
 	 * - static content
 	 * */
-
 }
 
 /**
  * A delegate used to propagate UI actions
  */
-interface RowRepresentableDelegate {
+interface DisplayableDelegate {
 	fun onRowSelected(row: RowRepresentable) {}
 	fun onActionSelected(row: RowRepresentable) {}
 }
 
 /**
  * An adapter capable of displaying a list of RowRepresentables
- * @param rowRepresentableDataSource the datasource providing rows
- * @param rowRepresentableDelegate the delegate, notified of UI actions
+ * @param dataSource the datasource providing rows
+ * @param delegate the delegate, notified of UI actions
  */
 class RowRepresentableAdapter(
-	var rowRepresentableDataSource: RowRepresentableDataSource,
-	var rowRepresentableDelegate: RowRepresentableDelegate? = null
+	var dataSource: RowRepresentableDataSource,
+	var delegate: RowRepresentableDelegate? = null
 ) :
 	RecyclerView.Adapter<RecyclerView.ViewHolder>() {
 
-	/**
-	 * The list of rows to display
-	 */
-	private var rows: ArrayList<RowRepresentable> = ArrayList()
-
-	init {
-		this.rows = rowRepresentableDataSource.adapterRows()
-	}
-
 	override fun getItemViewType(position: Int): Int {
-		return this.rows[position].viewType
+		return this.dataSource.viewTypeForPosition(position)
 	}
 
 	override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RecyclerView.ViewHolder {
@@ -89,28 +105,29 @@ class RowRepresentableAdapter(
 	}
 
 	override fun getItemCount(): Int {
-		return this.rows.size
+		return this.dataSource.numberOfRows()
 	}
 
 	override fun onBindViewHolder(holder: RecyclerView.ViewHolder, position: Int) {
-		val dynamicRow = this.rows[position]
+		val dynamicRow = this.dataSource.rowRepresentableForPosition(position)
 
 		val listener = View.OnClickListener {
-			rowRepresentableDelegate?.onRowSelected(dynamicRow)
+			delegate?.onRowSelected(dynamicRow)
+			delegate?.onIndexSelected(position)
 		}
 
 		val actionListener = View.OnClickListener {
-			rowRepresentableDelegate?.onActionSelected(dynamicRow)
+			delegate?.onActionSelected(dynamicRow)
 		}
 
-		(holder as BindableHolder).bind(dynamicRow, this.rowRepresentableDataSource, listener, actionListener)
+		(holder as BindableHolder).bind(dynamicRow, this.dataSource, listener, actionListener)
 	}
 
 	/**
 	 * Refresh the row in the adapter
 	 */
 	fun refreshRow(row: RowRepresentable) {
-		val index = rows.indexOf(row)
+		val index = this.dataSource.indexForRow(row)
 		if (index >= 0) {
 			notifyItemChanged(index)
 		}
@@ -120,9 +137,11 @@ class RowRepresentableAdapter(
 	 * Update UI
 	 */
 	fun updateRows(newRows: ArrayList<RowRepresentable>) {
-		val diffResult = DiffUtil.calculateDiff(RowRepresentableDiffCallback(newRows, rows, rowRepresentableDataSource))
+		/*
+		val diffResult = DiffUtil.calculateDiff(RowRepresentableDiffCallback(newRows, rows, dataSource))
 		this.rows = newRows
 		diffResult.dispatchUpdatesTo(this)
+		*/
 	}
 
 }
\ No newline at end of file
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/DataListFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/DataListFragment.kt
index a5e69cc5..5b7798c4 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/DataListFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/DataListFragment.kt
@@ -5,17 +5,21 @@ import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
 import androidx.recyclerview.widget.LinearLayoutManager
+import io.realm.ObjectChangeSet
 import io.realm.Realm
 import io.realm.RealmResults
 import kotlinx.android.synthetic.main.fragment_data_list.*
 import net.pokeranalytics.android.R
+import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.ui.activity.EditableDataActivity
 import net.pokeranalytics.android.ui.adapter.components.*
 import net.pokeranalytics.android.ui.fragment.components.PokerAnalyticsFragment
+import net.pokeranalytics.android.ui.view.RowRepresentable
+import net.pokeranalytics.android.ui.view.RowViewType
 import net.pokeranalytics.android.ui.view.SettingRow
 import timber.log.Timber
 
-class DataListFragment : PokerAnalyticsFragment(), LiveDataDelegate {
+class DataListFragment : PokerAnalyticsFragment(), RowRepresentableDataSource, RowRepresentableDelegate{
 
 	private lateinit var dataType: SettingRow
 
@@ -31,20 +35,28 @@ class DataListFragment : PokerAnalyticsFragment(), LiveDataDelegate {
 		initUI()
 	}
 
-	override fun data(position: Int): LiveDataDataSource {
-        return (items[position] as LiveDataDataSource)
+	override fun rowRepresentableForPosition(position: Int): RowRepresentable {
+		return this.items[position] as RowRepresentable
 	}
 
-	override fun onRowSelected(position: Int) {
+	override fun numberOfRows(): Int {
+		return this.items.size
+	}
+
+	override fun viewTypeForPosition(position: Int): Int {
+		return RowViewType.DATA.ordinal
+	}
+
+	override fun indexForRow(row: RowRepresentable): Int {
+		return this.items.indexOf(row)
+	}
+
+	override fun onIndexSelected(position: Int) {
 		this.dataType.relatedResultsRepresentable?.let {
-			EditableDataActivity.newInstance(requireContext(), it.ordinal, this.data(position).primaryKey)
+			EditableDataActivity.newInstance(requireContext(), it.ordinal, (this.items[position] as ObjectSavable).uniqueIdentifier())
 		}
 	}
 
-    override fun size(): Int {
-        return items.size
-    }
-
     private fun initData() {
     }
 
@@ -54,7 +66,7 @@ class DataListFragment : PokerAnalyticsFragment(), LiveDataDelegate {
 	private fun initUI() {
 
 		val viewManager = LinearLayoutManager(requireContext())
-		val dataListAdapter = LiveDataAdapter(this)
+		val dataListAdapter = RowRepresentableAdapter(this, this)
 
 		recyclerView.apply {
 			setHasFixedSize(true)
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
index 27c2dcc4..bde542bb 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
@@ -12,10 +12,7 @@ import net.pokeranalytics.android.R
 import net.pokeranalytics.android.model.LiveData
 import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.ui.activity.components.PokerAnalyticsActivity
-import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
-import net.pokeranalytics.android.ui.adapter.components.RowRepresentableAdapter
-import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
-import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDelegate
+import net.pokeranalytics.android.ui.adapter.components.*
 import net.pokeranalytics.android.ui.fragment.components.PokerAnalyticsFragment
 import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheetDelegate
 import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheetFragment
@@ -66,16 +63,16 @@ class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate,
 
 	override fun clickOnClear(row: RowRepresentable) {
 		Toast.makeText(requireContext(), "Clear: $row", Toast.LENGTH_SHORT).show()
-		(this.item as RowEditable).updateValue(null, row)
 		this.getRealm().executeTransaction {
+			(this.item as RowEditable).updateValue(null, row)
 			it.copyToRealmOrUpdate(this.item)
 		}
 		rowRepresentableAdapter.refreshRow(row)
 	}
 
 	override fun setValue(value: Any?, row: RowRepresentable) {
-		(this.item as RowEditable).updateValue(value, row)
 		this.getRealm().executeTransaction {
+			(this.item as RowEditable).updateValue(value, row)
 			it.copyToRealmOrUpdate(this.item)
 		}
 		rowRepresentableAdapter.refreshRow(row)
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/view/RowRepresentable.kt b/app/src/main/java/net/pokeranalytics/android/ui/view/RowRepresentable.kt
index 6174bc78..f3770e73 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/view/RowRepresentable.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/view/RowRepresentable.kt
@@ -29,11 +29,17 @@ interface Localizable {
 	}
 }
 
+
+interface RowRepresentable : Displayable {
+	fun getDisplayName(): String {
+		return "UNKNOWN NAME"
+	}
+}
 /**
  * An interface used so that enums values can be represented visually
  *  as rows in RecyclerViews
  */
-interface RowRepresentable : Localizable {
+interface Displayable: Localizable {
 	/**
 	 * The type of view associated with the row
 	 */
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt b/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
index 73bf2d22..3dab41d9 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
@@ -3,6 +3,8 @@ package net.pokeranalytics.android.ui.view
 import android.view.LayoutInflater
 import android.view.View
 import android.view.ViewGroup
+import androidx.appcompat.widget.AppCompatTextView
+import androidx.constraintlayout.widget.ConstraintLayout
 import androidx.recyclerview.widget.RecyclerView
 import kotlinx.android.synthetic.main.row_header_title_value.view.*
 import kotlinx.android.synthetic.main.row_title.view.*
@@ -26,7 +28,9 @@ enum class RowViewType {
 	EDIT_TEXT,
 	TITLE,
 	TITLE_VALUE,
-	TITLE_VALUE_ACTION;
+	TITLE_VALUE_ACTION,
+	DATA,
+	BOTTOM_SHEET_DATA;
 
 	inner class FakeViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView),
 		BindableHolder {
@@ -34,6 +38,20 @@ enum class RowViewType {
 		}
 	}
 
+	inner class DataViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView), BindableHolder {
+		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
+			itemView.findViewById<AppCompatTextView>(R.id.rowTitle_title).text = row.getDisplayName()
+			itemView.findViewById<ConstraintLayout>(R.id.rowTitle_container).setOnClickListener(listener)
+		}
+	}
+
+	inner class BottomSheetDataViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView), BindableHolder {
+		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
+			itemView.findViewById<AppCompatTextView>(R.id.title).text = row.getDisplayName()
+			itemView.findViewById<ConstraintLayout>(R.id.container).setOnClickListener(listener)
+		}
+	}
+
 	inner class TitleViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView),
 		BindableHolder {
 		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
@@ -115,7 +133,20 @@ enum class RowViewType {
 					false
 				)
 			)
-
+			DATA -> DataViewHolder(
+				LayoutInflater.from(parent.context).inflate(
+					R.layout.row_title,
+					parent,
+					false
+				)
+			)
+			BOTTOM_SHEET_DATA -> BottomSheetDataViewHolder(
+				LayoutInflater.from(parent.context).inflate(
+					R.layout.row_bottom_sheet_title,
+					parent,
+					false
+				)
+			)
 			else -> FakeViewHolder(parent)
 		}
 	}

From 4242e4cfb76359dc37a4b71bdd22bb87efc037ba Mon Sep 17 00:00:00 2001
From: Razmig Sarkissian <razmig.sarkissian@staxriver.com>
Date: Fri, 22 Feb 2019 13:27:21 +0100
Subject: [PATCH 09/18] remove usage of LiveDataAdapter and related interface

---
 .../pokeranalytics/android/model/LiveData.kt  |  4 +-
 .../android/model/realm/Bankroll.kt           |  5 +-
 .../android/model/realm/Game.kt               |  4 +-
 .../android/model/realm/Location.kt           |  4 +-
 .../android/model/realm/Session.kt            | 15 ++----
 .../android/model/realm/TournamentFeature.kt  |  4 +-
 .../android/model/realm/TransactionType.kt    |  4 +-
 .../ui/adapter/components/LiveDataAdapter.kt  |  1 -
 .../ui/fragment/EditableDataFragment.kt       |  2 +-
 .../bottomsheet/BottomSheetListFragment.kt    | 46 ++++++++++-------
 .../BottomSheetListGameFragment.kt            | 49 ++++++++++++-------
 11 files changed, 72 insertions(+), 66 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt b/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt
index 48a1065d..1d79a807 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/LiveData.kt
@@ -66,8 +66,8 @@ enum class LiveData : Localizable {
 			}
 		}
 
-    fun deleteData(realm:Realm, data:LiveDataDataSource) {
-        realm.where(this.relatedEntity).equalTo("id", data.primaryKey).findAll().deleteAllFromRealm()
+    fun deleteData(realm:Realm, data:ObjectSavable) {
+        realm.where(this.relatedEntity).equalTo("id", data.uniqueIdentifier()).findAll().deleteAllFromRealm()
     }
 
     fun updateOrCreate(realm:Realm, primaryKey:String?): RealmObject {
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt
index 6ef2e2d3..be0cc9d9 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Bankroll.kt
@@ -12,7 +12,7 @@ import net.pokeranalytics.android.ui.view.*
 import java.util.*
 import kotlin.collections.ArrayList
 
-open class Bankroll(name: String = "") : RealmObject(), RowRepresentableDataSource, LiveDataDataSource,
+open class Bankroll(name: String = "") : RealmObject(), RowRepresentableDataSource,
     RowEditable, ObjectSavable, RowRepresentable {
 
     companion object {
@@ -47,9 +47,6 @@ open class Bankroll(name: String = "") : RealmObject(), RowRepresentableDataSour
         return this.id
     }
 
-    override val title: String get() = this.name
-    override val primaryKey: String get() = this.id
-
     override fun adapterRows(): ArrayList<RowRepresentable> {
         val rows = ArrayList<RowRepresentable>()
         rows.add(SimpleRow.NAME)
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt
index 186a9579..2ff50761 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Game.kt
@@ -11,7 +11,7 @@ import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheet
 import net.pokeranalytics.android.ui.view.*
 import java.util.*
 
-open class Game : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable,
+open class Game : RealmObject(), RowRepresentableDataSource, RowEditable, ObjectSavable,
     RowRepresentable {
 
     @PrimaryKey
@@ -23,8 +23,6 @@ open class Game : RealmObject(), RowRepresentableDataSource, LiveDataDataSource,
     // A shorter name for the game
     var shortName: String? = null
 
-    override val title: String get() = this.name
-    override val primaryKey: String get() = this.id
     override fun getDisplayName(): String {
         return this.name
     }
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt
index b6fa2e76..733d6de6 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Location.kt
@@ -13,7 +13,7 @@ import net.pokeranalytics.android.ui.view.SimpleRow
 import java.util.*
 
 
-open class Location : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable, RowRepresentable {
+open class Location : RealmObject(), RowRepresentableDataSource, RowEditable, ObjectSavable, RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -27,8 +27,6 @@ open class Location : RealmObject(), RowRepresentableDataSource, LiveDataDataSou
     // the latitude of the location
     var latitude: Double? = null
 
-    override val title: String get() = this.name
-    override val primaryKey: String get() = this.id
     override fun getDisplayName(): String {
         return this.name
     }
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index e023d1e5..9da0cf4b 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -23,7 +23,7 @@ import net.pokeranalytics.android.util.toCurrency
 import java.util.*
 import kotlin.collections.ArrayList
 
-open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource, LiveDataDataSource,
+open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource,
 	RowEditable {
 
 	@PrimaryKey
@@ -194,7 +194,7 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 			gameTitle += limit
 		}
 		if (game != null) {
-			gameTitle += game?.title
+			gameTitle += game?.name
 		}
 		return gameTitle
 	}
@@ -256,9 +256,9 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 		return when (row) {
 			SessionRow.BUY_IN -> buyin.toCurrency()
 			SessionRow.BLINDS -> if (cgSmallBlind != null && cgBigBlind != null) "$cgSmallBlind / $cgBigBlind" else "--"
-			SessionRow.GAME -> game?.title ?: "--"
-			SessionRow.LOCATION -> location?.title ?: "--"
-			SessionRow.BANKROLL -> bankroll?.title ?: "--"
+			SessionRow.GAME -> game?.name ?: "--"
+			SessionRow.LOCATION -> location?.name ?: "--"
+			SessionRow.BANKROLL -> bankroll?.name ?: "--"
 			SessionRow.TABLE_SIZE -> tableSize?.toString() ?: "--"
 			SessionRow.START_DATE -> if (timeFrame != null) timeFrame?.startDate?.short() ?: "--" else "--"
 			SessionRow.END_DATE -> if (timeFrame != null) timeFrame?.endDate?.short() ?: "--" else "--"
@@ -276,11 +276,6 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 		}
 	}
 
-
-	override var title: String = "Change that: $creationDate"
-	override val primaryKey: String get() = this.id
-
-
 	override fun getBottomSheetData(row: RowRepresentable): ArrayList<BottomSheetData> {
 
 		val data = ArrayList<BottomSheetData>()
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt
index edd3e5a8..a765249b 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/TournamentFeature.kt
@@ -12,7 +12,7 @@ import net.pokeranalytics.android.ui.view.SimpleRow
 import net.pokeranalytics.android.ui.view.TournamentFeatureRow
 import java.util.*
 
-open class TournamentFeature : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable, RowRepresentable {
+open class TournamentFeature : RealmObject(), RowRepresentableDataSource, RowEditable, ObjectSavable, RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -20,8 +20,6 @@ open class TournamentFeature : RealmObject(), RowRepresentableDataSource, LiveDa
     // The name of the feature
     var name: String = ""
 
-    override val title: String get() = this.name
-    override val primaryKey: String get() = this.id
     override fun getDisplayName(): String {
         return this.name
     }
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt
index 1f4dc94f..8e526eab 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/TransactionType.kt
@@ -13,7 +13,7 @@ import net.pokeranalytics.android.ui.view.TransactionTypeRow
 import java.util.*
 
 
-open class TransactionType : RealmObject(), RowRepresentableDataSource, LiveDataDataSource, RowEditable, ObjectSavable, RowRepresentable {
+open class TransactionType : RealmObject(), RowRepresentableDataSource, RowEditable, ObjectSavable, RowRepresentable {
 
     @PrimaryKey
     var id = UUID.randomUUID().toString()
@@ -30,8 +30,6 @@ open class TransactionType : RealmObject(), RowRepresentableDataSource, LiveData
     // The predefined kind, if necessary, like: Withdrawal, deposit, or tips
     var kind: Int? = null
 
-    override val title: String get() = this.name
-    override val primaryKey: String get() = this.id
     override fun getDisplayName(): String {
         return this.name
     }
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt b/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt
index 5cb5cbfe..f2e3b683 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/adapter/components/LiveDataAdapter.kt
@@ -16,7 +16,6 @@ enum class LiveDataViewType {
 
 interface LiveDataDataSource {
     val title: String
-    val primaryKey: String
 }
 
 interface LiveDataDelegate {
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
index bde542bb..9b9de841 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/EditableDataFragment.kt
@@ -138,7 +138,7 @@ class EditableDataFragment : PokerAnalyticsFragment(), RowRepresentableDelegate,
 					Toast.makeText(requireContext(), "isManaged", Toast.LENGTH_SHORT).show()
 					Timber.d("is managed")
 					this.getRealm().executeTransaction {
-						this.liveDataType.deleteData(it, (this.item as LiveDataDataSource))
+						this.liveDataType.deleteData(it, (this.item as ObjectSavable))
 					}
 				} else {
 					Toast.makeText(requireContext(), "isNotManaged", Toast.LENGTH_SHORT).show()
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt
index 2131c53e..abca4c07 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListFragment.kt
@@ -8,16 +8,14 @@ import io.realm.RealmResults
 import kotlinx.android.synthetic.main.bottom_sheet_list.*
 import kotlinx.android.synthetic.main.fragment_bottom_sheet.view.*
 import net.pokeranalytics.android.R
-import net.pokeranalytics.android.model.realm.Game
-import net.pokeranalytics.android.ui.adapter.components.LiveDataAdapter
-import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
-import net.pokeranalytics.android.ui.adapter.components.LiveDataDelegate
-import net.pokeranalytics.android.ui.adapter.components.LiveDataViewType
+import net.pokeranalytics.android.ui.adapter.components.*
+import net.pokeranalytics.android.ui.view.RowRepresentable
+import net.pokeranalytics.android.ui.view.RowViewType
 
-class BottomSheetListFragment : BottomSheetFragment(), LiveDataDelegate {
+class BottomSheetListFragment : BottomSheetFragment(), RowRepresentableDataSource, RowRepresentableDelegate {
 
 	private var realmData: RealmResults<*>? = null
-	private lateinit var dataAdapter: LiveDataAdapter
+	private lateinit var dataAdapter: RowRepresentableAdapter
 
 	override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
 		super.onViewCreated(view, savedInstanceState)
@@ -30,15 +28,32 @@ class BottomSheetListFragment : BottomSheetFragment(), LiveDataDelegate {
 		dataAdapter.notifyDataSetChanged()
 	}
 
-	override fun data(position: Int): LiveDataDataSource {
+	override fun rowRepresentableForPosition(position: Int): RowRepresentable {
 		realmData?.let {
-			return it[position] as LiveDataDataSource
+			return it[position] as RowRepresentable
 		}
-		//TODO: Change that
-		return Game()
+		return super.rowRepresentableForPosition(position)
 	}
 
-	override fun onRowSelected(position: Int) {
+	override fun numberOfRows(): Int {
+		realmData?.let {
+			return it.size
+		}
+		return super.numberOfRows()
+	}
+
+	override fun viewTypeForPosition(position: Int): Int {
+		return RowViewType.BOTTOM_SHEET_DATA.ordinal
+	}
+
+	override fun indexForRow(row: RowRepresentable): Int {
+		realmData?.let {
+			return it.indexOf(row)
+		}
+		return super.indexForRow(row)
+	}
+
+	override fun onIndexSelected(position: Int) {
 		realmData?.let {
 			val selectedData = it[position]
 			selectedData?.let {data ->
@@ -46,10 +61,7 @@ class BottomSheetListFragment : BottomSheetFragment(), LiveDataDelegate {
 				dismiss()
 			}
 		}
-	}
-
-	override fun size(): Int {
-		return realmData?.size ?: 0
+		super.onIndexSelected(position)
 	}
 
 	/**
@@ -70,7 +82,7 @@ class BottomSheetListFragment : BottomSheetFragment(), LiveDataDelegate {
 		LayoutInflater.from(requireContext()).inflate(R.layout.bottom_sheet_list, view?.bottomSheetContainer, true)
 
 		val viewManager = LinearLayoutManager(requireContext())
-		dataAdapter = LiveDataAdapter(this, LiveDataViewType.BOTTOM_SHEET_DATA)
+		dataAdapter = RowRepresentableAdapter(this, this)
 
 		reyclerView.apply {
 			setHasFixedSize(true)
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
index f97672fd..e566a340 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
@@ -10,17 +10,16 @@ import kotlinx.android.synthetic.main.fragment_bottom_sheet.view.*
 import net.pokeranalytics.android.R
 import net.pokeranalytics.android.model.realm.Game
 import net.pokeranalytics.android.ui.adapter.LimitTypesAdapter
-import net.pokeranalytics.android.ui.adapter.components.LiveDataAdapter
-import net.pokeranalytics.android.ui.adapter.components.LiveDataDataSource
-import net.pokeranalytics.android.ui.adapter.components.LiveDataDelegate
-import net.pokeranalytics.android.ui.adapter.components.LiveDataViewType
+import net.pokeranalytics.android.ui.adapter.components.*
+import net.pokeranalytics.android.ui.view.RowRepresentable
+import net.pokeranalytics.android.ui.view.RowViewType
 import timber.log.Timber
 
 
-class BottomSheetListGameFragment : BottomSheetFragment(), LiveDataDelegate {
+class BottomSheetListGameFragment : BottomSheetFragment(), RowRepresentableDataSource, RowRepresentableDelegate {
 
 	private var realmData: RealmResults<*>? = null
-	private lateinit var dataAdapter: LiveDataAdapter
+	private lateinit var dataAdapter: RowRepresentableAdapter
 
 	override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
 		super.onViewCreated(view, savedInstanceState)
@@ -33,15 +32,33 @@ class BottomSheetListGameFragment : BottomSheetFragment(), LiveDataDelegate {
 		dataAdapter.notifyDataSetChanged()
 	}
 
-	override fun data(position: Int): LiveDataDataSource {
+
+	override fun rowRepresentableForPosition(position: Int): RowRepresentable {
+		realmData?.let {
+			return it[position] as RowRepresentable
+		}
+		return super.rowRepresentableForPosition(position)
+	}
+
+	override fun numberOfRows(): Int {
 		realmData?.let {
-			return it[position] as LiveDataDataSource
+			return it.size
 		}
-		//TODO: Change that
-		return Game()
+		return super.numberOfRows()
+	}
+
+	override fun viewTypeForPosition(position: Int): Int {
+		return RowViewType.BOTTOM_SHEET_DATA.ordinal
 	}
 
-	override fun onRowSelected(position: Int) {
+	override fun indexForRow(row: RowRepresentable): Int {
+		realmData?.let {
+			return it.indexOf(row)
+		}
+		return super.indexForRow(row)
+	}
+
+	override fun onIndexSelected(position: Int) {
 		realmData?.let {
 			val selectedData = it[position]
 			selectedData?.let {data ->
@@ -49,13 +66,7 @@ class BottomSheetListGameFragment : BottomSheetFragment(), LiveDataDelegate {
 				dismiss()
 			}
 		}
-	}
-
-	override fun size(): Int {
-
-		Timber.d("Games: ${realmData?.size}")
-
-		return realmData?.size ?: 0
+		super.onIndexSelected(position)
 	}
 
 	/**
@@ -91,7 +102,7 @@ class BottomSheetListGameFragment : BottomSheetFragment(), LiveDataDelegate {
 
 
 		val viewManager2 = LinearLayoutManager(requireContext())
-		dataAdapter = LiveDataAdapter(this, LiveDataViewType.BOTTOM_SHEET_DATA)
+		dataAdapter = RowRepresentableAdapter(this, this)
 
 		recyclerView2.apply {
 			setHasFixedSize(true)

From 7ab7b11c2ae5cea39702c03f3698b96ae5e384c1 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 13:37:47 +0100
Subject: [PATCH 10/18] Improve Session

---
 .../android/model/realm/Session.kt            | 28 +++++++++++++++++--
 .../android/ui/adapter/LimitTypesAdapter.kt   |  1 +
 app/src/main/res/layout/fragment_session.xml  |  1 +
 3 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index e023d1e5..49cc1552 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -18,6 +18,7 @@ import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.SessionRow
 import net.pokeranalytics.android.util.data.sessionDao
 import net.pokeranalytics.android.util.getDuration
+import net.pokeranalytics.android.util.round
 import net.pokeranalytics.android.util.short
 import net.pokeranalytics.android.util.toCurrency
 import java.util.*
@@ -149,6 +150,13 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 		return startDate.getDuration(context, enDate)
 	}
 
+	/**
+	 * Return the formatted blinds
+	 */
+	fun getBlinds() : String {
+		return if (cgSmallBlind == null) "--" else "$${cgSmallBlind?.round()}/${cgBigBlind?.round()}"
+	}
+
 	/**
 	 * Delete the object from realm
 	 * TODO: Cascade delete?
@@ -255,7 +263,8 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 	override fun stringForRow(row: RowRepresentable): String {
 		return when (row) {
 			SessionRow.BUY_IN -> buyin.toCurrency()
-			SessionRow.BLINDS -> if (cgSmallBlind != null && cgBigBlind != null) "$cgSmallBlind / $cgBigBlind" else "--"
+			SessionRow.CASHED_OUT -> result?.cashout?.toCurrency() ?: "--"
+			SessionRow.BLINDS -> getBlinds()
 			SessionRow.GAME -> game?.title ?: "--"
 			SessionRow.LOCATION -> location?.title ?: "--"
 			SessionRow.BANKROLL -> bankroll?.title ?: "--"
@@ -305,6 +314,9 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 					)
 				)
 			}
+			SessionRow.CASHED_OUT -> {
+				data.add(BottomSheetData(result?.cashout, inputType = InputType.TYPE_CLASS_NUMBER))
+			}
 			SessionRow.TIPS -> {
 				// Disable the buttons with value = 0, add current value & set the 2 edit texts
 				// TODO: manage tips
@@ -345,10 +357,14 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 	override fun updateValue(value: Any?, row: RowRepresentable) {
 		realm.beginTransaction()
 		when (row) {
-
 			SessionRow.BUY_IN -> {
 				val localResult = if (result != null) result as Result else realm.createObject(Result::class.java)
-				localResult.buyin = value as Double
+				localResult.buyin = value as Double?
+				result = localResult
+			}
+			SessionRow.CASHED_OUT -> {
+				val localResult = if (result != null) result as Result else realm.createObject(Result::class.java)
+				localResult.cashout = if (value == null) null else (value as String).toDouble()
 				result = localResult
 			}
 			SessionRow.TABLE_SIZE -> tableSize = value as Int?
@@ -367,6 +383,12 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 				} catch (e: Exception) {
 					null
 				}
+
+				cgBigBlind?.let {
+					if (cgSmallBlind == null || cgSmallBlind == 0.0) {
+						cgSmallBlind = it / 2.0
+					}
+				}
 			}
 			//TODO: Update
 			SessionRow.START_DATE -> if (value is Date?) {
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/adapter/LimitTypesAdapter.kt b/app/src/main/java/net/pokeranalytics/android/ui/adapter/LimitTypesAdapter.kt
index 81e5af9a..52832cbd 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/adapter/LimitTypesAdapter.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/adapter/LimitTypesAdapter.kt
@@ -21,6 +21,7 @@ class LimitTypesAdapter(private var tableSizes: ArrayList<String>) : RecyclerVie
 			itemView.title.text = tableSize
 			itemView.container.setOnClickListener {
 				onClickOnItem?.invoke(adapterPosition)
+
 			}
 		}
 	}
diff --git a/app/src/main/res/layout/fragment_session.xml b/app/src/main/res/layout/fragment_session.xml
index 2ff2e6e1..2d74bac0 100644
--- a/app/src/main/res/layout/fragment_session.xml
+++ b/app/src/main/res/layout/fragment_session.xml
@@ -69,6 +69,7 @@
 					app:layout_constraintTop_toTopOf="parent" />
 
 				<androidx.appcompat.widget.AppCompatTextView
+					android:id="@+id/appCompatTextView"
 					style="@style/PokerAnalyticsTheme.TextView.RowTitle"
 					android:layout_width="wrap_content"
 					android:layout_height="wrap_content"

From 3c31ada7bfce2683287c6da6c8e743047861c7b3 Mon Sep 17 00:00:00 2001
From: Razmig Sarkissian <razmig.sarkissian@staxriver.com>
Date: Fri, 22 Feb 2019 13:41:48 +0100
Subject: [PATCH 11/18] remove usage of TableSizeGridAdapter

---
 .../BottomSheetTableSizeGridFragment.kt       | 121 +++++++++++++++++-
 .../android/ui/view/RowViewType.kt            |  18 ++-
 2 files changed, 136 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetTableSizeGridFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetTableSizeGridFragment.kt
index 868d17e7..38159ee5 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetTableSizeGridFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetTableSizeGridFragment.kt
@@ -1,16 +1,131 @@
 package net.pokeranalytics.android.ui.fragment.components.bottomsheet
 
+import android.content.Context
 import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
 import androidx.recyclerview.widget.GridLayoutManager
 import kotlinx.android.synthetic.main.bottom_sheet_grid.*
 import kotlinx.android.synthetic.main.fragment_bottom_sheet.view.*
-import net.pokeranalytics.android.ui.adapter.TableSizeGridAdapter
+import net.pokeranalytics.android.R
+import net.pokeranalytics.android.ui.adapter.components.*
 import net.pokeranalytics.android.ui.view.GridSpacingItemDecoration
+import net.pokeranalytics.android.ui.view.RowRepresentable
+import net.pokeranalytics.android.ui.view.RowViewType
 import net.pokeranalytics.android.util.px
 
+class TableSize(var numberOfPlayer:Int): RowRepresentable {
+	companion object {
+		val all = Array(8, init =
+		{ index -> TableSize(index+2)})
+	}
+
+	override val resId: Int?
+		get() {
+			return if (this.numberOfPlayer == 2) {
+				R.string.heads_up
+			} else {
+				R.string.max
+			}
+		}
+
+	override fun localizedTitle(context: Context): String {
+		this.resId?.let {
+			return if (this.numberOfPlayer == 2) {
+				context.getString(it)
+			} else {
+				"$this.numberOfPlayer$context.getString(it)"
+			}
+		}
+		return super.localizedTitle(context)
+	}
+
+	override val viewType: Int
+		get() = RowViewType.TITLE_GRID.ordinal
+}
+
+class BottomSheetTableSizeGridFragment : BottomSheetFragment(), RowRepresentableDataSource, RowRepresentableDelegate {
+
+	private lateinit var dataAdapter: RowRepresentableAdapter
+	private var defaultSize: Int? = null
+
+	override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
+		super.onViewCreated(view, savedInstanceState)
+		initData()
+		initUI()
+	}
+
+	override fun onResume() {
+		super.onResume()
+		dataAdapter.notifyDataSetChanged()
+	}
+
+	override fun getValue(): Any? {
+		return defaultSize
+	}
+
+	/**
+	 * Init data
+	 */
+	private fun initData() {
+
+		val bottomSheetData = getData()
+		if (bottomSheetData.isNotEmpty() && bottomSheetData.first().defaultValue != null) {
+			defaultSize = bottomSheetData.first().defaultValue as Int?
+		}
+	}
 
+	/**
+	 * Init UI
+	 */
+	private fun initUI() {
+
+		setAddButtonVisible(false)
+
+		LayoutInflater.from(requireContext())
+			.inflate(net.pokeranalytics.android.R.layout.bottom_sheet_grid, view?.bottomSheetContainer, true)
+
+		val viewManager = GridLayoutManager(requireContext(), 3)
+		dataAdapter = RowRepresentableAdapter(this, this)
+
+		val spanCount = 3
+		val spacing = 2.px
+		val includeEdge = false
+
+		reyclerView.apply {
+			setHasFixedSize(true)
+			layoutManager = viewManager
+			adapter = dataAdapter
+			addItemDecoration(GridSpacingItemDecoration(spanCount, spacing, includeEdge))
+		}
+	}
+
+	override fun rowRepresentableForPosition(position: Int): RowRepresentable {
+		return TableSize.all[position]
+	}
+
+	override fun indexForRow(row: RowRepresentable): Int {
+		return TableSize.all.indexOf(row)
+	}
+
+	override fun numberOfRows(): Int {
+		return TableSize.all.size
+	}
+
+	override fun onRowSelected(row: RowRepresentable) {
+		bottomSheetDelegate.setValue((this.row as TableSize).numberOfPlayer, this.row)
+		dismiss()
+	}
+
+	override fun stringForRow(row: RowRepresentable): String {
+		this.context?.let {
+			return row.localizedTitle(it)
+		}
+		return "UNKNOWN CONTEXT FOR ROW $row"
+	}
+}
+
+/*
 class BottomSheetTableSizeGridFragment : BottomSheetFragment() {
 
 	private var dataList: ArrayList<String> = ArrayList()
@@ -79,4 +194,6 @@ class BottomSheetTableSizeGridFragment : BottomSheetFragment() {
 
 	}
 
-}
\ No newline at end of file
+}
+
+		*/
\ No newline at end of file
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt b/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
index 3dab41d9..d1dbeb88 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
@@ -6,11 +6,13 @@ import android.view.ViewGroup
 import androidx.appcompat.widget.AppCompatTextView
 import androidx.constraintlayout.widget.ConstraintLayout
 import androidx.recyclerview.widget.RecyclerView
+import kotlinx.android.synthetic.main.row_bottom_sheet_grid_title.view.*
 import kotlinx.android.synthetic.main.row_header_title_value.view.*
 import kotlinx.android.synthetic.main.row_title.view.*
 import kotlinx.android.synthetic.main.row_title_value.view.*
 import kotlinx.android.synthetic.main.row_title_value_action.view.*
 import net.pokeranalytics.android.R
+import net.pokeranalytics.android.ui.adapter.TableSizeGridAdapter
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
 
 /**
@@ -30,7 +32,8 @@ enum class RowViewType {
 	TITLE_VALUE,
 	TITLE_VALUE_ACTION,
 	DATA,
-	BOTTOM_SHEET_DATA;
+	BOTTOM_SHEET_DATA,
+	TITLE_GRID;
 
 	inner class FakeViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView),
 		BindableHolder {
@@ -38,6 +41,13 @@ enum class RowViewType {
 		}
 	}
 
+	inner class CellSessionViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView), BindableHolder {
+		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
+			itemView.title.text = row.localizedTitle(itemView.context)
+			itemView.container.setOnClickListener(listener)
+		}
+	}
+
 	inner class DataViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView), BindableHolder {
 		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
 			itemView.findViewById<AppCompatTextView>(R.id.rowTitle_title).text = row.getDisplayName()
@@ -147,6 +157,12 @@ enum class RowViewType {
 					false
 				)
 			)
+			TITLE_GRID -> CellSessionViewHolder(
+				LayoutInflater.from(parent.context).inflate(
+					R.layout.row_bottom_sheet_grid_title,
+					parent,
+					false)
+			)
 			else -> FakeViewHolder(parent)
 		}
 	}

From 1dc6f591e51c5866ab664a5059f9f86f60f85609 Mon Sep 17 00:00:00 2001
From: Razmig Sarkissian <razmig.sarkissian@staxriver.com>
Date: Fri, 22 Feb 2019 14:41:34 +0100
Subject: [PATCH 12/18] remove usage of HistoryAdapter

---
 .../android/model/realm/Session.kt            |  6 ++-
 .../android/ui/activity/HomeActivity.kt       |  3 +-
 .../android/ui/fragment/HistoryFragment.kt    | 52 ++++++++++++-------
 .../android/ui/view/RowViewType.kt            | 19 ++++++-
 4 files changed, 57 insertions(+), 23 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index 9da0cf4b..67d2feeb 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -24,7 +24,7 @@ import java.util.*
 import kotlin.collections.ArrayList
 
 open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource,
-	RowEditable {
+	RowEditable, RowRepresentable {
 
 	@PrimaryKey
 	var id = UUID.randomUUID().toString()
@@ -242,6 +242,10 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 			return 0.0
 		}
 
+	override fun getDisplayName(): String {
+		return "session ${this.creationDate}"
+	}
+
 	override fun adapterRows(): ArrayList<RowRepresentable> {
 		val rows = ArrayList<RowRepresentable>()
 		rows.addAll(SessionRow.getRowsForState(getState()))
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/activity/HomeActivity.kt b/app/src/main/java/net/pokeranalytics/android/ui/activity/HomeActivity.kt
index b8203b78..b1c395b9 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/activity/HomeActivity.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/activity/HomeActivity.kt
@@ -3,6 +3,7 @@ package net.pokeranalytics.android.ui.activity
 import android.content.Context
 import android.content.Intent
 import android.os.Bundle
+import androidx.fragment.app.Fragment
 import com.google.android.material.bottomnavigation.BottomNavigationView
 import kotlinx.android.synthetic.main.activity_home.*
 import net.pokeranalytics.android.R
@@ -68,7 +69,7 @@ class HomeActivity : PokerAnalyticsActivity() {
 			else -> ""
 		}
 
-		val fragment = when(index) {
+		val fragment: Fragment = when(index) {
 			0 -> HistoryFragment()
 			1 -> StatsFragment()
 			else -> SettingsFragment()
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
index 517aeca3..e43a9487 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
@@ -9,13 +9,19 @@ import androidx.recyclerview.widget.LinearLayoutManager
 import io.realm.RealmResults
 import kotlinx.android.synthetic.main.fragment_history.*
 import net.pokeranalytics.android.R
+import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.model.realm.Session
 import net.pokeranalytics.android.ui.activity.SessionActivity
 import net.pokeranalytics.android.ui.adapter.HistoryAdapter
+import net.pokeranalytics.android.ui.adapter.components.RowRepresentableAdapter
+import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
+import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDelegate
 import net.pokeranalytics.android.ui.fragment.components.PokerAnalyticsFragment
+import net.pokeranalytics.android.ui.view.RowRepresentable
+import net.pokeranalytics.android.ui.view.RowViewType
 import net.pokeranalytics.android.util.data.sessionDao
 
-class HistoryFragment : PokerAnalyticsFragment() {
+class HistoryFragment : PokerAnalyticsFragment(), RowRepresentableDataSource, RowRepresentableDelegate {
 
 	companion object {
 		fun newInstance(): HistoryFragment {
@@ -26,8 +32,8 @@ class HistoryFragment : PokerAnalyticsFragment() {
 		}
 	}
 
-	private lateinit var historyAdapter: HistoryAdapter
-	private var realmSessions: RealmResults<Session>? = null
+	private lateinit var historyAdapter: RowRepresentableAdapter
+	private lateinit var realmSessions: RealmResults<Session>
 
 	override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
 		return inflater.inflate(R.layout.fragment_history, container, false)
@@ -41,7 +47,7 @@ class HistoryFragment : PokerAnalyticsFragment() {
 
 	override fun onDestroyView() {
 		super.onDestroyView()
-		realmSessions?.removeAllChangeListeners()
+		realmSessions.removeAllChangeListeners()
 	}
 
 	/**
@@ -70,28 +76,34 @@ class HistoryFragment : PokerAnalyticsFragment() {
 	 * Init data
 	 */
 	private fun initData() {
-
 		realmSessions = getRealm().sessionDao().findAllSessions()
-		realmSessions?.let {
+		val viewManager = LinearLayoutManager(requireContext())
+		historyAdapter = RowRepresentableAdapter(this, this)
 
-			val viewManager = LinearLayoutManager(requireContext())
-			historyAdapter = HistoryAdapter(it)
+		recyclerView.apply {
+			setHasFixedSize(true)
+			layoutManager = viewManager
+			adapter = historyAdapter
+		}
+	}
 
-			recyclerView.apply {
-				setHasFixedSize(true)
-				layoutManager = viewManager
-				adapter = historyAdapter
-			}
+	override fun rowRepresentableForPosition(position: Int): RowRepresentable {
+		return this.realmSessions[position] as RowRepresentable
+	}
 
-			historyAdapter.onClickOnSession = {position, session ->
-				SessionActivity.newInstance(requireContext(), sessionId = session.id)
-			}
+	override fun numberOfRows(): Int {
+		return this.realmSessions.size
+	}
 
-			it.addChangeListener { newSessions ->
-				historyAdapter.notifyDataSetChanged()
-			}
-		}
+	override fun viewTypeForPosition(position: Int): Int {
+		return RowViewType.ROW_SESSION.ordinal
+	}
 
+	override fun indexForRow(row: RowRepresentable): Int {
+		return this.realmSessions.indexOf(row)
 	}
 
+	override fun onRowSelected(row: RowRepresentable) {
+		SessionActivity.newInstance(requireContext(), sessionId = (row as ObjectSavable).uniqueIdentifier())
+	}
 }
\ No newline at end of file
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt b/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
index d1dbeb88..a0845f5d 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/view/RowViewType.kt
@@ -8,10 +8,12 @@ import androidx.constraintlayout.widget.ConstraintLayout
 import androidx.recyclerview.widget.RecyclerView
 import kotlinx.android.synthetic.main.row_bottom_sheet_grid_title.view.*
 import kotlinx.android.synthetic.main.row_header_title_value.view.*
+import kotlinx.android.synthetic.main.row_history_session.view.*
 import kotlinx.android.synthetic.main.row_title.view.*
 import kotlinx.android.synthetic.main.row_title_value.view.*
 import kotlinx.android.synthetic.main.row_title_value_action.view.*
 import net.pokeranalytics.android.R
+import net.pokeranalytics.android.model.realm.Session
 import net.pokeranalytics.android.ui.adapter.TableSizeGridAdapter
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
 
@@ -33,7 +35,8 @@ enum class RowViewType {
 	TITLE_VALUE_ACTION,
 	DATA,
 	BOTTOM_SHEET_DATA,
-	TITLE_GRID;
+	TITLE_GRID,
+	ROW_SESSION;
 
 	inner class FakeViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView),
 		BindableHolder {
@@ -41,6 +44,14 @@ enum class RowViewType {
 		}
 	}
 
+	inner class RowSessionViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView),
+		BindableHolder {
+		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
+				itemView.sessionRow.setData(row as Session)
+				itemView.sessionRow.setOnClickListener(listener)
+		}
+	}
+
 	inner class CellSessionViewHolder(itemView: View) : RecyclerView.ViewHolder(itemView), BindableHolder {
 		override fun bind(row: RowRepresentable, rowRepresentableDataSource: RowRepresentableDataSource?, listener: View.OnClickListener, actionListener: View.OnClickListener?) {
 			itemView.title.text = row.localizedTitle(itemView.context)
@@ -163,6 +174,12 @@ enum class RowViewType {
 					parent,
 					false)
 			)
+			ROW_SESSION -> RowSessionViewHolder(
+				LayoutInflater.from(parent.context).inflate(
+					R.layout.row_history_session,
+					parent,
+					false)
+			)
 			else -> FakeViewHolder(parent)
 		}
 	}

From 4d52f794dcf60295c32a1f0a4dcab731a43058ee Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 15:13:55 +0100
Subject: [PATCH 13/18] Finish merge

---
 .../java/net/pokeranalytics/android/model/realm/Session.kt | 7 ++++++-
 .../pokeranalytics/android/ui/fragment/HistoryFragment.kt  | 1 -
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index ae1cf2ad..e34e855d 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -8,6 +8,7 @@ import io.realm.annotations.PrimaryKey
 import net.pokeranalytics.android.R
 import net.pokeranalytics.android.calculus.SessionInterface
 import net.pokeranalytics.android.model.LiveData
+import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.model.extensions.SessionState
 import net.pokeranalytics.android.model.extensions.getState
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
@@ -24,7 +25,11 @@ import java.util.*
 import kotlin.collections.ArrayList
 
 open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource,
-	RowEditable, RowRepresentable {
+	RowEditable, RowRepresentable, ObjectSavable {
+
+	override fun uniqueIdentifier(): String {
+		return id
+	}
 
 	@PrimaryKey
 	var id = UUID.randomUUID().toString()
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
index e43a9487..fac9aac3 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
@@ -12,7 +12,6 @@ import net.pokeranalytics.android.R
 import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.model.realm.Session
 import net.pokeranalytics.android.ui.activity.SessionActivity
-import net.pokeranalytics.android.ui.adapter.HistoryAdapter
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableAdapter
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDelegate

From deec0aed26c67b1e71c8f63f53abecba95173894 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 15:16:55 +0100
Subject: [PATCH 14/18] Foramt code

---
 .../net/pokeranalytics/android/model/realm/Session.kt     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index e34e855d..9775dd2c 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -27,10 +27,6 @@ import kotlin.collections.ArrayList
 open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource,
 	RowEditable, RowRepresentable, ObjectSavable {
 
-	override fun uniqueIdentifier(): String {
-		return id
-	}
-
 	@PrimaryKey
 	var id = UUID.randomUUID().toString()
 
@@ -254,6 +250,10 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 			return 0.0
 		}
 
+	override fun uniqueIdentifier(): String {
+		return this.id
+	}
+
 	override fun getDisplayName(): String {
 		return "session ${this.creationDate}"
 	}

From d21f0bdfd180b115065ece07a59153f82d7e1167 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 15:17:01 +0100
Subject: [PATCH 15/18] Update UI when resume

---
 .../pokeranalytics/android/ui/fragment/HistoryFragment.kt    | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
index fac9aac3..0fbe03a7 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
@@ -49,6 +49,11 @@ class HistoryFragment : PokerAnalyticsFragment(), RowRepresentableDataSource, Ro
 		realmSessions.removeAllChangeListeners()
 	}
 
+	override fun onResume() {
+		super.onResume()
+		historyAdapter.notifyDataSetChanged()
+	}
+
 	/**
 	 * Init UI
 	 */

From 1a56cfb4196440e80ff4564ed2f80cc026fc62be Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 15:23:47 +0100
Subject: [PATCH 16/18] Clean unnecessary files & code

---
 .../android/model/realm/Session.kt            | 71 +------------------
 .../android/util/data/LiveRealmData.kt        | 21 ------
 .../android/util/data/Realm+Dao.kt            | 11 ---
 3 files changed, 2 insertions(+), 101 deletions(-)
 delete mode 100644 app/src/main/java/net/pokeranalytics/android/util/data/LiveRealmData.kt
 delete mode 100644 app/src/main/java/net/pokeranalytics/android/util/data/Realm+Dao.kt

diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index 9775dd2c..0b3a11ec 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -2,7 +2,8 @@ package net.pokeranalytics.android.model.realm
 
 import android.content.Context
 import android.text.InputType
-import io.realm.*
+import io.realm.RealmList
+import io.realm.RealmObject
 import io.realm.annotations.Ignore
 import io.realm.annotations.PrimaryKey
 import net.pokeranalytics.android.R
@@ -16,7 +17,6 @@ import net.pokeranalytics.android.ui.fragment.components.bottomsheet.BottomSheet
 import net.pokeranalytics.android.ui.view.RowEditable
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.SessionRow
-import net.pokeranalytics.android.util.data.sessionDao
 import net.pokeranalytics.android.util.getDuration
 import net.pokeranalytics.android.util.round
 import net.pokeranalytics.android.util.short
@@ -420,70 +420,3 @@ enum class TournamentKind {
 	MTT,
 	SNG
 }
-
-
-/**
- * Session Dao
- */
-class SessionDao(realmDb: Realm) {
-
-	var realm: Realm = realmDb
-
-	/**
-	 * Create or update session
-	 */
-	fun createOrUpdateSession(session: Session): Session {
-		realm.beginTransaction()
-		val sessionToSave = realm.copyToRealmOrUpdate(session)
-		realm.commitTransaction()
-		return realm.copyFromRealm(sessionToSave)
-	}
-
-	/**
-	 * Create or update sessions
-	 */
-	fun createOrUpdateSessions(sessions: List<Session>): List<Session> {
-
-		realm.beginTransaction()
-
-		// Update
-		val sessionsToSave = realm.copyToRealmOrUpdate(sessions)
-		realm.commitTransaction()
-
-		return realm.copyFromRealm(sessionsToSave)
-	}
-
-	/**
-	 * Find all sessions
-	 */
-	fun findAllSessions(): RealmResults<Session> {
-		return realm.where(Session::class.java).findAll().sort("creationDate", Sort.DESCENDING)
-	}
-
-	/**
-	 * Find session by id
-	 */
-	fun findSessionById(sessionId: Int): Session? {
-		return realm.copyFromRealm(realm.where(Session::class.java).equalTo("id", sessionId).findFirst())
-	}
-
-	/**
-	 * Delete session
-	 */
-	fun deleteSession(sessionId: Int) {
-		realm.beginTransaction()
-		realm.sessionDao().findSessionById(sessionId)?.deleteFromRealm()
-		realm.commitTransaction()
-	}
-
-	/**
-	 * Delete all sessions
-	 */
-	fun deleteAllSessions() {
-		realm.beginTransaction()
-		realm.sessionDao().findAllSessions().deleteAllFromRealm()
-		realm.commitTransaction()
-	}
-
-
-}
\ No newline at end of file
diff --git a/app/src/main/java/net/pokeranalytics/android/util/data/LiveRealmData.kt b/app/src/main/java/net/pokeranalytics/android/util/data/LiveRealmData.kt
deleted file mode 100644
index 3edd4d69..00000000
--- a/app/src/main/java/net/pokeranalytics/android/util/data/LiveRealmData.kt
+++ /dev/null
@@ -1,21 +0,0 @@
-package net.pokeranalytics.android.util.data
-
-import androidx.lifecycle.LiveData
-import io.realm.RealmChangeListener
-import io.realm.RealmModel
-import io.realm.RealmResults
-
-
-class LiveRealmData<T : RealmModel>(private val results: RealmResults<T>) : LiveData<RealmResults<T>>() {
-	private val listener = RealmChangeListener<RealmResults<T>> { results -> value = results }
-	
-	override fun onActive() {
-		results.addChangeListener(listener)
-		value = results
-	}
-	
-	override fun onInactive() {
-		results.removeChangeListener(listener)
-	}
-	
-}
\ No newline at end of file
diff --git a/app/src/main/java/net/pokeranalytics/android/util/data/Realm+Dao.kt b/app/src/main/java/net/pokeranalytics/android/util/data/Realm+Dao.kt
deleted file mode 100644
index f09412c6..00000000
--- a/app/src/main/java/net/pokeranalytics/android/util/data/Realm+Dao.kt
+++ /dev/null
@@ -1,11 +0,0 @@
-@file:JvmName("RealmUtils") // pretty name for utils class if called from
-package net.pokeranalytics.android.util.data
-
-import io.realm.Realm
-import io.realm.RealmModel
-import io.realm.RealmResults
-import net.pokeranalytics.android.model.realm.SessionDao
-
-fun Realm.sessionDao(): SessionDao = SessionDao(this)
-
-fun <T:RealmModel> RealmResults<T>.asLiveData() = LiveRealmData(this)
\ No newline at end of file

From 53fcd28c78b0d899bce56fb8f8dc5cb3c54b1671 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 15:37:34 +0100
Subject: [PATCH 17/18] Update UI

---
 .../pokeranalytics/android/ui/fragment/HistoryFragment.kt  | 5 +++--
 .../pokeranalytics/android/ui/fragment/SessionFragment.kt  | 4 +---
 app/src/main/res/font/roboto_medium.xml                    | 7 +++++++
 app/src/main/res/font/roboto_mono_medium.xml               | 7 +++++++
 app/src/main/res/layout/row_session_view.xml               | 1 +
 app/src/main/res/values/preloaded_fonts.xml                | 2 ++
 app/src/main/res/values/styles.xml                         | 2 +-
 7 files changed, 22 insertions(+), 6 deletions(-)
 create mode 100644 app/src/main/res/font/roboto_medium.xml
 create mode 100644 app/src/main/res/font/roboto_mono_medium.xml

diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
index 0fbe03a7..b9a2a85c 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/HistoryFragment.kt
@@ -7,6 +7,8 @@ import android.view.ViewGroup
 import android.widget.Toast
 import androidx.recyclerview.widget.LinearLayoutManager
 import io.realm.RealmResults
+import io.realm.Sort
+import io.realm.kotlin.where
 import kotlinx.android.synthetic.main.fragment_history.*
 import net.pokeranalytics.android.R
 import net.pokeranalytics.android.model.ObjectSavable
@@ -18,7 +20,6 @@ import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDelegate
 import net.pokeranalytics.android.ui.fragment.components.PokerAnalyticsFragment
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.RowViewType
-import net.pokeranalytics.android.util.data.sessionDao
 
 class HistoryFragment : PokerAnalyticsFragment(), RowRepresentableDataSource, RowRepresentableDelegate {
 
@@ -80,7 +81,7 @@ class HistoryFragment : PokerAnalyticsFragment(), RowRepresentableDataSource, Ro
 	 * Init data
 	 */
 	private fun initData() {
-		realmSessions = getRealm().sessionDao().findAllSessions()
+		realmSessions = getRealm().where<Session>().findAll().sort("timeFrame.startDate", Sort.DESCENDING)
 		val viewManager = LinearLayoutManager(requireContext())
 		historyAdapter = RowRepresentableAdapter(this, this)
 
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt
index 71576060..f06fca4b 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/SessionFragment.kt
@@ -36,7 +36,7 @@ class SessionFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Bott
 	private lateinit var currentSession: Session
 	private lateinit var sessionAdapter: RowRepresentableAdapter
 	private var sessionMenu: Menu? = null
-
+	private val oldRows: ArrayList<RowRepresentable> = ArrayList()
 
 	override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View? {
 		return inflater.inflate(R.layout.fragment_session, container, false)
@@ -235,8 +235,6 @@ class SessionFragment : PokerAnalyticsFragment(), RowRepresentableDelegate, Bott
 		}
 	}
 
-	val oldRows: ArrayList<RowRepresentable> = ArrayList()
-
 	/**
 	 * Update the state of the session (start / pause)
 	 */
diff --git a/app/src/main/res/font/roboto_medium.xml b/app/src/main/res/font/roboto_medium.xml
new file mode 100644
index 00000000..59df90bc
--- /dev/null
+++ b/app/src/main/res/font/roboto_medium.xml
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+<font-family xmlns:app="http://schemas.android.com/apk/res-auto"
+        app:fontProviderAuthority="com.google.android.gms.fonts"
+        app:fontProviderPackage="com.google.android.gms"
+        app:fontProviderQuery="name=Roboto&amp;weight=500"
+        app:fontProviderCerts="@array/com_google_android_gms_fonts_certs">
+</font-family>
diff --git a/app/src/main/res/font/roboto_mono_medium.xml b/app/src/main/res/font/roboto_mono_medium.xml
new file mode 100644
index 00000000..485dd2d9
--- /dev/null
+++ b/app/src/main/res/font/roboto_mono_medium.xml
@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>
+<font-family xmlns:app="http://schemas.android.com/apk/res-auto"
+        app:fontProviderAuthority="com.google.android.gms.fonts"
+        app:fontProviderPackage="com.google.android.gms"
+        app:fontProviderQuery="name=Roboto Mono&amp;weight=500"
+        app:fontProviderCerts="@array/com_google_android_gms_fonts_certs">
+</font-family>
diff --git a/app/src/main/res/layout/row_session_view.xml b/app/src/main/res/layout/row_session_view.xml
index 9216e55c..ff4b18a8 100644
--- a/app/src/main/res/layout/row_session_view.xml
+++ b/app/src/main/res/layout/row_session_view.xml
@@ -14,6 +14,7 @@
 		android:layout_marginTop="8dp"
 		android:gravity="center"
 		android:textAllCaps="true"
+		app:fontFamily="@font/roboto_mono_medium"
 		app:layout_constraintBottom_toTopOf="@+id/dateNumber"
 		app:layout_constraintStart_toStartOf="@+id/guidelineStart"
 		app:layout_constraintTop_toTopOf="parent"
diff --git a/app/src/main/res/values/preloaded_fonts.xml b/app/src/main/res/values/preloaded_fonts.xml
index 212d1794..5030a3b4 100644
--- a/app/src/main/res/values/preloaded_fonts.xml
+++ b/app/src/main/res/values/preloaded_fonts.xml
@@ -5,5 +5,7 @@
 		<item>@font/roboto_black</item>
 		<item>@font/roboto_bold</item>
 		<item>@font/roboto_light</item>
+		<item>@font/roboto_medium</item>
+		<item>@font/roboto_mono_medium</item>
 	</array>
 </resources>
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index d124ef6f..c0eb1bdd 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -108,7 +108,7 @@
 
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.Date">
 		<item name="android:textSize">12sp</item>
-		<item name="android:fontFamily">@font/roboto</item>r
+		<item name="android:fontFamily">@font/roboto</item>
 	</style>
 
 	<style name="PokerAnalyticsTheme.TextView.SessionRow.DateNumber">

From a3b00a072674f2cd3cdedb39739da7a14da76319 Mon Sep 17 00:00:00 2001
From: Aurelien Hubert <aurel.hubert@gmail.com>
Date: Fri, 22 Feb 2019 16:50:06 +0100
Subject: [PATCH 18/18] Improve Game selection

---
 .../android/PokerAnalyticsApplication.kt      |  3 +
 .../net/pokeranalytics/android/model/Limit.kt | 26 ++++++++
 .../android/model/realm/Session.kt            | 27 ++++++--
 .../BottomSheetListGameFragment.kt            | 33 ++++++----
 .../android/ui/view/SessionRowView.kt         | 13 +---
 .../res/color/chips_background_states.xml     |  5 ++
 .../res/layout/bottom_sheet_game_list.xml     | 62 ++++++++++++++++---
 app/src/main/res/layout/fragment_session.xml  |  4 +-
 app/src/main/res/values/styles.xml            |  9 +++
 9 files changed, 146 insertions(+), 36 deletions(-)
 create mode 100644 app/src/main/java/net/pokeranalytics/android/model/Limit.kt
 create mode 100755 app/src/main/res/color/chips_background_states.xml

diff --git a/app/src/main/java/net/pokeranalytics/android/PokerAnalyticsApplication.kt b/app/src/main/java/net/pokeranalytics/android/PokerAnalyticsApplication.kt
index a9b1286f..f884ce17 100644
--- a/app/src/main/java/net/pokeranalytics/android/PokerAnalyticsApplication.kt
+++ b/app/src/main/java/net/pokeranalytics/android/PokerAnalyticsApplication.kt
@@ -5,6 +5,7 @@ import io.realm.Realm
 import io.realm.RealmConfiguration
 import io.realm.RealmResults
 import io.realm.kotlin.where
+import net.pokeranalytics.android.model.Limit
 import net.pokeranalytics.android.model.realm.Game
 import net.pokeranalytics.android.model.realm.Session
 import net.pokeranalytics.android.util.PokerAnalyticsLogs
@@ -52,6 +53,8 @@ class PokerAnalyticsApplication : Application() {
 			*/
 		}
 
+		Limit.init(this)
+
 		if (BuildConfig.DEBUG) {
 			// Logs
 			Timber.plant(PokerAnalyticsLogs())
diff --git a/app/src/main/java/net/pokeranalytics/android/model/Limit.kt b/app/src/main/java/net/pokeranalytics/android/model/Limit.kt
new file mode 100644
index 00000000..ca9f5402
--- /dev/null
+++ b/app/src/main/java/net/pokeranalytics/android/model/Limit.kt
@@ -0,0 +1,26 @@
+package net.pokeranalytics.android.model
+
+import android.content.Context
+import net.pokeranalytics.android.R
+
+class Limit {
+
+	companion object {
+		private val values = ArrayList<String>()
+		fun init(context: Context) {
+			values.clear()
+			values.addAll(context.resources.getStringArray(R.array.limit_short_name))
+		}
+
+		/**
+		 * Get a limit name
+		 */
+		fun get(index: Int) : String? {
+			if (index >= 0 && index < values.size) {
+				return values[index]
+			}
+			return ""
+		}
+	}
+
+}
\ No newline at end of file
diff --git a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
index 0b3a11ec..21d64b97 100644
--- a/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
+++ b/app/src/main/java/net/pokeranalytics/android/model/realm/Session.kt
@@ -8,6 +8,7 @@ import io.realm.annotations.Ignore
 import io.realm.annotations.PrimaryKey
 import net.pokeranalytics.android.R
 import net.pokeranalytics.android.calculus.SessionInterface
+import net.pokeranalytics.android.model.Limit
 import net.pokeranalytics.android.model.LiveData
 import net.pokeranalytics.android.model.ObjectSavable
 import net.pokeranalytics.android.model.extensions.SessionState
@@ -21,6 +22,7 @@ import net.pokeranalytics.android.util.getDuration
 import net.pokeranalytics.android.util.round
 import net.pokeranalytics.android.util.short
 import net.pokeranalytics.android.util.toCurrency
+import timber.log.Timber
 import java.util.*
 import kotlin.collections.ArrayList
 
@@ -198,13 +200,13 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 	 */
 	fun getGameTitle(): String {
 		var gameTitle = ""
-		if (limit != null) {
-			gameTitle += limit
+		limit?.let {
+			gameTitle += Limit.get(it) + " "
 		}
 		if (game != null) {
 			gameTitle += game?.name
 		}
-		return gameTitle
+		return if (gameTitle.isNotBlank()) gameTitle else "--"
 	}
 
 	companion object {
@@ -272,7 +274,7 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 		return when (row) {
 			SessionRow.BUY_IN -> buyin.toCurrency()
 			SessionRow.BLINDS -> if (cgSmallBlind != null && cgBigBlind != null) "$cgSmallBlind / $cgBigBlind" else "--"
-			SessionRow.GAME -> game?.name ?: "--"
+			SessionRow.GAME -> getGameTitle()
 			SessionRow.LOCATION -> location?.name ?: "--"
 			SessionRow.BANKROLL -> bankroll?.name ?: "--"
 			SessionRow.TABLE_SIZE -> tableSize?.toString() ?: "--"
@@ -370,7 +372,22 @@ open class Session : RealmObject(), SessionInterface, RowRepresentableDataSource
 				result = localResult
 			}
 			SessionRow.TABLE_SIZE -> tableSize = value as Int?
-			SessionRow.GAME -> game = value as Game?
+			SessionRow.GAME -> {
+				if (value is ArrayList<*>) {
+					Timber.d("${value[0]}")
+					Timber.d("${value[1]}")
+					limit = try {
+						(value[0] as Int?)
+					} catch (e: Exception) {
+						null
+					}
+					game = try {
+						(value[1] as Game?)
+					} catch (e: Exception) {
+						null
+					}
+				}
+			}
 			SessionRow.BANKROLL -> bankroll = value as Bankroll?
 			SessionRow.LOCATION -> location = value as Location?
 			SessionRow.COMMENT -> comment = value as String? ?: ""
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
index ad37aa93..deb023f0 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/fragment/components/bottomsheet/BottomSheetListGameFragment.kt
@@ -4,22 +4,24 @@ import android.os.Bundle
 import android.view.LayoutInflater
 import android.view.View
 import androidx.recyclerview.widget.LinearLayoutManager
+import com.google.android.material.chip.Chip
 import io.realm.RealmResults
 import kotlinx.android.synthetic.main.bottom_sheet_game_list.*
 import kotlinx.android.synthetic.main.fragment_bottom_sheet.view.*
 import net.pokeranalytics.android.R
-import net.pokeranalytics.android.ui.adapter.LimitTypesAdapter
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableAdapter
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDataSource
 import net.pokeranalytics.android.ui.adapter.components.RowRepresentableDelegate
 import net.pokeranalytics.android.ui.view.RowRepresentable
 import net.pokeranalytics.android.ui.view.RowViewType
+import timber.log.Timber
 
 
 class BottomSheetListGameFragment : BottomSheetFragment(), RowRepresentableDataSource, RowRepresentableDelegate {
 
 	private var realmData: RealmResults<*>? = null
 	private lateinit var dataAdapter: RowRepresentableAdapter
+	private val values = ArrayList<Any?>()
 
 	override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
 		super.onViewCreated(view, savedInstanceState)
@@ -32,6 +34,9 @@ class BottomSheetListGameFragment : BottomSheetFragment(), RowRepresentableDataS
 		dataAdapter.notifyDataSetChanged()
 	}
 
+	override fun getValue(): Any? {
+		return values
+	}
 
 	override fun rowRepresentableForPosition(position: Int): RowRepresentable {
 		realmData?.let {
@@ -61,8 +66,9 @@ class BottomSheetListGameFragment : BottomSheetFragment(), RowRepresentableDataS
 	override fun onIndexSelected(position: Int) {
 		realmData?.let {
 			val selectedData = it[position]
-			selectedData?.let {data ->
-				bottomSheetDelegate.setValue(data, row)
+			selectedData?.let { data ->
+				values[1] = data
+				bottomSheetDelegate.setValue(values, row)
 				dismiss()
 			}
 		}
@@ -86,18 +92,23 @@ class BottomSheetListGameFragment : BottomSheetFragment(), RowRepresentableDataS
 
 		LayoutInflater.from(requireContext()).inflate(R.layout.bottom_sheet_game_list, view?.bottomSheetContainer, true)
 
+		values.add(0, null)
+		values.add(1, null)
+
 		val limits = ArrayList<String>()
 		limits.addAll(resources.getStringArray(R.array.limit_short_name))
 
+		chipGroup.removeAllViews()
+		for ((index, limit) in limits.withIndex()) {
+			val chip = Chip(requireContext())
+			chip.text = limit
+			chip.id = index
+			chipGroup.addView(chip)
+		}
 
-		val viewManager1 = LinearLayoutManager(requireContext())
-		val gameDataAdapter1 = LimitTypesAdapter(limits)
-
-		recyclerView1.apply {
-			setHasFixedSize(true)
-			layoutManager = viewManager1
-			adapter = gameDataAdapter1
-			isNestedScrollingEnabled = false
+		chipGroup.setOnCheckedChangeListener { chipGroup, i ->
+			Timber.d("Chip selected: $i")
+			values[0] = i
 		}
 
 
diff --git a/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt b/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt
index 0161b68c..b286df75 100644
--- a/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt
+++ b/app/src/main/java/net/pokeranalytics/android/ui/view/SessionRowView.kt
@@ -10,6 +10,7 @@ import net.pokeranalytics.android.R
 import net.pokeranalytics.android.model.realm.Session
 import net.pokeranalytics.android.util.getDayNumber
 import net.pokeranalytics.android.util.getShortDayName
+import net.pokeranalytics.android.util.toCurrency
 
 
 class SessionRowView : FrameLayout {
@@ -55,17 +56,9 @@ class SessionRowView : FrameLayout {
 		rowHistorySession.dateDay.text = session.creationDate.getShortDayName()
 		rowHistorySession.dateNumber.text = session.creationDate.getDayNumber()
 
-		// TODO
-
-		var gameTitle = session.getGameTitle()
-		if (gameTitle.isEmpty()) {
-			gameTitle = "Game title here"
-		}
-
-		rowHistorySession.gameType.text = gameTitle
-
+		rowHistorySession.gameType.text = session.getGameTitle()
 		rowHistorySession.gameInfo.text = session.getDuration(context)
-		rowHistorySession.gameResult.text = "$ 0"
+		rowHistorySession.gameResult.text = session.result?.netResult?.toCurrency() ?: "$0"
 
 	}
 
diff --git a/app/src/main/res/color/chips_background_states.xml b/app/src/main/res/color/chips_background_states.xml
new file mode 100755
index 00000000..7d57a042
--- /dev/null
+++ b/app/src/main/res/color/chips_background_states.xml
@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="utf-8"?>
+<selector xmlns:android="http://schemas.android.com/apk/res/android">
+	<item android:color="@color/colorAccent" android:state_checked="true" />
+	<item android:color="@android:color/transparent" />
+</selector>
\ No newline at end of file
diff --git a/app/src/main/res/layout/bottom_sheet_game_list.xml b/app/src/main/res/layout/bottom_sheet_game_list.xml
index 7f658f14..7f5a61b1 100644
--- a/app/src/main/res/layout/bottom_sheet_game_list.xml
+++ b/app/src/main/res/layout/bottom_sheet_game_list.xml
@@ -2,16 +2,62 @@
 <androidx.constraintlayout.widget.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="wrap_content"
+	android:layout_height="240dp"
 	android:orientation="horizontal">
 
-	<androidx.recyclerview.widget.RecyclerView
-		android:id="@+id/recyclerView1"
-		android:layout_width="80dp"
+	<FrameLayout
+		android:id="@+id/chips"
+		android:layout_width="0dp"
+		android:paddingTop="8dp"
+		android:paddingBottom="8dp"
 		android:layout_height="wrap_content"
-		app:layout_constraintBottom_toBottomOf="parent"
+		app:layout_constraintEnd_toEndOf="parent"
 		app:layout_constraintStart_toStartOf="parent"
-		app:layout_constraintTop_toTopOf="parent" />
+		app:layout_constraintTop_toTopOf="parent">
+
+		<com.google.android.material.chip.ChipGroup
+			android:id="@+id/chipGroup"
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:layout_gravity="center"
+			android:orientation="horizontal"
+			app:chipSpacingHorizontal="16dp"
+			app:singleSelection="true">
+
+			<!--
+			<com.google.android.material.chip.Chip
+				android:id="@+id/chip1"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:text="NL" />
+
+			<com.google.android.material.chip.Chip
+				android:id="@+id/chip2"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:text="PL" />
+
+			<com.google.android.material.chip.Chip
+				android:id="@+id/chip3"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:text="FL" />
+
+			<com.google.android.material.chip.Chip
+				android:id="@+id/chip4"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:text="SL" />
+
+			<com.google.android.material.chip.Chip
+				android:id="@+id/chip5"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:text="ML" />
+			-->
+
+		</com.google.android.material.chip.ChipGroup>
+	</FrameLayout>
 
 	<androidx.recyclerview.widget.RecyclerView
 		android:id="@+id/recyclerView2"
@@ -19,7 +65,7 @@
 		android:layout_height="0dp"
 		app:layout_constraintBottom_toBottomOf="parent"
 		app:layout_constraintEnd_toEndOf="parent"
-		app:layout_constraintStart_toEndOf="@+id/recyclerView1"
-		app:layout_constraintTop_toTopOf="parent" />
+		app:layout_constraintStart_toStartOf="parent"
+		app:layout_constraintTop_toBottomOf="@+id/chips" />
 
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/fragment_session.xml b/app/src/main/res/layout/fragment_session.xml
index 2d74bac0..6da823a1 100644
--- a/app/src/main/res/layout/fragment_session.xml
+++ b/app/src/main/res/layout/fragment_session.xml
@@ -47,7 +47,7 @@
 					android:layout_height="wrap_content"
 					android:layout_marginStart="16dp"
 					android:layout_marginBottom="16dp"
-					android:text="2h33"
+					android:text="-"
 					android:textSize="24sp"
 					app:fontFamily="@font/roboto_bold"
 					app:layout_constraintBottom_toBottomOf="parent"
@@ -75,7 +75,7 @@
 					android:layout_height="wrap_content"
 					android:layout_marginEnd="16dp"
 					android:layout_marginBottom="16dp"
-					android:text="$ 325"
+					android:text="-"
 					android:textColor="@color/green"
 					android:textSize="28sp"
 					app:layout_constraintBottom_toBottomOf="parent"
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index c0eb1bdd..aa5feb46 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -15,6 +15,7 @@
 		<item name="editTextStyle">@style/PokerAnalyticsTheme.EditText</item>
 		<item name="android:textViewStyle">@style/PokerAnalyticsTheme.TextView</item>
 		<item name="alertDialogTheme">@style/PokerAnalyticsTheme.AlertDialog</item>
+		<item name="chipStyle">@style/PokerAnalyticsTheme.Chip</item>
 
 	</style>
 
@@ -159,5 +160,13 @@
 		<item name="android:fontFamily">@font/roboto_bold</item>
 	</style>
 
+	<!-- Chip -->
+	<style name="PokerAnalyticsTheme.Chip" parent="Widget.MaterialComponents.Chip.Choice">
+		<item name="chipBackgroundColor">@color/chips_background_states</item>
+		<item name="android:textColor">@color/white</item>
+		<item name="chipStrokeColor">@color/colorAccent</item>
+		<item name="chipStrokeWidth">1dp</item>
+
+	</style>
 
 </resources>