Merge pull request #4619 from Bnyro/master

fix: chapter layout UI and UX issues
libre-tube · Aug 25, 2023 · eaa0b58 · eaa0b58
2 parents b2e0674 + d03ee80
commit eaa0b58
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 1 deletion.
diff --git a/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt b/app/src/main/java/com/github/libretube/ui/fragments/PlayerFragment.kt
@@ -325,6 +325,7 @@ class PlayerFragment : Fragment(), OnlinePlayerOptions {
                 binding.player.hideController()
                 binding.player.useController = false
                 commentsViewModel.setCommentSheetExpand(false)
+                chaptersBottomSheet?.dismiss()
                 transitionEndId = endId
                 transitionStartId = startId
             }

diff --git a/app/src/main/res/layout/bottom_sheet.xml b/app/src/main/res/layout/bottom_sheet.xml
@@ -28,14 +28,15 @@
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
                 android:orientation="vertical"
-                android:paddingBottom="16dp"
+                android:paddingBottom="12dp"
                 android:visibility="gone">
 
                 <TextView
                     android:id="@+id/bottom_sheet_title"
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
                     android:layout_marginHorizontal="10dp"
+                    android:layout_marginBottom="5dp"
                     android:textSize="27sp" />
 
                 <com.google.android.material.divider.MaterialDivider

diff --git a/app/src/main/res/layout/chapters_row.xml b/app/src/main/res/layout/chapters_row.xml
@@ -5,6 +5,7 @@
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
     android:layout_marginHorizontal="5dp"
+    app:cardElevation="0dp"
     app:cardBackgroundColor="@android:color/transparent"
     app:cardCornerRadius="10dp">