playlist options to bottom sheet

app/src/main/java/com/github/libretube/adapters/PlaylistsAdapter.kt

@@ -9,7 +9,7 @@ import androidx.recyclerview.widget.RecyclerView
 import com.github.libretube.R
 import com.github.libretube.api.RetrofitInstance
 import com.github.libretube.databinding.PlaylistsRowBinding
-import com.github.libretube.sheets.PlaylistOptionsDialog
+import com.github.libretube.sheets.PlaylistOptionsBottomSheet
 import com.github.libretube.extensions.TAG
 import com.github.libretube.obj.PlaylistId
 import com.github.libretube.obj.Playlists
@@ -71,13 +71,13 @@ class PlaylistsAdapter(
             }
 
             root.setOnLongClickListener {
-                val playlistOptionsDialog = PlaylistOptionsDialog(
+                val playlistOptionsDialog = PlaylistOptionsBottomSheet(
                     playlistId = playlist.id!!,
                     isOwner = true
                 )
                 playlistOptionsDialog.show(
                     childFragmentManager,
-                    PlaylistOptionsDialog::class.java.name
+                    PlaylistOptionsBottomSheet::class.java.name
                 )
                 true
             }

app/src/main/java/com/github/libretube/adapters/SearchAdapter.kt

@@ -10,7 +10,7 @@ import com.github.libretube.api.SubscriptionHelper
 import com.github.libretube.databinding.ChannelRowBinding
 import com.github.libretube.databinding.PlaylistSearchRowBinding
 import com.github.libretube.databinding.VideoRowBinding
-import com.github.libretube.sheets.PlaylistOptionsDialog
+import com.github.libretube.sheets.PlaylistOptionsBottomSheet
 import com.github.libretube.sheets.VideoOptionsBottomSheet
 import com.github.libretube.extensions.formatShort
 import com.github.libretube.extensions.setFormattedDuration
@@ -172,8 +172,8 @@ class SearchAdapter(
             }
             root.setOnLongClickListener {
                 val playlistId = item.url!!.toID()
-                PlaylistOptionsDialog(playlistId, false)
+                PlaylistOptionsBottomSheet(playlistId, false)
-                    .show(childFragmentManager, PlaylistOptionsDialog::class.java.name)
+                    .show(childFragmentManager, PlaylistOptionsBottomSheet::class.java.name)
                 true
             }
         }

app/src/main/java/com/github/libretube/fragments/PlaylistFragment.kt

@@ -14,7 +14,7 @@ import com.github.libretube.adapters.PlaylistAdapter
 import com.github.libretube.api.RetrofitInstance
 import com.github.libretube.constants.IntentData
 import com.github.libretube.databinding.FragmentPlaylistBinding
-import com.github.libretube.sheets.PlaylistOptionsDialog
+import com.github.libretube.sheets.PlaylistOptionsBottomSheet
 import com.github.libretube.extensions.BaseFragment
 import com.github.libretube.extensions.TAG
 import com.github.libretube.extensions.toID
@@ -86,10 +86,10 @@ class PlaylistFragment : BaseFragment() {
                 // show playlist options
                 binding.optionsMenu.setOnClickListener {
                     val optionsDialog =
-                        PlaylistOptionsDialog(playlistId!!, isOwner)
+                        PlaylistOptionsBottomSheet(playlistId!!, isOwner)
                     optionsDialog.show(
                         childFragmentManager,
-                        PlaylistOptionsDialog::class.java.name
+                        PlaylistOptionsBottomSheet::class.java.name
                     )
                 }
 

app/src/main/java/com/github/libretube/sheets/PlaylistOptionsBottomSheet.kt

@@ -0,0 +1,156 @@
+package com.github.libretube.sheets
+
+import android.os.Bundle
+import android.text.InputType
+import android.util.Log
+import android.widget.Toast
+import com.github.libretube.R
+import com.github.libretube.api.RetrofitInstance
+import com.github.libretube.databinding.DialogTextPreferenceBinding
+import com.github.libretube.dialogs.ShareDialog
+import com.github.libretube.extensions.TAG
+import com.github.libretube.extensions.toID
+import com.github.libretube.obj.PlaylistId
+import com.github.libretube.util.BackgroundHelper
+import com.github.libretube.util.PreferenceHelper
+import com.github.libretube.views.BottomSheet
+import com.google.android.material.dialog.MaterialAlertDialogBuilder
+import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.runBlocking
+import retrofit2.HttpException
+import java.io.IOException
+
+class PlaylistOptionsBottomSheet(
+    private val playlistId: String,
+    private val isOwner: Boolean
+) : BottomSheet() {
+
+    override fun onCreate(savedInstanceState: Bundle?) {
+        // options for the dialog
+        var optionsList = listOf(
+            context?.getString(R.string.playOnBackground)!!,
+            context?.getString(R.string.clonePlaylist)!!,
+            context?.getString(R.string.share)!!
+        )
+
+        if (isOwner) {
+            optionsList = optionsList +
+                context?.getString(R.string.renamePlaylist)!! +
+                context?.getString(R.string.deletePlaylist)!! -
+                context?.getString(R.string.clonePlaylist)!!
+        }
+
+        setSimpleItems(optionsList) { which ->
+            when (optionsList[which]) {
+                // play the playlist in the background
+                context?.getString(R.string.playOnBackground) -> {
+                    runBlocking {
+                        val playlist =
+                            if (isOwner) {
+                                RetrofitInstance.authApi.getPlaylist(playlistId)
+                            } else {
+                                RetrofitInstance.api.getPlaylist(playlistId)
+                            }
+                        BackgroundHelper.playOnBackground(
+                            context = requireContext(),
+                            videoId = playlist.relatedStreams!![0].url!!.toID(),
+                            playlistId = playlistId
+                        )
+                    }
+                }
+                // Clone the playlist to the users Piped account
+                context?.getString(R.string.clonePlaylist) -> {
+                    val token = PreferenceHelper.getToken()
+                    if (token != "") {
+                        importPlaylist(token, playlistId)
+                    } else {
+                        Toast.makeText(
+                            context,
+                            R.string.login_first,
+                            Toast.LENGTH_SHORT
+                        ).show()
+                    }
+                }
+                // share the playlist
+                context?.getString(R.string.share) -> {
+                    val shareDialog = ShareDialog(playlistId, true)
+                    // using parentFragmentManager, childFragmentManager doesn't work here
+                    shareDialog.show(parentFragmentManager, ShareDialog::class.java.name)
+                }
+                context?.getString(R.string.deletePlaylist) -> {
+                    deletePlaylist(
+                        playlistId
+                    )
+                }
+                context?.getString(R.string.renamePlaylist) -> {
+                    val binding = DialogTextPreferenceBinding.inflate(layoutInflater)
+                    binding.input.hint = context?.getString(R.string.playlistName)
+                    binding.input.inputType = InputType.TYPE_CLASS_TEXT
+
+                    MaterialAlertDialogBuilder(requireContext())
+                        .setTitle(R.string.renamePlaylist)
+                        .setView(binding.root)
+                        .setPositiveButton(R.string.okay) { _, _ ->
+                            if (binding.input.text.toString() == "") {
+                                Toast.makeText(
+                                    context,
+                                    R.string.emptyPlaylistName,
+                                    Toast.LENGTH_SHORT
+                                ).show()
+                                return@setPositiveButton
+                            }
+                            renamePlaylist(playlistId, binding.input.text.toString())
+                        }
+                        .setNegativeButton(R.string.cancel, null)
+                        .show()
+                }
+            }
+        }
+        super.onCreate(savedInstanceState)
+    }
+
+    private fun importPlaylist(token: String, playlistId: String) {
+        CoroutineScope(Dispatchers.IO).launch {
+            val response = try {
+                RetrofitInstance.authApi.importPlaylist(token, PlaylistId(playlistId))
+            } catch (e: IOException) {
+                println(e)
+                return@launch
+            } catch (e: HttpException) {
+                return@launch
+            }
+            Log.e(TAG(), response.toString())
+        }
+    }
+
+    private fun renamePlaylist(id: String, newName: String) {
+        CoroutineScope(Dispatchers.IO).launch {
+            try {
+                RetrofitInstance.authApi.renamePlaylist(
+                    PreferenceHelper.getToken(),
+                    PlaylistId(
+                        playlistId = id,
+                        newName = newName
+                    )
+                )
+            } catch (e: Exception) {
+                return@launch
+            }
+        }
+    }
+
+    private fun deletePlaylist(id: String) {
+        CoroutineScope(Dispatchers.IO).launch {
+            try {
+                RetrofitInstance.authApi.deletePlaylist(
+                    PreferenceHelper.getToken(),
+                    PlaylistId(id)
+                )
+            } catch (e: Exception) {
+                return@launch
+            }
+        }
+    }
+}

app/src/main/java/com/github/libretube/sheets/PlaylistOptionsDialog.kt

@@ -1,168 +0,0 @@
-package com.github.libretube.sheets
-
-import android.app.Dialog
-import android.os.Bundle
-import android.text.InputType
-import android.util.Log
-import android.widget.ArrayAdapter
-import android.widget.Toast
-import androidx.fragment.app.DialogFragment
-import com.github.libretube.R
-import com.github.libretube.api.RetrofitInstance
-import com.github.libretube.databinding.DialogTextPreferenceBinding
-import com.github.libretube.dialogs.ShareDialog
-import com.github.libretube.extensions.TAG
-import com.github.libretube.extensions.toID
-import com.github.libretube.obj.PlaylistId
-import com.github.libretube.util.BackgroundHelper
-import com.github.libretube.util.PreferenceHelper
-import com.google.android.material.dialog.MaterialAlertDialogBuilder
-import kotlinx.coroutines.CoroutineScope
-import kotlinx.coroutines.Dispatchers
-import kotlinx.coroutines.launch
-import kotlinx.coroutines.runBlocking
-import retrofit2.HttpException
-import java.io.IOException
-
-class PlaylistOptionsDialog(
-    private val playlistId: String,
-    private val isOwner: Boolean
-) : DialogFragment() {
-
-    override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {
-        // options for the dialog
-        var optionsList = listOf(
-            context?.getString(R.string.playOnBackground),
-            context?.getString(R.string.clonePlaylist),
-            context?.getString(R.string.share)
-        )
-
-        if (isOwner) {
-            optionsList = optionsList +
-                context?.getString(R.string.renamePlaylist)!! +
-                context?.getString(R.string.deletePlaylist)!! -
-                context?.getString(R.string.clonePlaylist)!!
-        }
-
-        val dialog = MaterialAlertDialogBuilder(requireContext())
-            .setNegativeButton(R.string.cancel) { dialog, _ ->
-                dialog.dismiss()
-            }
-            .setAdapter(
-                ArrayAdapter(
-                    requireContext(),
-                    R.layout.video_options_dialog_item,
-                    optionsList
-                )
-            ) { _, which ->
-                when (optionsList[which]) {
-                    // play the playlist in the background
-                    context?.getString(R.string.playOnBackground) -> {
-                        runBlocking {
-                            val playlist =
-                                if (isOwner) {
-                                    RetrofitInstance.authApi.getPlaylist(playlistId)
-                                } else {
-                                    RetrofitInstance.api.getPlaylist(playlistId)
-                                }
-                            BackgroundHelper.playOnBackground(
-                                context = requireContext(),
-                                videoId = playlist.relatedStreams!![0].url!!.toID(),
-                                playlistId = playlistId
-                            )
-                        }
-                    }
-                    // Clone the playlist to the users Piped account
-                    context?.getString(R.string.clonePlaylist) -> {
-                        val token = PreferenceHelper.getToken()
-                        if (token != "") {
-                            importPlaylist(token, playlistId)
-                        } else {
-                            Toast.makeText(
-                                context,
-                                R.string.login_first,
-                                Toast.LENGTH_SHORT
-                            ).show()
-                        }
-                    }
-                    // share the playlist
-                    context?.getString(R.string.share) -> {
-                        val shareDialog = ShareDialog(playlistId, true)
-                        // using parentFragmentManager, childFragmentManager doesn't work here
-                        shareDialog.show(parentFragmentManager, ShareDialog::class.java.name)
-                    }
-                    context?.getString(R.string.deletePlaylist) -> {
-                        deletePlaylist(
-                            playlistId
-                        )
-                    }
-                    context?.getString(R.string.renamePlaylist) -> {
-                        val binding = DialogTextPreferenceBinding.inflate(layoutInflater)
-                        binding.input.hint = context?.getString(R.string.playlistName)
-                        binding.input.inputType = InputType.TYPE_CLASS_TEXT
-
-                        MaterialAlertDialogBuilder(requireContext())
-                            .setTitle(R.string.renamePlaylist)
-                            .setView(binding.root)
-                            .setPositiveButton(R.string.okay) { _, _ ->
-                                if (binding.input.text.toString() == "") {
-                                    Toast.makeText(
-                                        context,
-                                        R.string.emptyPlaylistName,
-                                        Toast.LENGTH_SHORT
-                                    ).show()
-                                    return@setPositiveButton
-                                }
-                                renamePlaylist(playlistId, binding.input.text.toString())
-                            }
-                            .setNegativeButton(R.string.cancel, null)
-                            .show()
-                    }
-                }
-            }
-        return dialog.show()
-    }
-
-    private fun importPlaylist(token: String, playlistId: String) {
-        CoroutineScope(Dispatchers.IO).launch {
-            val response = try {
-                RetrofitInstance.authApi.importPlaylist(token, PlaylistId(playlistId))
-            } catch (e: IOException) {
-                println(e)
-                return@launch
-            } catch (e: HttpException) {
-                return@launch
-            }
-            Log.e(TAG(), response.toString())
-        }
-    }
-
-    private fun renamePlaylist(id: String, newName: String) {
-        CoroutineScope(Dispatchers.IO).launch {
-            try {
-                RetrofitInstance.authApi.renamePlaylist(
-                    PreferenceHelper.getToken(),
-                    PlaylistId(
-                        playlistId = id,
-                        newName = newName
-                    )
-                )
-            } catch (e: Exception) {
-                return@launch
-            }
-        }
-    }
-
-    private fun deletePlaylist(id: String) {
-        CoroutineScope(Dispatchers.IO).launch {
-            try {
-                RetrofitInstance.authApi.deletePlaylist(
-                    PreferenceHelper.getToken(),
-                    PlaylistId(id)
-                )
-            } catch (e: Exception) {
-                return@launch
-            }
-        }
-    }
-}