diff --git a/app/src/main/java/org/mozilla/fenix/settings/logins/SavedLoginSiteInfoFragment.kt b/app/src/main/java/org/mozilla/fenix/settings/logins/SavedLoginSiteInfoFragment.kt index 1e4e529d4..767576dc8 100644 --- a/app/src/main/java/org/mozilla/fenix/settings/logins/SavedLoginSiteInfoFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/settings/logins/SavedLoginSiteInfoFragment.kt @@ -5,6 +5,7 @@ package org.mozilla.fenix.settings.logins import android.content.Context +import android.content.DialogInterface import android.os.Bundle import android.text.InputType import android.view.Menu @@ -13,6 +14,7 @@ import android.view.MenuItem import android.view.View import android.view.WindowManager import androidx.annotation.StringRes +import androidx.appcompat.app.AlertDialog import androidx.appcompat.content.res.AppCompatResources.getDrawable import androidx.fragment.app.Fragment import androidx.lifecycle.lifecycleScope @@ -84,7 +86,7 @@ class SavedLoginSiteInfoFragment : Fragment(R.layout.fragment_saved_login_site_i override fun onOptionsItemSelected(item: MenuItem): Boolean = when (item.itemId) { R.id.delete_login_button -> { - deleteLogin() + displayDeleteLoginDialog() true } else -> false @@ -145,6 +147,22 @@ class SavedLoginSiteInfoFragment : Fragment(R.layout.fragment_saved_login_site_i showToolbar(args.savedLoginItem.url) } + private fun displayDeleteLoginDialog() { + activity?.let { activity -> + AlertDialog.Builder(activity).apply { + setMessage(R.string.login_deletion_confirmation) + setNegativeButton(android.R.string.cancel) { dialog: DialogInterface, _ -> + dialog.cancel() + } + setPositiveButton(R.string.dialog_delete_positive) { dialog: DialogInterface, _ -> + deleteLogin() + dialog.dismiss() + } + create() + }.show() + } + } + /** * Click listener for a textview's copy button. * @param value Value to be copied diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index eef0506e7..e161ed1bf 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -1257,4 +1257,8 @@ Verified By: %1$s Delete + + Are you sure you want to delete this login? + + Delete