From ae0b405ba5b2a1bc3fdacdb8acf103910270a90a Mon Sep 17 00:00:00 2001
From: jsericksk <jsericksk@gmail.com>
Date: Fri, 16 Feb 2024 23:48:37 -0300
Subject: [PATCH 1/5] Add grid view option to chapters menu

---
 .idea/.name                                            |  1 +
 .idea/gradle.xml                                       |  3 +--
 .../org/koitharu/kotatsu/core/prefs/AppSettings.kt     |  5 +++++
 .../kotatsu/details/ui/ChaptersMenuProvider.kt         |  7 ++++++-
 .../koitharu/kotatsu/details/ui/DetailsViewModel.kt    | 10 ++++++++++
 app/src/main/res/menu/opt_chapters.xml                 |  7 +++++++
 6 files changed, 30 insertions(+), 3 deletions(-)
 create mode 100644 .idea/.name

diff --git a/.idea/.name b/.idea/.name
new file mode 100644
index 000000000..643c6ed7e
--- /dev/null
+++ b/.idea/.name
@@ -0,0 +1 @@
+Kotatsu
\ No newline at end of file
diff --git a/.idea/gradle.xml b/.idea/gradle.xml
index 9f47dfb43..0897082f7 100644
--- a/.idea/gradle.xml
+++ b/.idea/gradle.xml
@@ -4,9 +4,8 @@
   <component name="GradleSettings">
     <option name="linkedExternalProjectsSettings">
       <GradleProjectSettings>
-        <option name="testRunner" value="GRADLE" />
         <option name="externalProjectPath" value="$PROJECT_DIR$" />
-        <option name="gradleJvm" value="jbr-17" />
+        <option name="gradleJvm" value="#GRADLE_LOCAL_JAVA_HOME" />
         <option name="modules">
           <set>
             <option value="$PROJECT_DIR$" />
diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/core/prefs/AppSettings.kt b/app/src/main/kotlin/org/koitharu/kotatsu/core/prefs/AppSettings.kt
index 931211d43..b6aa1e68b 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/core/prefs/AppSettings.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/core/prefs/AppSettings.kt
@@ -182,6 +182,10 @@ class AppSettings @Inject constructor(@ApplicationContext context: Context) {
 		get() = prefs.getBoolean(KEY_REVERSE_CHAPTERS, false)
 		set(value) = prefs.edit { putBoolean(KEY_REVERSE_CHAPTERS, value) }
 
+	var chaptersGridView: Boolean
+		get() = prefs.getBoolean(KEY_GRID_VIEW_CHAPTERS, false)
+		set(value) = prefs.edit { putBoolean(KEY_GRID_VIEW_CHAPTERS, value) }
+
 	val zoomMode: ZoomMode
 		get() = prefs.getEnumValue(KEY_ZOOM_MODE, ZoomMode.FIT_CENTER)
 
@@ -538,6 +542,7 @@ class AppSettings @Inject constructor(@ApplicationContext context: Context) {
 		const val KEY_HISTORY_GROUPING = "history_grouping"
 		const val KEY_READING_INDICATORS = "reading_indicators"
 		const val KEY_REVERSE_CHAPTERS = "reverse_chapters"
+		const val KEY_GRID_VIEW_CHAPTERS = "grid_view_chapters"
 		const val KEY_HISTORY_EXCLUDE_NSFW = "history_exclude_nsfw"
 		const val KEY_PAGES_NUMBERS = "pages_numbers"
 		const val KEY_SCREENSHOTS_POLICY = "screenshots_policy"
diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/ChaptersMenuProvider.kt b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/ChaptersMenuProvider.kt
index e42fa2ca8..8bc0cf218 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/ChaptersMenuProvider.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/ChaptersMenuProvider.kt
@@ -28,8 +28,9 @@ class ChaptersMenuProvider(
 	}
 
 	override fun onPrepareMenu(menu: Menu) {
-		menu.findItem(R.id.action_reversed)?.isChecked = viewModel.isChaptersReversed.value == true
 		menu.findItem(R.id.action_search)?.isVisible = viewModel.isChaptersEmpty.value == false
+		menu.findItem(R.id.action_reversed)?.isChecked = viewModel.isChaptersReversed.value == true
+		menu.findItem(R.id.action_grid_view)?.isChecked = viewModel.isChaptersInGridView.value == true
 	}
 
 	override fun onMenuItemSelected(menuItem: MenuItem): Boolean = when (menuItem.itemId) {
@@ -37,6 +38,10 @@ class ChaptersMenuProvider(
 			viewModel.setChaptersReversed(!menuItem.isChecked)
 			true
 		}
+		R.id.action_grid_view-> {
+			viewModel.setChaptersInGridView(!menuItem.isChecked)
+			true
+		}
 
 		else -> false
 	}
diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/DetailsViewModel.kt b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/DetailsViewModel.kt
index e9f6d59a2..d4ca3bb18 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/DetailsViewModel.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/DetailsViewModel.kt
@@ -119,6 +119,12 @@ class DetailsViewModel @Inject constructor(
 		valueProducer = { chaptersReverse },
 	)
 
+	val isChaptersInGridView = settings.observeAsStateFlow(
+		scope = viewModelScope + Dispatchers.Default,
+		key = AppSettings.KEY_GRID_VIEW_CHAPTERS,
+		valueProducer = { chaptersGridView },
+	)
+
 	val historyInfo: StateFlow<HistoryInfo> = combine(
 		manga,
 		selectedBranch,
@@ -278,6 +284,10 @@ class DetailsViewModel @Inject constructor(
 		settings.chaptersReverse = newValue
 	}
 
+	fun setChaptersInGridView(newValue: Boolean) {
+		settings.chaptersGridView = newValue
+	}
+
 	fun setSelectedBranch(branch: String?) {
 		selectedBranch.value = branch
 	}
diff --git a/app/src/main/res/menu/opt_chapters.xml b/app/src/main/res/menu/opt_chapters.xml
index ee53218ae..8fee83bac 100644
--- a/app/src/main/res/menu/opt_chapters.xml
+++ b/app/src/main/res/menu/opt_chapters.xml
@@ -18,4 +18,11 @@
 		android:title="@string/reverse"
 		app:showAsAction="never" />
 
+	<item
+		android:id="@+id/action_grid_view"
+		android:checkable="true"
+		android:orderInCategory="30"
+		android:title="@string/chapters_grid_view"
+		app:showAsAction="never" />
+
 </menu>

From 44adbde5362b8b8aa6f92955733b8ee470a481da Mon Sep 17 00:00:00 2001
From: jsericksk <jsericksk@gmail.com>
Date: Sat, 17 Feb 2024 01:24:24 -0300
Subject: [PATCH 2/5] Create chapterGridItemAD and its associated
 item_chapter_grid

---
 .../details/ui/adapter/chapterGridItemAD.kt   | 58 ++++++++++++++++
 app/src/main/res/layout/item_chapter_grid.xml | 68 +++++++++++++++++++
 2 files changed, 126 insertions(+)
 create mode 100644 app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/chapterGridItemAD.kt
 create mode 100644 app/src/main/res/layout/item_chapter_grid.xml

diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/chapterGridItemAD.kt b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/chapterGridItemAD.kt
new file mode 100644
index 000000000..28cc446f2
--- /dev/null
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/chapterGridItemAD.kt
@@ -0,0 +1,58 @@
+package org.koitharu.kotatsu.details.ui.adapter
+
+import android.graphics.Typeface
+import androidx.core.content.ContextCompat
+import androidx.core.view.isVisible
+import com.hannesdorfmann.adapterdelegates4.dsl.adapterDelegateViewBinding
+import org.koitharu.kotatsu.R
+import org.koitharu.kotatsu.core.ui.list.AdapterDelegateClickListenerAdapter
+import org.koitharu.kotatsu.core.ui.list.OnListItemClickListener
+import org.koitharu.kotatsu.core.util.ext.drawableStart
+import org.koitharu.kotatsu.core.util.ext.getThemeColorStateList
+import org.koitharu.kotatsu.databinding.ItemChapterGridBinding
+import org.koitharu.kotatsu.details.ui.model.ChapterListItem
+import org.koitharu.kotatsu.list.ui.model.ListModel
+import kotlin.math.roundToInt
+
+fun chapterGridItemAD(
+	clickListener: OnListItemClickListener<ChapterListItem>,
+) = adapterDelegateViewBinding<ChapterListItem, ListModel, ItemChapterGridBinding>(
+	{ inflater, parent -> ItemChapterGridBinding.inflate(inflater, parent, false) },
+) {
+
+	val eventListener = AdapterDelegateClickListenerAdapter(this, clickListener)
+	itemView.setOnClickListener(eventListener)
+	itemView.setOnLongClickListener(eventListener)
+
+	bind { payloads ->
+		if (payloads.isEmpty()) {
+			binding.textViewTitle.text = item.chapter.number.roundToInt().toString()
+		}
+		when {
+			item.isCurrent -> {
+				binding.textViewTitle.drawableStart = ContextCompat.getDrawable(context, R.drawable.ic_current_chapter)
+				binding.textViewTitle.setTextColor(context.getThemeColorStateList(android.R.attr.textColorPrimary))
+				binding.textViewTitle.typeface = Typeface.DEFAULT_BOLD
+			}
+
+			item.isUnread -> {
+				binding.textViewTitle.drawableStart = if (item.isNew) {
+					ContextCompat.getDrawable(context, R.drawable.ic_new)
+				} else {
+					null
+				}
+				binding.textViewTitle.setTextColor(context.getThemeColorStateList(android.R.attr.textColorPrimary))
+				binding.textViewTitle.typeface = Typeface.DEFAULT
+			}
+
+			else -> {
+				binding.textViewTitle.drawableStart = null
+				binding.textViewTitle.setTextColor(context.getThemeColorStateList(android.R.attr.textColorHint))
+				binding.textViewTitle.typeface = Typeface.DEFAULT
+			}
+		}
+		binding.imageViewBookmarked.isVisible = item.isBookmarked
+		binding.imageViewDownloaded.isVisible = item.isDownloaded
+	}
+}
+
diff --git a/app/src/main/res/layout/item_chapter_grid.xml b/app/src/main/res/layout/item_chapter_grid.xml
new file mode 100644
index 000000000..be845c63d
--- /dev/null
+++ b/app/src/main/res/layout/item_chapter_grid.xml
@@ -0,0 +1,68 @@
+<?xml version="1.0" encoding="utf-8"?>
+<androidx.constraintlayout.widget.ConstraintLayout
+	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="wrap_content">
+
+	<com.google.android.material.card.MaterialCardView
+		android:layout_width="0dp"
+		android:layout_height="0dp"
+		android:clipChildren="false"
+		app:cardCornerRadius="16dp"
+		app:cardUseCompatPadding="true"
+		app:layout_constraintBottom_toBottomOf="parent"
+		app:layout_constraintDimensionRatio="H,1:1"
+		app:layout_constraintEnd_toEndOf="parent"
+		app:layout_constraintStart_toStartOf="parent"
+		app:layout_constraintTop_toTopOf="parent">
+
+		<LinearLayout
+			android:layout_width="wrap_content"
+			android:layout_height="wrap_content"
+			android:layout_gravity="center"
+			android:orientation="vertical">
+
+			<TextView
+				android:id="@+id/textView_title"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:layout_gravity="center"
+				android:singleLine="true"
+				android:textAppearance="?attr/textAppearanceBodyLarge"
+				tools:text="150"
+				tools:textColor="?android:textColorPrimary" />
+
+			<LinearLayout
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:layout_gravity="center"
+				android:layout_marginTop="6dp"
+				android:orientation="horizontal">
+
+				<ImageView
+					android:id="@+id/imageView_bookmarked"
+					android:layout_width="20dp"
+					android:layout_height="20dp"
+					android:layout_marginStart="6dp"
+					android:layout_marginEnd="6dp"
+					android:contentDescription="@string/bookmarks"
+					app:srcCompat="@drawable/ic_bookmark" />
+
+				<ImageView
+					android:id="@+id/imageView_downloaded"
+					android:layout_width="20dp"
+					android:layout_height="20dp"
+					android:layout_marginStart="6dp"
+					android:layout_marginEnd="6dp"
+					android:contentDescription="@string/downloaded"
+					app:srcCompat="@drawable/ic_save_ok" />
+
+			</LinearLayout>
+
+		</LinearLayout>
+
+	</com.google.android.material.card.MaterialCardView>
+
+</androidx.constraintlayout.widget.ConstraintLayout>

From 0c823f1056579d6d6beb40413aab7b42b9b82bec Mon Sep 17 00:00:00 2001
From: jsericksk <jsericksk@gmail.com>
Date: Sat, 17 Feb 2024 02:09:35 -0300
Subject: [PATCH 3/5] Add feature to toggle between list and grid view for
 chapters

---
 .../kotatsu/details/ui/adapter/ChaptersAdapter.kt | 14 ++++++++++++--
 .../details/ui/pager/chapters/ChaptersFragment.kt | 15 +++++++++++++--
 .../koitharu/kotatsu/reader/ui/ChaptersSheet.kt   |  2 +-
 app/src/main/res/layout/item_chapter_grid.xml     |  2 +-
 4 files changed, 27 insertions(+), 6 deletions(-)

diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/ChaptersAdapter.kt b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/ChaptersAdapter.kt
index 2cdfbd26c..c14824000 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/ChaptersAdapter.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/adapter/ChaptersAdapter.kt
@@ -10,14 +10,24 @@ import org.koitharu.kotatsu.list.ui.adapter.listHeaderAD
 import org.koitharu.kotatsu.list.ui.model.ListModel
 
 class ChaptersAdapter(
-	onItemClickListener: OnListItemClickListener<ChapterListItem>,
+	private val onItemClickListener: OnListItemClickListener<ChapterListItem>,
+	chaptersInGridView: Boolean,
 ) : BaseListAdapter<ListModel>(), FastScroller.SectionIndexer {
 
 	init {
-		addDelegate(ListItemType.CHAPTER, chapterListItemAD(onItemClickListener))
+		setChapterAdapterDelegate(chaptersInGridView)
 		addDelegate(ListItemType.HEADER, listHeaderAD(null))
 	}
 
+	fun setChapterAdapterDelegate(chaptersInGridView: Boolean) {
+		delegatesManager.removeDelegate(ListItemType.CHAPTER.ordinal)
+		if (chaptersInGridView) {
+			addDelegate(ListItemType.CHAPTER, chapterGridItemAD(onItemClickListener))
+		} else {
+			addDelegate(ListItemType.CHAPTER, chapterListItemAD(onItemClickListener))
+		}
+	}
+
 	override fun getSectionText(context: Context, position: Int): CharSequence? {
 		return findHeader(position)?.getText(context)
 	}
diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/pager/chapters/ChaptersFragment.kt b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/pager/chapters/ChaptersFragment.kt
index 671576b52..d9ada0fb3 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/pager/chapters/ChaptersFragment.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/details/ui/pager/chapters/ChaptersFragment.kt
@@ -1,6 +1,7 @@
 package org.koitharu.kotatsu.details.ui.pager.chapters
 
 import android.os.Bundle
+import android.util.Log
 import android.view.LayoutInflater
 import android.view.Menu
 import android.view.MenuItem
@@ -11,6 +12,8 @@ import androidx.core.graphics.Insets
 import androidx.core.view.isVisible
 import androidx.fragment.app.activityViewModels
 import androidx.lifecycle.Lifecycle
+import androidx.recyclerview.widget.GridLayoutManager
+import androidx.recyclerview.widget.LinearLayoutManager
 import com.google.android.material.snackbar.Snackbar
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.flow.flowOn
@@ -55,7 +58,7 @@ class ChaptersFragment :
 
 	override fun onViewBindingCreated(binding: FragmentChaptersBinding, savedInstanceState: Bundle?) {
 		super.onViewBindingCreated(binding, savedInstanceState)
-		chaptersAdapter = ChaptersAdapter(this)
+		chaptersAdapter = ChaptersAdapter(this, viewModel.isChaptersInGridView.value)
 		selectionController = ListSelectionController(
 			activity = requireActivity(),
 			decoration = ChaptersSelectionDecoration(binding.root.context),
@@ -67,7 +70,15 @@ class ChaptersFragment :
 			checkNotNull(selectionController).attachToRecyclerView(this)
 			setHasFixedSize(true)
 			isNestedScrollingEnabled = false
-			adapter = chaptersAdapter
+		}
+		viewModel.isChaptersInGridView.observe(viewLifecycleOwner) { chaptersInGridView ->
+			chaptersAdapter?.setChapterAdapterDelegate(chaptersInGridView)
+			binding.recyclerViewChapters.adapter = chaptersAdapter
+			binding.recyclerViewChapters.layoutManager = if (chaptersInGridView) {
+				GridLayoutManager(context, 4)
+			} else {
+				LinearLayoutManager(context)
+			}
 		}
 		viewModel.isLoading.observe(viewLifecycleOwner, this::onLoadingStateChanged)
 		viewModel.chapters
diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt b/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt
index e32516fe2..f8bff285f 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt
@@ -75,7 +75,7 @@ class ChaptersSheet : BaseAdaptiveSheet<SheetChaptersBinding>(),
 			-1
 		}
 		binding.recyclerView.addItemDecoration(TypedListSpacingDecoration(binding.recyclerView.context, true))
-		binding.recyclerView.adapter = ChaptersAdapter(this).also { adapter ->
+		binding.recyclerView.adapter = ChaptersAdapter(this, false).also { adapter ->
 			if (currentPosition >= 0) {
 				val targetPosition = (currentPosition - 1).coerceAtLeast(0)
 				val offset =
diff --git a/app/src/main/res/layout/item_chapter_grid.xml b/app/src/main/res/layout/item_chapter_grid.xml
index be845c63d..670de9ff8 100644
--- a/app/src/main/res/layout/item_chapter_grid.xml
+++ b/app/src/main/res/layout/item_chapter_grid.xml
@@ -9,9 +9,9 @@
 	<com.google.android.material.card.MaterialCardView
 		android:layout_width="0dp"
 		android:layout_height="0dp"
+		android:layout_margin="6dp"
 		android:clipChildren="false"
 		app:cardCornerRadius="16dp"
-		app:cardUseCompatPadding="true"
 		app:layout_constraintBottom_toBottomOf="parent"
 		app:layout_constraintDimensionRatio="H,1:1"
 		app:layout_constraintEnd_toEndOf="parent"

From c2e30b300992fdd3a60ae93cb64965cb70487918 Mon Sep 17 00:00:00 2001
From: jsericksk <jsericksk@gmail.com>
Date: Sun, 18 Feb 2024 23:20:43 -0300
Subject: [PATCH 4/5] Update changes

---
 app/src/main/res/values/strings.xml | 1 +
 1 file changed, 1 insertion(+)

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 639bfbc4e..080711eff 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -173,6 +173,7 @@
 	<string name="text_clear_updates_feed_prompt">Clear all update history permanently?</string>
 	<string name="check_for_new_chapters">Check for new chapters</string>
 	<string name="reverse">Reverse</string>
+	<string name="chapters_grid_view">Grid view</string>
 	<string name="sign_in">Sign in</string>
 	<string name="auth_required">Sign in to view this content</string>
 	<string name="default_s">Default: %s</string>

From 81d4a3cf686d631e557b317a556ba8818e91d564 Mon Sep 17 00:00:00 2001
From: jsericksk <jsericksk@gmail.com>
Date: Mon, 19 Feb 2024 17:32:02 -0300
Subject: [PATCH 5/5] Fix grid view in ChaptersSheet

---
 .../org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt    | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt b/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt
index f8bff285f..72a3cecb6 100644
--- a/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt
+++ b/app/src/main/kotlin/org/koitharu/kotatsu/reader/ui/ChaptersSheet.kt
@@ -6,6 +6,8 @@ import android.view.View
 import android.view.ViewGroup
 import androidx.fragment.app.FragmentManager
 import androidx.fragment.app.activityViewModels
+import androidx.recyclerview.widget.GridLayoutManager
+import androidx.recyclerview.widget.LinearLayoutManager
 import dagger.hilt.android.AndroidEntryPoint
 import org.koitharu.kotatsu.R
 import org.koitharu.kotatsu.core.model.MangaHistory
@@ -75,7 +77,8 @@ class ChaptersSheet : BaseAdaptiveSheet<SheetChaptersBinding>(),
 			-1
 		}
 		binding.recyclerView.addItemDecoration(TypedListSpacingDecoration(binding.recyclerView.context, true))
-		binding.recyclerView.adapter = ChaptersAdapter(this, false).also { adapter ->
+		val chaptersInGridView = settings.chaptersGridView
+		binding.recyclerView.adapter = ChaptersAdapter(this, chaptersInGridView).also { adapter ->
 			if (currentPosition >= 0) {
 				val targetPosition = (currentPosition - 1).coerceAtLeast(0)
 				val offset =
@@ -87,6 +90,11 @@ class ChaptersSheet : BaseAdaptiveSheet<SheetChaptersBinding>(),
 				adapter.items = chapters
 			}
 		}
+		binding.recyclerView.layoutManager = if (chaptersInGridView) {
+			GridLayoutManager(context, 4)
+		} else {
+			LinearLayoutManager(context)
+		}
 	}
 
 	override fun onItemClick(item: ChapterListItem, view: View) {