From 453231b5fd131e46afec9edbda2328d497cb8cd4 Mon Sep 17 00:00:00 2001 From: Sawyer Blatz Date: Fri, 7 Feb 2020 09:41:17 -0800 Subject: [PATCH] For #7753: Increases touch target of crash page --- .../fenix/crashes/CrashReporterFragment.kt | 20 +++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/crashes/CrashReporterFragment.kt b/app/src/main/java/org/mozilla/fenix/crashes/CrashReporterFragment.kt index 4af8ad013..eff300e69 100644 --- a/app/src/main/java/org/mozilla/fenix/crashes/CrashReporterFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/crashes/CrashReporterFragment.kt @@ -13,6 +13,7 @@ import kotlinx.android.synthetic.main.fragment_crash_reporter.* import mozilla.components.lib.crash.Crash import org.mozilla.fenix.R import org.mozilla.fenix.ext.hideToolbar +import org.mozilla.fenix.ext.increaseTapArea import org.mozilla.fenix.ext.requireComponents import org.mozilla.fenix.ext.settings @@ -37,11 +38,18 @@ class CrashReporterFragment : Fragment(R.layout.fragment_crash_reporter) { settings = requireContext().settings() ) - restoreTabButton.setOnClickListener { - controller.handleCloseAndRestore(sendCrashCheckbox.isChecked) + restoreTabButton.apply { + increaseTapArea(TAP_INCREASE_DP) + setOnClickListener { + controller.handleCloseAndRestore(sendCrashCheckbox.isChecked) + } } - closeTabButton.setOnClickListener { - controller.handleCloseAndRemove(sendCrashCheckbox.isChecked) + + closeTabButton.apply { + increaseTapArea(TAP_INCREASE_DP) + setOnClickListener { + controller.handleCloseAndRemove(sendCrashCheckbox.isChecked) + } } } @@ -49,4 +57,8 @@ class CrashReporterFragment : Fragment(R.layout.fragment_crash_reporter) { super.onResume() hideToolbar() } + + companion object { + private const val TAP_INCREASE_DP = 12 + } }