Small visual changes

Author: Dan Oprea

app/src/main/java/com/dan/timelapse/BusyDialog.kt

@@ -8,7 +8,7 @@ import android.view.ViewGroup
 import androidx.fragment.app.DialogFragment
 import com.dan.timelapse.databinding.BusyDialogBinding
 
-class BusyDialog( private var message: String): DialogFragment() {
+class BusyDialog( private var message: String, private var progress: Int, private var total: Int): DialogFragment() {
 
     companion object {
         private const val FRAGMENT_TAG = "busy"
@@ -29,27 +29,15 @@ class BusyDialog( private var message: String): DialogFragment() {
             }
         }
 
-        fun show(message: String) {
+        fun show(message: String, progress: Int = -1, total: Int = -1) {
             runSafe {
                 if (null == currentDialog) {
-                    val dialog = BusyDialog(message)
+                    val dialog = BusyDialog(message, progress, total)
                     dialog.isCancelable = false
                     dialog.show(activity.supportFragmentManager, FRAGMENT_TAG)
                     currentDialog = dialog
                 } else {
-                    currentDialog?.binding?.textBusyMessage?.text = message
-                }
-            }
-        }
-
-        fun updateProgress(progress: Int, total: Int) {
-            if (total <= 0 || progress < 0 || progress > total) return
-
-            runSafe {
-                currentDialog?.binding?.progressBar?.apply {
-                    this.max = total
-                    this.progress = progress
-                    this.isIndeterminate = false
+                    currentDialog?.update(message, progress, total)
                 }
             }
         }
@@ -64,10 +52,34 @@ class BusyDialog( private var message: String): DialogFragment() {
 
     private var binding: BusyDialogBinding? = null
 
+    fun update(message: String, progress: Int = -1, total: Int = -1) {
+        this.message = message
+        this.progress = progress
+        this.total = total
+        update()
+    }
+
+    private fun update() {
+        val infinite = progress < 0 || total <= 0 || progress > total
+        val title = if (progress < 0) message else "$message ($progress)"
+        binding?.let {
+            it.textBusyMessage.text = title
+            if (infinite) {
+                it.progressBar.isIndeterminate = true
+            } else {
+                it.progressBar.progress = 0
+                it.progressBar.max = total
+                it.progressBar.progress = progress
+                it.progressBar.isIndeterminate = false
+            }
+        }
+    }
+
     override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View {
         val binding = BusyDialogBinding.inflate( inflater )
         binding.textBusyMessage.text = message
         this.binding = binding
+        update()
         return binding.root
     }
 }

app/src/main/java/com/dan/timelapse/MainFragment.kt

@@ -20,6 +20,9 @@ class MainFragment(activity: MainActivity) : AppFragment(activity) {
         const val INTENT_OPEN_IMAGES = 3
         const val INTENT_OPEN_FOLDER = 4
 
+        const val TITLE_GENERATE = "Generate"
+        const val TITLE_SAVE = "Save"
+
         fun show(activity: MainActivity) {
             activity.pushView("TimeLapse", MainFragment(activity))
         }
@@ -239,13 +242,13 @@ class MainFragment(activity: MainActivity) : AppFragment(activity) {
         videoStop()
         videoUri = null
         binding.video.setVideoURI(null)
-        runAsync("Generate") {
+        runAsync(TITLE_GENERATE) {
             generateAsync()
         }
     }
 
     private fun handleSave() {
-        runAsync("Save") {
+        runAsync(TITLE_SAVE) {
             saveAsync()
         }
     }
@@ -341,7 +344,7 @@ class MainFragment(activity: MainActivity) : AppFragment(activity) {
 
             frameConsumer.start()
             framesInput.forEachFrame { index, size, frame ->
-                BusyDialog.updateProgress(index, size)
+                BusyDialog.show(TITLE_GENERATE, index, size)
                 frameConsumer.consume(frame)
             }
             frameConsumer.stop()