优化

app/src/main/java/io/legado/app/ui/book/info/edit/BookInfoEditActivity.kt

@@ -8,6 +8,7 @@ import android.view.MenuItem
 import androidx.activity.viewModels
 import io.legado.app.R
 import io.legado.app.base.VMBaseActivity
+import io.legado.app.constant.BookType
 import io.legado.app.data.entities.Book
 import io.legado.app.databinding.ActivityBookInfoEditBinding
 import io.legado.app.ui.book.changecover.ChangeCoverDialog
@@ -70,6 +71,13 @@ class BookInfoEditActivity :
     private fun upView(book: Book) = binding.run {
         tieBookName.setText(book.name)
         tieBookAuthor.setText(book.author)
+        spType.setSelection(
+            when (book.type) {
+                BookType.image -> 2
+                BookType.audio -> 1
+                else -> 0
+            }
+        )
         tieCoverUrl.setText(book.getDisplayCover())
         tieBookIntro.setText(book.getDisplayIntro())
         upCover()
@@ -85,6 +93,11 @@ class BookInfoEditActivity :
         viewModel.book?.let { book ->
             book.name = tieBookName.text?.toString() ?: ""
             book.author = tieBookAuthor.text?.toString() ?: ""
+            book.type = when (spType.selectedItemPosition) {
+                2 -> BookType.image
+                1 -> BookType.audio
+                else -> BookType.default
+            }
             val customCoverUrl = tieCoverUrl.text?.toString()
             book.customCoverUrl = if (customCoverUrl == book.coverUrl) null else customCoverUrl
             book.customIntro = tieBookIntro.text?.toString()

app/src/main/res/layout/activity_book_info_edit.xml

@@ -12,7 +12,56 @@
         android:layout_height="wrap_content"
         app:title="@string/book_info_edit" />
 
-    <ScrollView
+    <LinearLayout
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:orientation="horizontal"
+        android:padding="5dp">
+
+        <io.legado.app.ui.widget.image.CoverImageView
+            android:id="@+id/iv_cover"
+            android:layout_width="90dp"
+            android:layout_height="126dp"
+            android:contentDescription="@string/img_cover"
+            android:scaleType="centerCrop"
+            android:src="@drawable/image_cover_default" />
+
+        <LinearLayout
+            android:layout_width="match_parent"
+            android:layout_height="match_parent"
+            android:orientation="vertical"
+            android:padding="5dp">
+
+            <io.legado.app.ui.widget.text.TextInputLayout
+                android:id="@+id/til_book_name"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:hint="@string/book_name">
+
+                <io.legado.app.lib.theme.view.ThemeEditText
+                    android:id="@+id/tie_book_name"
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:singleLine="true" />
+            </io.legado.app.ui.widget.text.TextInputLayout>
+
+            <io.legado.app.ui.widget.text.TextInputLayout
+                android:id="@+id/til_book_author"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:hint="@string/author">
+
+                <io.legado.app.lib.theme.view.ThemeEditText
+                    android:id="@+id/tie_book_author"
+                    android:layout_width="match_parent"
+                    android:layout_height="wrap_content"
+                    android:singleLine="true" />
+            </io.legado.app.ui.widget.text.TextInputLayout>
+
+        </LinearLayout>
+    </LinearLayout>
+
+    <androidx.core.widget.NestedScrollView
         android:layout_width="match_parent"
         android:layout_height="wrap_content">
 
@@ -21,54 +70,27 @@
             android:layout_height="wrap_content"
             android:orientation="vertical">
 
-
             <LinearLayout
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
                 android:orientation="horizontal"
-                android:padding="5dp">
+                android:layout_gravity="center_vertical">
+
+                <TextView
+                    android:layout_width="wrap_content"
+                    android:layout_height="wrap_content"
+                    android:text="@string/book_type"
+                    android:layout_marginLeft="12dp"
+                    android:layout_marginRight="3dp"
+                    tools:ignore="RtlHardcoded" />
 
-                <io.legado.app.ui.widget.image.CoverImageView
+                <androidx.appcompat.widget.AppCompatSpinner
-                    android:id="@+id/iv_cover"
+                    android:id="@+id/sp_type"
-                    android:layout_width="90dp"
+                    android:layout_width="wrap_content"
-                    android:layout_height="126dp"
+                    android:layout_height="wrap_content"
-                    android:contentDescription="@string/img_cover"
+                    android:entries="@array/book_type"
-                    android:scaleType="centerCrop"
+                    app:theme="@style/Spinner" />
-                    android:src="@drawable/image_cover_default" />
 
-                <LinearLayout
-                    android:layout_width="match_parent"
-                    android:layout_height="match_parent"
-                    android:orientation="vertical"
-                    android:padding="5dp">
-
-                    <io.legado.app.ui.widget.text.TextInputLayout
-                        android:id="@+id/til_book_name"
-                        android:layout_width="match_parent"
-                        android:layout_height="wrap_content"
-                        android:hint="@string/book_name">
-
-                        <io.legado.app.lib.theme.view.ThemeEditText
-                            android:id="@+id/tie_book_name"
-                            android:layout_width="match_parent"
-                            android:layout_height="wrap_content"
-                            android:singleLine="true" />
-                    </io.legado.app.ui.widget.text.TextInputLayout>
-
-                    <io.legado.app.ui.widget.text.TextInputLayout
-                        android:id="@+id/til_book_author"
-                        android:layout_width="match_parent"
-                        android:layout_height="wrap_content"
-                        android:hint="@string/author">
-
-                        <io.legado.app.lib.theme.view.ThemeEditText
-                            android:id="@+id/tie_book_author"
-                            android:layout_width="match_parent"
-                            android:layout_height="wrap_content"
-                            android:singleLine="true" />
-                    </io.legado.app.ui.widget.text.TextInputLayout>
-
-                </LinearLayout>
             </LinearLayout>
 
             <io.legado.app.ui.widget.text.TextInputLayout
@@ -132,6 +154,6 @@
             </io.legado.app.ui.widget.text.TextInputLayout>
 
         </LinearLayout>
-    </ScrollView>
+    </androidx.core.widget.NestedScrollView>
 
 </LinearLayout>