From f9fbf5de0c79c8664b2130a04edc3c64c1706a18 Mon Sep 17 00:00:00 2001 From: Konrad Pozniak Date: Fri, 27 Dec 2024 09:05:04 +0100 Subject: [PATCH] improve CaptionDialog for small screens (#4831) (or giant soft keyboards) https://mastodon.social/@bewitchedmind/113703199056088283 Before / After This makes it also look a bit weird on large screens, but I found no easy way to limit the size. Its now basically back to where it was in Tusky 26. closes https://github.com/tuskyapp/Tusky/issues/4832 --- .../tusky/components/compose/dialog/CaptionDialog.kt | 5 +++++ app/src/main/res/values/dimens.xml | 2 ++ 2 files changed, 7 insertions(+) diff --git a/app/src/main/java/com/keylesspalace/tusky/components/compose/dialog/CaptionDialog.kt b/app/src/main/java/com/keylesspalace/tusky/components/compose/dialog/CaptionDialog.kt index be7ff237c..8a6047379 100644 --- a/app/src/main/java/com/keylesspalace/tusky/components/compose/dialog/CaptionDialog.kt +++ b/app/src/main/java/com/keylesspalace/tusky/components/compose/dialog/CaptionDialog.kt @@ -49,8 +49,13 @@ class CaptionDialog : DialogFragment() { override fun onCreateDialog(savedInstanceState: Bundle?): Dialog { val localId = arguments?.getInt(LOCAL_ID_ARG) ?: error("Missing localId") + val inset = requireContext().resources.getDimensionPixelSize(R.dimen.caption_dialog_inset) return MaterialAlertDialogBuilder(requireContext()) .setView(createView(savedInstanceState)) + .setBackgroundInsetTop(inset) + .setBackgroundInsetEnd(inset) + .setBackgroundInsetBottom(inset) + .setBackgroundInsetStart(inset) .setPositiveButton(android.R.string.ok) { _, _ -> listener.onUpdateDescription(localId, binding.imageDescriptionText.text.toString()) } diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml index 0d867eac0..c0e64436a 100644 --- a/app/src/main/res/values/dimens.xml +++ b/app/src/main/res/values/dimens.xml @@ -79,4 +79,6 @@ 88dp 32dp + 24dp +