From 42604ac3b619d83c02c6bcf63241e02b4a9217af Mon Sep 17 00:00:00 2001
From: nazar-kutz <vitaminkuna@gmail.com>
Date: Thu, 22 Apr 2021 15:02:10 +0300
Subject: [PATCH] RTL: fix slider on Announcement Time dialog

---
 .../seekbar_progress_announcement_time.xml    | 15 ----
 .../seekbar_thumb_announcement_time.xml       | 13 ---
 .../seekbar_tickmark_announcement_time.xml    |  8 --
 .../layout/bottom_sheet_announcement_time.xml | 14 +--
 .../AnnouncementTimeBottomSheet.java          | 86 ++++++-------------
 5 files changed, 30 insertions(+), 106 deletions(-)
 delete mode 100644 OsmAnd/res/drawable/seekbar_progress_announcement_time.xml
 delete mode 100644 OsmAnd/res/drawable/seekbar_thumb_announcement_time.xml
 delete mode 100644 OsmAnd/res/drawable/seekbar_tickmark_announcement_time.xml

diff --git a/OsmAnd/res/drawable/seekbar_progress_announcement_time.xml b/OsmAnd/res/drawable/seekbar_progress_announcement_time.xml
deleted file mode 100644
index 794539923e..0000000000
--- a/OsmAnd/res/drawable/seekbar_progress_announcement_time.xml
+++ /dev/null
@@ -1,15 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<layer-list xmlns:android="http://schemas.android.com/apk/res/android">
-	<item android:id="@+id/background">
-		<shape android:shape="rectangle">
-			<solid android:color="#4d007eb3" />
-			<corners android:radius="30dp" />
-		</shape>
-	</item>
-	<item android:id="@+id/progress">
-		<shape android:shape="rectangle">
-			<solid android:color="@color/color_white" />
-			<corners android:radius="30dp" />
-		</shape>
-	</item>
-</layer-list>
\ No newline at end of file
diff --git a/OsmAnd/res/drawable/seekbar_thumb_announcement_time.xml b/OsmAnd/res/drawable/seekbar_thumb_announcement_time.xml
deleted file mode 100644
index a126aee222..0000000000
--- a/OsmAnd/res/drawable/seekbar_thumb_announcement_time.xml
+++ /dev/null
@@ -1,13 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<layer-list xmlns:android="http://schemas.android.com/apk/res/android"
-	android:layout_width="match_parent"
-	android:layout_height="match_parent">
-	<item android:id="@+id/thump">
-		<shape android:shape="oval">
-			<size
-				android:width="12dp"
-				android:height="12dp" />
-			<solid android:color="@color/profile_icon_color_blue_light" />
-		</shape>
-	</item>
-</layer-list>
\ No newline at end of file
diff --git a/OsmAnd/res/drawable/seekbar_tickmark_announcement_time.xml b/OsmAnd/res/drawable/seekbar_tickmark_announcement_time.xml
deleted file mode 100644
index d6662da81f..0000000000
--- a/OsmAnd/res/drawable/seekbar_tickmark_announcement_time.xml
+++ /dev/null
@@ -1,8 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<shape xmlns:android="http://schemas.android.com/apk/res/android"
-    android:shape="oval">
-    <size
-        android:width="2dp"
-        android:height="2dp" />
-    <solid android:color="#17181A" />
-</shape>
\ No newline at end of file
diff --git a/OsmAnd/res/layout/bottom_sheet_announcement_time.xml b/OsmAnd/res/layout/bottom_sheet_announcement_time.xml
index 246ffe3476..308ad0278a 100644
--- a/OsmAnd/res/layout/bottom_sheet_announcement_time.xml
+++ b/OsmAnd/res/layout/bottom_sheet_announcement_time.xml
@@ -62,18 +62,12 @@
 		osmand:typeface="@string/font_roboto_medium"
 		tools:text="Normal" />
 
-	<SeekBar
-		android:id="@+id/seek_bar_arrival"
-		style="@style/Widget.AppCompat.SeekBar.Discrete"
+	<com.google.android.material.slider.Slider
+		android:id="@+id/arrival_slider"
 		android:layout_width="match_parent"
 		android:layout_height="wrap_content"
-		android:layout_marginTop="@dimen/pages_item_margin"
-		android:maxHeight="2dp"
-		android:paddingTop="11dp"
-		android:paddingBottom="11dp"
-		osmand:tickMark="@drawable/seekbar_tickmark_announcement_time"
-		tools:max="3"
-		tools:progress="1" />
+		android:layout_marginLeft="@dimen/content_padding"
+		android:layout_marginRight="@dimen/content_padding" />
 
 	<View
 		android:id="@+id/divider"
diff --git a/OsmAnd/src/net/osmand/plus/settings/bottomsheets/AnnouncementTimeBottomSheet.java b/OsmAnd/src/net/osmand/plus/settings/bottomsheets/AnnouncementTimeBottomSheet.java
index bbec0c6211..c697213c30 100644
--- a/OsmAnd/src/net/osmand/plus/settings/bottomsheets/AnnouncementTimeBottomSheet.java
+++ b/OsmAnd/src/net/osmand/plus/settings/bottomsheets/AnnouncementTimeBottomSheet.java
@@ -1,26 +1,24 @@
 package net.osmand.plus.settings.bottomsheets;
 
-import android.graphics.drawable.ClipDrawable;
-import android.graphics.drawable.Drawable;
-import android.graphics.drawable.GradientDrawable;
-import android.graphics.drawable.LayerDrawable;
 import android.os.Bundle;
-import android.view.Gravity;
 import android.view.View;
+import android.view.View.OnClickListener;
 import android.widget.ImageView;
-import android.widget.SeekBar;
 
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
-import androidx.core.content.ContextCompat;
 import androidx.fragment.app.Fragment;
 import androidx.fragment.app.FragmentManager;
 
+import com.google.android.material.slider.Slider;
+import com.google.android.material.slider.Slider.OnChangeListener;
+
 import net.osmand.PlatformUtil;
 import net.osmand.plus.OsmandApplication;
 import net.osmand.plus.R;
 import net.osmand.plus.UiUtilities;
 import net.osmand.plus.base.bottomsheetmenu.BaseBottomSheetItem;
+import net.osmand.plus.base.bottomsheetmenu.BaseBottomSheetItem.Builder;
 import net.osmand.plus.helpers.AndroidUiHelper;
 import net.osmand.plus.routing.data.AnnounceTimeDistances;
 import net.osmand.plus.settings.backend.ApplicationMode;
@@ -33,8 +31,7 @@ import org.apache.commons.logging.Log;
 import static net.osmand.plus.settings.bottomsheets.SingleSelectPreferenceBottomSheet.SELECTED_ENTRY_INDEX_KEY;
 
 
-public class AnnouncementTimeBottomSheet extends BasePreferenceBottomSheet
-		implements SeekBar.OnSeekBarChangeListener {
+public class AnnouncementTimeBottomSheet extends BasePreferenceBottomSheet {
 
 	public static final String TAG = AnnouncementTimeBottomSheet.class.getSimpleName();
 	private static final Log LOG = PlatformUtil.getLog(AnnouncementTimeBottomSheet.class);
@@ -46,7 +43,7 @@ public class AnnouncementTimeBottomSheet extends BasePreferenceBottomSheet
 	private int selectedEntryIndex = -1;
 
 	private TextViewEx tvSeekBarLabel;
-	private SeekBar seekBarArrival;
+	private Slider slider;
 	private ImageView ivArrow;
 	private TextViewEx tvIntervalsDescr;
 
@@ -110,22 +107,6 @@ public class AnnouncementTimeBottomSheet extends BasePreferenceBottomSheet
 		dismiss();
 	}
 
-	@Override
-	public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
-		if (progress != selectedEntryIndex) {
-			selectedEntryIndex = progress;
-			updateViews();
-		}
-	}
-
-	@Override
-	public void onStartTrackingTouch(SeekBar seekBar) {
-	}
-
-	@Override
-	public void onStopTrackingTouch(SeekBar seekBar) {
-	}
-
 	private ListPreferenceEx getListPreference() {
 		return (ListPreferenceEx) getPreference();
 	}
@@ -135,30 +116,40 @@ public class AnnouncementTimeBottomSheet extends BasePreferenceBottomSheet
 				.inflate(R.layout.bottom_sheet_announcement_time, null);
 
 		tvSeekBarLabel = rootView.findViewById(R.id.tv_seek_bar_label);
-		seekBarArrival = rootView.findViewById(R.id.seek_bar_arrival);
+		slider = rootView.findViewById(R.id.arrival_slider);
 		ivArrow = rootView.findViewById(R.id.iv_arrow);
 		tvIntervalsDescr = rootView.findViewById(R.id.tv_interval_descr);
+		int appModeColor = getAppMode().getProfileColor(nightMode);
 
-		setProfileColorToSeekBar();
-		seekBarArrival.setOnSeekBarChangeListener(this);
-		seekBarArrival.setProgress(selectedEntryIndex);
-		seekBarArrival.setMax(listPreference.getEntries().length - 1);
-		rootView.findViewById(R.id.description_container).setOnClickListener(new View.OnClickListener() {
+		slider.setValue(selectedEntryIndex);
+		slider.setValueFrom(0);
+		slider.setValueTo(listPreference.getEntries().length - 1);
+		slider.setStepSize(1);
+		slider.addOnChangeListener(new OnChangeListener() {
+			@Override
+			public void onValueChange(@NonNull Slider slider, float value, boolean fromUser) {
+				int intValue = (int) value;
+				if (intValue != selectedEntryIndex) {
+					selectedEntryIndex = intValue;
+					updateViews();
+				}
+			}
+		});
+		UiUtilities.setupSlider(slider, nightMode, appModeColor, true);
+		rootView.findViewById(R.id.description_container).setOnClickListener(new OnClickListener() {
 			@Override
 			public void onClick(View v) {
 				toggleDescriptionVisibility();
 			}
 		});
 
-		return new BaseBottomSheetItem.Builder()
+		return new Builder()
 				.setCustomView(rootView)
 				.create();
 	}
 
 	private void updateViews() {
-		seekBarArrival.setProgress(selectedEntryIndex);
 		tvSeekBarLabel.setText(listPreference.getEntries()[selectedEntryIndex]);
-
 		float value = (float) listPreference.getEntryValues()[selectedEntryIndex];
 		announceTimeDistances.setArrivalDistances(value);
 		tvIntervalsDescr.setText(announceTimeDistances.getIntervalsDescription(app));
@@ -170,31 +161,6 @@ public class AnnouncementTimeBottomSheet extends BasePreferenceBottomSheet
 		AndroidUiHelper.updateVisibility(tvIntervalsDescr, !collapsed);
 	}
 
-	private void setProfileColorToSeekBar() {
-		int color = getAppMode().getProfileColor(nightMode);
-
-		LayerDrawable seekBarProgressLayer =
-				(LayerDrawable) ContextCompat.getDrawable(app, R.drawable.seekbar_progress_announcement_time);
-
-		GradientDrawable background = (GradientDrawable) seekBarProgressLayer.findDrawableByLayerId(R.id.background);
-		background.setColor(color);
-		background.setAlpha(70);
-
-		GradientDrawable progress = (GradientDrawable) seekBarProgressLayer.findDrawableByLayerId(R.id.progress);
-		progress.setColor(color);
-		Drawable clippedProgress = new ClipDrawable(progress, Gravity.CENTER_VERTICAL | Gravity.START, 1);
-
-		seekBarArrival.setProgressDrawable(new LayerDrawable(new Drawable[] {
-				background, clippedProgress
-		}));
-
-		LayerDrawable seekBarThumpLayer =
-				(LayerDrawable) ContextCompat.getDrawable(app, R.drawable.seekbar_thumb_announcement_time);
-		GradientDrawable thump = (GradientDrawable) seekBarThumpLayer.findDrawableByLayerId(R.id.thump);
-		thump.setColor(color);
-		seekBarArrival.setThumb(thump);
-	}
-
 	public static void showInstance(@NonNull FragmentManager fm, String prefKey, Fragment target,
 									@Nullable ApplicationMode appMode, boolean usedOnMap) {
 		try {