diff --git a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt index 95b833c3d..3bb93c1a6 100644 --- a/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt +++ b/app/src/main/java/org/mozilla/fenix/home/HomeFragment.kt @@ -98,6 +98,7 @@ import org.mozilla.fenix.settings.deletebrowsingdata.deleteAndQuit import org.mozilla.fenix.utils.FragmentPreDrawManager import org.mozilla.fenix.utils.allowUndo import org.mozilla.fenix.whatsnew.WhatsNew +import kotlin.math.min @SuppressWarnings("TooManyFunctions", "LargeClass") class HomeFragment : Fragment() { @@ -612,10 +613,13 @@ class HomeFragment : Fragment() { context?.let { val layout = LayoutInflater.from(it) .inflate(R.layout.pbm_shortcut_popup, null) - val trackingOnboarding = + val privateBrowsingRecommend = PopupWindow( layout, - (resources.displayMetrics.widthPixels / CFR_WIDTH_DIVIDER).toInt(), + min( + (resources.displayMetrics.widthPixels / CFR_WIDTH_DIVIDER).toInt(), + (resources.displayMetrics.heightPixels / CFR_WIDTH_DIVIDER).toInt() + ), LinearLayout.LayoutParams.WRAP_CONTENT, true ) @@ -623,19 +627,20 @@ class HomeFragment : Fragment() { setOnClickListener { context.metrics.track(Event.PrivateBrowsingAddShortcutCFR) PrivateShortcutCreateManager.createPrivateShortcut(context) - trackingOnboarding.dismiss() + privateBrowsingRecommend.dismiss() } } layout.findViewById