fix

WirelessAlien · WirelessAlien · commit fc87d6ec8a51 · 2025-06-17T15:47:26.000+05:30
diff --git a/app/src/main/java/com/wirelessalien/zipxtract/activity/OpenWithActivity.kt b/app/src/main/java/com/wirelessalien/zipxtract/activity/OpenWithActivity.kt
@@ -30,8 +30,10 @@ import android.view.View
 import android.widget.Toast
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.content.ContextCompat
+import com.google.android.material.chip.Chip
 import com.google.android.material.dialog.MaterialAlertDialogBuilder
 import com.wirelessalien.zipxtract.R
+import com.wirelessalien.zipxtract.databinding.DialogArchiveTypeBinding
 import com.wirelessalien.zipxtract.databinding.DialogCrashLogBinding
 import com.wirelessalien.zipxtract.databinding.PasswordInputOpenWithBinding
 import com.wirelessalien.zipxtract.service.ExtractArchiveService
@@ -172,7 +174,7 @@ class OpenWithActivity : AppCompatActivity() {
                 if (filePaths.isNotEmpty()) {
                     showArchiveTypeDialog(filePaths)
                 } else {
-                    Toast.makeText(this@OpenWithActivity, getString(R.string.no_file_selected), Toast.LENGTH_SHORT).show() // Assuming new string
+                    Toast.makeText(this@OpenWithActivity, getString(R.string.no_file_selected), Toast.LENGTH_SHORT).show()
                     finish()
                 }
             }
@@ -186,18 +188,36 @@ class OpenWithActivity : AppCompatActivity() {
             finish()
             return
         }
+        val binding = DialogArchiveTypeBinding.inflate(layoutInflater)
+        val chipGroup = binding.chipGroupArchiveTypes
+
         val archiveTypes = arrayOf("ZIP", "7Z", "TAR")
+        archiveTypes.forEach { type ->
+            val chip = Chip(this).apply {
+                text = type
+                isCheckable = true
+            }
+            chipGroup.addView(chip)
+        }
+
         MaterialAlertDialogBuilder(this, R.style.MaterialDialog)
             .setTitle(getString(R.string.select_archive_type_title))
-            .setItems(archiveTypes) { _, which ->
-                val selectedType = archiveTypes[which]
-                val mainActivityIntent = Intent(this, MainActivity::class.java).apply {
-                    action = MainActivity.ACTION_CREATE_ARCHIVE
-                    putStringArrayListExtra(MainActivity.EXTRA_FILE_PATHS, ArrayList(filePaths))
-                    putExtra(MainActivity.EXTRA_ARCHIVE_TYPE, selectedType)
+            .setView(binding.root)
+            .setPositiveButton(getString(R.string.ok)) { _, _ ->
+                val selectedChipId = chipGroup.checkedChipId
+                if (selectedChipId != View.NO_ID) {
+                    val selectedChip = chipGroup.findViewById<Chip>(selectedChipId)
+                    val selectedType = selectedChip.text.toString()
+                    val mainActivityIntent = Intent(this, MainActivity::class.java).apply {
+                        action = MainActivity.ACTION_CREATE_ARCHIVE
+                        putStringArrayListExtra(MainActivity.EXTRA_FILE_PATHS, ArrayList(filePaths))
+                        putExtra(MainActivity.EXTRA_ARCHIVE_TYPE, selectedType)
+                    }
+                    startActivity(mainActivityIntent)
+                    finish()
+                } else {
+                    Toast.makeText(this, getString(R.string.general_error_msg), Toast.LENGTH_SHORT).show()
                 }
-                startActivity(mainActivityIntent)
-                finish()
             }
             .setOnCancelListener {
                 finish()
diff --git a/app/src/main/java/com/wirelessalien/zipxtract/fragment/MainFragment.kt b/app/src/main/java/com/wirelessalien/zipxtract/fragment/MainFragment.kt
@@ -494,9 +494,9 @@ class MainFragment : Fragment(), FileAdapter.OnItemClickListener, FileAdapter.On
             transaction.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_OPEN)
 
             val newFragment = when (archiveType) {
-                "Zip" -> ZipOptionDialogFragment.newInstance(validFilePaths)
-                "7z" -> SevenZOptionDialogFragment.newInstance(validFilePaths)
-                "Tar" -> TarOptionsDialogFragment.newInstance(validFilePaths)
+                "ZIP" -> ZipOptionDialogFragment.newInstance(validFilePaths)
+                "7Z" -> SevenZOptionDialogFragment.newInstance(validFilePaths)
+                "TAR" -> TarOptionsDialogFragment.newInstance(validFilePaths)
                 else -> null
             }
 
diff --git a/app/src/main/res/layout/dialog_archive_type.xml b/app/src/main/res/layout/dialog_archive_type.xml
@@ -18,13 +18,15 @@
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
     android:orientation="vertical"
     android:padding="16dp">
 
-    <com.google.android.material.progressindicator.LinearProgressIndicator
-        android:id="@+id/progressIndicator"
-        android:layout_width="wrap_content"
+    <com.google.android.material.chip.ChipGroup
+        android:id="@+id/chipGroupArchiveTypes"
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:layout_gravity="center"
-        android:visibility="gone" />
+        app:singleSelection="true"
+        app:chipSpacingHorizontal="8dp"
+        app:chipSpacingVertical="8dp"/>
 </LinearLayout>
