From ea6483df74dc58e63786e463c23165efe406a63c Mon Sep 17 00:00:00 2001 From: FineFindus Date: Thu, 30 Jan 2025 16:09:23 +0100 Subject: [PATCH] fix(MarkableTimeBar): align segments to timebar Due to a error in the timebar position calculation, segments could be msialigned by a pixel. This is fixed by using a caculation that more closely matches how the actual timebar is drawn. --- .../java/com/github/libretube/ui/views/MarkableTimeBar.kt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/com/github/libretube/ui/views/MarkableTimeBar.kt b/app/src/main/java/com/github/libretube/ui/views/MarkableTimeBar.kt index 4b103b0a1..ad0a9a6c1 100644 --- a/app/src/main/java/com/github/libretube/ui/views/MarkableTimeBar.kt +++ b/app/src/main/java/com/github/libretube/ui/views/MarkableTimeBar.kt @@ -38,7 +38,7 @@ class MarkableTimeBar( canvas.save() val horizontalOffset = (parent as View).marginLeft length = canvas.width - horizontalOffset * 2 - val marginY = canvas.height / 2 - progressBarHeight / 2 + val marginY = (canvas.height - progressBarHeight) / 2 val themeColor = ThemeHelper.getThemeColor(context, R.attr.colorOnSecondary) segments.forEach { @@ -49,7 +49,7 @@ class MarkableTimeBar( start.toLength() + horizontalOffset, marginY, end.toLength() + horizontalOffset, - canvas.height - marginY + marginY + progressBarHeight ), Paint().apply { color = if (PreferenceHelper.getBoolean("sb_enable_custom_colors", false)) {