Преглед изворни кода

Combine author and artist fields in manga info

arkon пре 4 година
родитељ
комит
395ca3630c

+ 4 - 25
app/src/main/java/eu/kanade/tachiyomi/ui/manga/info/MangaInfoController.kt

@@ -137,21 +137,6 @@ class MangaInfoController(private val fromSource: Boolean = false) :
             }
             .launchIn(scope)
 
-        binding.mangaArtist.longClicks()
-            .onEach {
-                activity?.copyToClipboard(
-                    binding.mangaArtistLabel.text.toString(),
-                    binding.mangaArtist.text.toString()
-                )
-            }
-            .launchIn(scope)
-
-        binding.mangaArtist.clicks()
-            .onEach {
-                performGlobalSearch(binding.mangaArtist.text.toString())
-            }
-            .launchIn(scope)
-
         binding.mangaAuthor.longClicks()
             .onEach {
                 activity?.copyToClipboard(
@@ -231,18 +216,12 @@ class MangaInfoController(private val fromSource: Boolean = false) :
             manga.title
         }
 
-        // Update artist TextView.
-        binding.mangaArtist.text = if (manga.artist.isNullOrBlank()) {
-            view.context.getString(R.string.unknown)
-        } else {
-            manga.artist
-        }
-
-        // Update author TextView.
-        binding.mangaAuthor.text = if (manga.author.isNullOrBlank()) {
+        // Update author/artist TextView.
+        val authors = listOf(manga.author, manga.artist).filter { !it.isNullOrBlank() }.distinct()
+        binding.mangaAuthor.text = if (authors.isEmpty()) {
             view.context.getString(R.string.unknown)
         } else {
-            manga.author
+            authors.joinToString(", ")
         }
 
         // If manga source is known update source TextView.

+ 4 - 39
app/src/main/res/layout-land/manga_info_controller.xml

@@ -53,61 +53,26 @@
                     app:layout_constraintStart_toStartOf="parent"
                     app:layout_constraintTop_toTopOf="parent" />
 
-                <TextView
-                    android:id="@+id/manga_author_label"
-                    style="@style/TextAppearance.Medium.Body2"
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:text="@string/manga_info_author_label"
-                    android:textIsSelectable="false"
-                    app:layout_constraintStart_toStartOf="parent"
-                    app:layout_constraintTop_toBottomOf="@+id/manga_full_title" />
-
                 <TextView
                     android:id="@+id/manga_author"
                     style="@style/TextAppearance.Regular.Body1.Secondary"
                     android:layout_width="0dp"
                     android:layout_height="wrap_content"
-                    android:layout_marginStart="8dp"
-                    android:ellipsize="end"
-                    android:maxLines="1"
-                    android:textIsSelectable="false"
-                    app:layout_constraintBaseline_toBaselineOf="@+id/manga_author_label"
-                    app:layout_constraintEnd_toEndOf="parent"
-                    app:layout_constraintStart_toEndOf="@+id/manga_author_label" />
-
-                <TextView
-                    android:id="@+id/manga_artist_label"
-                    style="@style/TextAppearance.Medium.Body2"
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:text="@string/manga_info_artist_label"
                     android:textIsSelectable="false"
                     app:layout_constraintStart_toStartOf="parent"
-                    app:layout_constraintTop_toBottomOf="@+id/manga_author_label" />
-
-                <TextView
-                    android:id="@+id/manga_artist"
-                    style="@style/TextAppearance.Regular.Body1.Secondary"
-                    android:layout_width="0dp"
-                    android:layout_height="wrap_content"
-                    android:layout_marginStart="8dp"
-                    android:ellipsize="end"
-                    android:maxLines="1"
-                    android:textIsSelectable="false"
-                    app:layout_constraintBaseline_toBaselineOf="@+id/manga_artist_label"
-                    app:layout_constraintEnd_toEndOf="parent"
-                    app:layout_constraintStart_toEndOf="@+id/manga_artist_label" />
+                    app:layout_constraintTop_toBottomOf="@+id/manga_full_title"
+                    tools:text="Author" />
 
                 <TextView
                     android:id="@+id/manga_chapters_label"
                     style="@style/TextAppearance.Medium.Body2"
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
+                    android:layout_marginTop="8dp"
                     android:text="@string/manga_info_last_chapter_label"
                     android:textIsSelectable="false"
                     app:layout_constraintStart_toStartOf="parent"
-                    app:layout_constraintTop_toBottomOf="@+id/manga_artist_label" />
+                    app:layout_constraintTop_toBottomOf="@+id/manga_author" />
 
                 <TextView
                     android:id="@+id/manga_chapters"

+ 4 - 40
app/src/main/res/layout/manga_info_controller.xml

@@ -70,7 +70,6 @@
                         android:layout_width="wrap_content"
                         android:layout_height="wrap_content"
                         android:maxLines="2"
-                        android:paddingBottom="8dp"
                         android:text="@string/manga_info_full_title_label"
                         android:textIsSelectable="false"
                         app:autoSizeMaxTextSize="20sp"
@@ -80,62 +79,27 @@
                         app:layout_constraintStart_toStartOf="parent"
                         app:layout_constraintTop_toTopOf="parent" />
 
-                    <TextView
-                        android:id="@+id/manga_author_label"
-                        style="@style/TextAppearance.Medium.Body2"
-                        android:layout_width="wrap_content"
-                        android:layout_height="wrap_content"
-                        android:text="@string/manga_info_author_label"
-                        android:textIsSelectable="false"
-
-                        app:layout_constraintStart_toStartOf="parent"
-                        app:layout_constraintTop_toBottomOf="@+id/manga_full_title" />
-
                     <TextView
                         android:id="@+id/manga_author"
                         style="@style/TextAppearance.Regular.Body1.Secondary"
                         android:layout_width="0dp"
                         android:layout_height="wrap_content"
-                        android:layout_marginStart="8dp"
-                        android:ellipsize="end"
-                        android:maxLines="1"
                         android:textIsSelectable="false"
-                        app:layout_constraintBaseline_toBaselineOf="@+id/manga_author_label"
                         app:layout_constraintEnd_toEndOf="parent"
-                        app:layout_constraintStart_toEndOf="@+id/manga_author_label" />
-
-                    <TextView
-                        android:id="@+id/manga_artist_label"
-                        style="@style/TextAppearance.Medium.Body2"
-                        android:layout_width="wrap_content"
-                        android:layout_height="wrap_content"
-                        android:text="@string/manga_info_artist_label"
-                        android:textIsSelectable="false"
                         app:layout_constraintStart_toStartOf="parent"
-                        app:layout_constraintTop_toBottomOf="@+id/manga_author_label" />
-
-                    <TextView
-                        android:id="@+id/manga_artist"
-                        style="@style/TextAppearance.Regular.Body1.Secondary"
-                        android:layout_width="0dp"
-                        android:layout_height="wrap_content"
-                        android:layout_marginStart="8dp"
-                        android:ellipsize="end"
-                        android:maxLines="1"
-                        android:textIsSelectable="false"
-                        app:layout_constraintBaseline_toBaselineOf="@+id/manga_artist_label"
-                        app:layout_constraintEnd_toEndOf="parent"
-                        app:layout_constraintStart_toEndOf="@+id/manga_artist_label" />
+                        app:layout_constraintTop_toBottomOf="@+id/manga_full_title"
+                        tools:text="Author" />
 
                     <TextView
                         android:id="@+id/manga_chapters_label"
                         style="@style/TextAppearance.Medium.Body2"
                         android:layout_width="wrap_content"
                         android:layout_height="wrap_content"
+                        android:layout_marginTop="8dp"
                         android:text="@string/manga_info_last_chapter_label"
                         android:textIsSelectable="false"
                         app:layout_constraintStart_toStartOf="parent"
-                        app:layout_constraintTop_toBottomOf="@+id/manga_artist_label" />
+                        app:layout_constraintTop_toBottomOf="@+id/manga_author" />
 
                     <TextView
                         android:id="@+id/manga_chapters"

+ 0 - 2
app/src/main/res/layout/source_list_item.xml

@@ -26,9 +26,7 @@
         android:layout_width="0dp"
         android:layout_height="wrap_content"
         android:layout_marginStart="8dp"
-        android:layout_marginTop="8dp"
         android:layout_marginEnd="8dp"
-        android:layout_marginBottom="8dp"
         android:ellipsize="end"
         android:maxLines="1"
         app:layout_constraintBottom_toBottomOf="parent"

+ 0 - 2
app/src/main/res/values/strings.xml

@@ -459,8 +459,6 @@
     <string name="manga_info_full_title_label">Title</string>
     <string name="manga_added_library">Added to library</string>
     <string name="manga_removed_library">Removed from library</string>
-    <string name="manga_info_author_label">Author</string>
-    <string name="manga_info_artist_label">Artist</string>
     <string name="manga_info_chapters_label">Chapters</string>
     <string name="manga_info_last_chapter_label">Last chapter</string>
     <string name="manga_info_latest_data_label">Updated</string>