[Android] Fix the previous/next bookmark buttons bound in the video player

Nicolas Pomepuy git at videolan.org
Fri Feb 28 12:32:39 UTC 2025


vlc-android | branch: master | Nicolas Pomepuy <nicolas at videolabs.io> | Wed Feb 19 11:08:26 2025 +0100| [30ccaaf90db0e096644c957f026d2c32c7d65e80] | committer: Duncan McNamara

Fix the previous/next bookmark buttons bound in the video player

> https://code.videolan.org/videolan/vlc-android/commit/30ccaaf90db0e096644c957f026d2c32c7d65e80
---

 application/vlc-android/res/layout/bookmarks.xml                        | 2 ++
 .../src/org/videolan/vlc/gui/video/VideoPlayerOverlayDelegate.kt        | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/application/vlc-android/res/layout/bookmarks.xml b/application/vlc-android/res/layout/bookmarks.xml
index 16cd32f602..a5cd213f40 100644
--- a/application/vlc-android/res/layout/bookmarks.xml
+++ b/application/vlc-android/res/layout/bookmarks.xml
@@ -124,6 +124,7 @@
             android:layout_height="wrap_content"
             android:layout_marginStart="16dp"
             android:layout_marginBottom="8dp"
+            android:nextFocusRight="@+id/next_bookmark"
             android:drawableLeft="@drawable/ic_bookmark_previous"
             android:text="@string/previous"
             app:layout_constraintBottom_toTopOf="@+id/progressbar_guideline"
@@ -135,6 +136,7 @@
             style="@style/Widget.MaterialComponents.Button.TextButton.Dialog"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
+            android:nextFocusLeft="@+id/previous_bookmark"
             android:layout_marginEnd="16dp"
             android:layout_marginBottom="8dp"
             android:drawableRight="@drawable/ic_bookmark_next"
diff --git a/application/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerOverlayDelegate.kt b/application/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerOverlayDelegate.kt
index 2d310e645d..f207204012 100644
--- a/application/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerOverlayDelegate.kt
+++ b/application/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerOverlayDelegate.kt
@@ -1097,7 +1097,7 @@ class VideoPlayerOverlayDelegate (private val player: VideoPlayerActivity) {
                 }
             }
             bookmarkListDelegate?.show()
-            val top = hudBinding.playerOverlaySeekbar.top
+            val top = hudBinding.playerOverlayTime.top
             bookmarkListDelegate?.setProgressHeight((top + 12.dp).toFloat())
         }
     }



More information about the Android mailing list