diff --git a/app/src/main/java/org/thoughtcrime/securesms/components/settings/conversation/preferences/BioTextPreference.kt b/app/src/main/java/org/thoughtcrime/securesms/components/settings/conversation/preferences/BioTextPreference.kt index 4a1d42dc81..79f282c0fa 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/components/settings/conversation/preferences/BioTextPreference.kt +++ b/app/src/main/java/org/thoughtcrime/securesms/components/settings/conversation/preferences/BioTextPreference.kt @@ -53,16 +53,15 @@ object BioTextPreference { } return SpannableStringBuilder(name).apply { - var appendedToName = false if (recipient.showVerified()) { - SpanUtil.appendCenteredImageSpan(this, ContextUtil.requireDrawable(context, R.drawable.ic_official_28), 28, 28) - appendedToName = true + SpanUtil.appendSpacer(this, 8) + SpanUtil.appendCenteredImageSpanWithoutSpace(this, ContextUtil.requireDrawable(context, R.drawable.ic_official_28), 28, 28) } else if (recipient.isSystemContact) { val drawable = ContextUtil.requireDrawable(context, R.drawable.symbol_person_circle_24).apply { setTint(ContextCompat.getColor(context, R.color.signal_colorOnSurface)) } - SpanUtil.appendCenteredImageSpan(this, drawable, 24, 24) - appendedToName = true + SpanUtil.appendSpacer(this, 8) + SpanUtil.appendCenteredImageSpanWithoutSpace(this, drawable, 24, 24) } if (recipient.isIndividual && !recipient.isSelf) { @@ -71,9 +70,6 @@ object BioTextPreference { setTint(ContextCompat.getColor(context, R.color.signal_colorOutline)) } - if (!appendedToName) { - append(" ") - } append(SpanUtil.buildCenteredImageSpan(drawable)) } } diff --git a/app/src/main/java/org/thoughtcrime/securesms/recipients/ui/bottomsheet/RecipientBottomSheetDialogFragment.java b/app/src/main/java/org/thoughtcrime/securesms/recipients/ui/bottomsheet/RecipientBottomSheetDialogFragment.java index e65d3cbb72..2a3fc5f564 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/recipients/ui/bottomsheet/RecipientBottomSheetDialogFragment.java +++ b/app/src/main/java/org/thoughtcrime/securesms/recipients/ui/bottomsheet/RecipientBottomSheetDialogFragment.java @@ -188,12 +188,12 @@ public final class RecipientBottomSheetDialogFragment extends BottomSheetDialogF fullName.setVisibility(TextUtils.isEmpty(name) ? View.GONE : View.VISIBLE); SpannableStringBuilder nameBuilder = new SpannableStringBuilder(name); if (recipient.showVerified()) { - SpanUtil.appendCenteredImageSpanWithoutSpace(nameBuilder, new ColorDrawable(Color.TRANSPARENT), 8, 8); + SpanUtil.appendSpacer(nameBuilder, 8); SpanUtil.appendCenteredImageSpanWithoutSpace(nameBuilder, ContextUtil.requireDrawable(requireContext(), R.drawable.ic_official_28), 28, 28); } else if (recipient.isSystemContact()) { Drawable drawable = ContextUtil.requireDrawable(requireContext(), R.drawable.symbol_person_circle_24); drawable.setTint(ContextCompat.getColor(requireContext(), R.color.signal_colorOnSurface)); - SpanUtil.appendCenteredImageSpanWithoutSpace(nameBuilder, new ColorDrawable(Color.TRANSPARENT), 8, 8); + SpanUtil.appendSpacer(nameBuilder, 8); SpanUtil.appendCenteredImageSpanWithoutSpace(nameBuilder, drawable, 24, 24); } diff --git a/app/src/main/java/org/thoughtcrime/securesms/util/SpanUtil.java b/app/src/main/java/org/thoughtcrime/securesms/util/SpanUtil.java index 3a78a7bc7f..9e166cf661 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/util/SpanUtil.java +++ b/app/src/main/java/org/thoughtcrime/securesms/util/SpanUtil.java @@ -147,6 +147,10 @@ public final class SpanUtil { builder.append(" ").append(SpanUtil.buildCenteredImageSpan(drawable)); } + public static void appendSpacer(@NonNull SpannableStringBuilder builder, int width) { + SpanUtil.appendCenteredImageSpanWithoutSpace(builder, new ColorDrawable(Color.TRANSPARENT), width, 8); + } + public static void appendCenteredImageSpanWithoutSpace(@NonNull SpannableStringBuilder builder, @NonNull Drawable drawable, int width, int height) { drawable.setBounds(0, 0, ViewUtil.dpToPx(width), ViewUtil.dpToPx(height)); builder.append(SpanUtil.buildCenteredImageSpan(drawable));