From 0a8d677fe8926a3efd6d974b1f1b44913d6d4022 Mon Sep 17 00:00:00 2001
From: Koitharu <nvasya95@gmail.com>
Date: Sat, 9 Oct 2021 15:44:57 +0300
Subject: [PATCH] Search by author name

---
 .../kotatsu/details/ui/DetailsFragment.kt       | 17 ++++++++++++++---
 .../res/layout-w600dp-land/fragment_details.xml |  7 +++++--
 .../res/layout-w600dp-port/fragment_details.xml |  7 +++++--
 app/src/main/res/layout/fragment_details.xml    |  8 ++++++--
 4 files changed, 30 insertions(+), 9 deletions(-)

diff --git a/app/src/main/java/org/koitharu/kotatsu/details/ui/DetailsFragment.kt b/app/src/main/java/org/koitharu/kotatsu/details/ui/DetailsFragment.kt
index bb6e7dcc2..9e6bd5533 100644
--- a/app/src/main/java/org/koitharu/kotatsu/details/ui/DetailsFragment.kt
+++ b/app/src/main/java/org/koitharu/kotatsu/details/ui/DetailsFragment.kt
@@ -29,6 +29,7 @@ import org.koitharu.kotatsu.databinding.FragmentDetailsBinding
 import org.koitharu.kotatsu.favourites.ui.categories.select.FavouriteCategoriesDialog
 import org.koitharu.kotatsu.reader.ui.ReaderActivity
 import org.koitharu.kotatsu.reader.ui.ReaderState
+import org.koitharu.kotatsu.search.ui.SearchActivity
 import org.koitharu.kotatsu.utils.FileSizeUtils
 import org.koitharu.kotatsu.utils.ext.*
 
@@ -45,6 +46,10 @@ class DetailsFragment : BaseFragment<FragmentDetailsBinding>(), View.OnClickList
 
 	override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
 		super.onViewCreated(view, savedInstanceState)
+		binding.textViewAuthor.setOnClickListener(this)
+		binding.buttonFavorite.setOnClickListener(this)
+		binding.buttonRead.setOnClickListener(this)
+		binding.buttonRead.setOnLongClickListener(this)
 		viewModel.manga.observe(viewLifecycleOwner, ::onMangaUpdated)
 		viewModel.isLoading.observe(viewLifecycleOwner, ::onLoadingStateChanged)
 		viewModel.favouriteCategories.observe(viewLifecycleOwner, ::onFavouriteChanged)
@@ -118,9 +123,6 @@ class DetailsFragment : BaseFragment<FragmentDetailsBinding>(), View.OnClickList
 			}
 
 			// Buttons
-			buttonFavorite.setOnClickListener(this@DetailsFragment)
-			buttonRead.setOnClickListener(this@DetailsFragment)
-			buttonRead.setOnLongClickListener(this@DetailsFragment)
 			buttonRead.isEnabled = !manga.chapters.isNullOrEmpty()
 
 			// Chips
@@ -178,6 +180,15 @@ class DetailsFragment : BaseFragment<FragmentDetailsBinding>(), View.OnClickList
 					)
 				}
 			}
+			R.id.textView_author -> {
+				startActivity(
+					SearchActivity.newIntent(
+						context = v.context,
+						source = manga.source,
+						query = manga.author ?: return,
+					)
+				)
+			}
 		}
 	}
 
diff --git a/app/src/main/res/layout-w600dp-land/fragment_details.xml b/app/src/main/res/layout-w600dp-land/fragment_details.xml
index c8bdefeb2..4ce68a753 100644
--- a/app/src/main/res/layout-w600dp-land/fragment_details.xml
+++ b/app/src/main/res/layout-w600dp-land/fragment_details.xml
@@ -5,8 +5,8 @@
 	xmlns:tools="http://schemas.android.com/tools"
 	android:layout_width="match_parent"
 	android:layout_height="match_parent"
-	android:scrollbars="vertical"
 	android:clipToPadding="false"
+	android:scrollbars="vertical"
 	app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
 	<androidx.constraintlayout.widget.ConstraintLayout
@@ -37,7 +37,7 @@
 				<org.koitharu.kotatsu.base.ui.widgets.CoverImageView
 					android:id="@+id/imageView_cover"
 					android:layout_width="match_parent"
-					android:layout_height="match_parent"
+					android:layout_height="wrap_content"
 					android:scaleType="centerCrop"
 					tools:background="@tools:sample/backgrounds/scenic"
 					tools:ignore="ContentDescription" />
@@ -81,12 +81,15 @@
 					android:layout_width="0dp"
 					android:layout_height="wrap_content"
 					android:layout_marginTop="4dp"
+					android:background="@drawable/list_selector"
 					android:requiresFadingEdge="horizontal"
 					android:textColor="?colorAccent"
 					android:textStyle="bold"
 					app:layout_constraintEnd_toEndOf="@id/textView_title"
+					app:layout_constraintHorizontal_bias="0"
 					app:layout_constraintStart_toStartOf="@id/textView_title"
 					app:layout_constraintTop_toBottomOf="@id/textView_subtitle"
+					app:layout_constraintWidth_default="wrap"
 					tools:text="@tools:sample/full_names" />
 
 				<TextView
diff --git a/app/src/main/res/layout-w600dp-port/fragment_details.xml b/app/src/main/res/layout-w600dp-port/fragment_details.xml
index d23a53352..5a7eb1a1e 100644
--- a/app/src/main/res/layout-w600dp-port/fragment_details.xml
+++ b/app/src/main/res/layout-w600dp-port/fragment_details.xml
@@ -5,8 +5,8 @@
 	xmlns:tools="http://schemas.android.com/tools"
 	android:layout_width="match_parent"
 	android:layout_height="match_parent"
-	android:scrollbars="vertical"
 	android:clipToPadding="false"
+	android:scrollbars="vertical"
 	app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
 	<androidx.constraintlayout.widget.ConstraintLayout
@@ -37,7 +37,7 @@
 				<org.koitharu.kotatsu.base.ui.widgets.CoverImageView
 					android:id="@+id/imageView_cover"
 					android:layout_width="match_parent"
-					android:layout_height="match_parent"
+					android:layout_height="wrap_content"
 					android:scaleType="centerCrop"
 					tools:background="@tools:sample/backgrounds/scenic"
 					tools:ignore="ContentDescription" />
@@ -82,12 +82,15 @@
 					android:layout_width="0dp"
 					android:layout_height="wrap_content"
 					android:layout_marginTop="4dp"
+					android:background="@drawable/list_selector"
 					android:requiresFadingEdge="horizontal"
 					android:textColor="?colorAccent"
 					android:textStyle="bold"
 					app:layout_constraintEnd_toEndOf="@id/textView_title"
+					app:layout_constraintHorizontal_bias="0"
 					app:layout_constraintStart_toStartOf="@id/textView_title"
 					app:layout_constraintTop_toBottomOf="@id/textView_subtitle"
+					app:layout_constraintWidth_default="wrap"
 					tools:text="@tools:sample/full_names" />
 
 				<TextView
diff --git a/app/src/main/res/layout/fragment_details.xml b/app/src/main/res/layout/fragment_details.xml
index e3a49813b..88c3d4950 100644
--- a/app/src/main/res/layout/fragment_details.xml
+++ b/app/src/main/res/layout/fragment_details.xml
@@ -5,8 +5,8 @@
 	xmlns:tools="http://schemas.android.com/tools"
 	android:layout_width="match_parent"
 	android:layout_height="match_parent"
-	android:scrollbars="vertical"
 	android:clipToPadding="false"
+	android:scrollbars="vertical"
 	app:layout_behavior="@string/appbar_scrolling_view_behavior">
 
 	<androidx.constraintlayout.widget.ConstraintLayout
@@ -37,7 +37,8 @@
 				<org.koitharu.kotatsu.base.ui.widgets.CoverImageView
 					android:id="@+id/imageView_cover"
 					android:layout_width="match_parent"
-					android:layout_height="match_parent"
+					android:layout_height="wrap_content"
+					android:orientation="horizontal"
 					android:scaleType="centerCrop"
 					tools:background="@tools:sample/backgrounds/scenic"
 					tools:ignore="ContentDescription" />
@@ -81,12 +82,15 @@
 					android:layout_width="0dp"
 					android:layout_height="wrap_content"
 					android:layout_marginTop="4dp"
+					android:background="@drawable/list_selector"
 					android:requiresFadingEdge="horizontal"
 					android:textColor="?colorAccent"
 					android:textStyle="bold"
 					app:layout_constraintEnd_toEndOf="@id/textView_title"
+					app:layout_constraintHorizontal_bias="0"
 					app:layout_constraintStart_toStartOf="@id/textView_title"
 					app:layout_constraintTop_toBottomOf="@id/textView_subtitle"
+					app:layout_constraintWidth_default="wrap"
 					tools:text="@tools:sample/full_names" />
 
 				<TextView