Skip to content

Commit 77ef560

Browse files
committed
Remove passing in text to GlideImageSpan
1 parent 2500ff4 commit 77ef560

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

app/src/main/java/com/perflyst/twire/adapters/ChatAdapter.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public void onBindViewHolder(@NonNull final ContactViewHolder holder, int positi
9090

9191
final SpannableStringBuilder builder = new SpannableStringBuilder();
9292
for (Badge badge : message.getBadges()) {
93-
final GlideImageSpan badgeSpan = new GlideImageSpan(context, badge.getUrl(2), holder.message, builder, 36, 1, badge.color);
93+
final GlideImageSpan badgeSpan = new GlideImageSpan(context, badge.getUrl(2), holder.message, 36, 1, badge.color);
9494
appendSpan(builder, " ", badgeSpan).append(" ");
9595
}
9696

@@ -113,7 +113,7 @@ public void onBindViewHolder(@NonNull final ContactViewHolder holder, int positi
113113
int emoteSize = settings.getEmoteSize();
114114
int emotePixels = emoteSize == 1 ? 28 : emoteSize == 2 ? 56 : 112;
115115

116-
final GlideImageSpan emoteSpan = new GlideImageSpan(context, emote.getEmoteUrl(emoteSize, isNightTheme), holder.message, builder, emotePixels, (float) emote.getBestAvailableSize(emoteSize) / emoteSize);
116+
final GlideImageSpan emoteSpan = new GlideImageSpan(context, emote.getEmoteUrl(emoteSize, isNightTheme), holder.message, emotePixels, (float) emote.getBestAvailableSize(emoteSize) / emoteSize);
117117

118118
builder.setSpan(emoteSpan, fromPosition + beforeMessage.length(), toPosition + 1 + beforeMessage.length(), Spanned.SPAN_INCLUSIVE_INCLUSIVE);
119119
}

app/src/main/java/com/perflyst/twire/misc/GlideImageSpan.java

+4-5
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
import android.graphics.drawable.ColorDrawable;
77
import android.graphics.drawable.Drawable;
88
import android.graphics.drawable.LayerDrawable;
9-
import android.text.SpannableStringBuilder;
109
import android.util.Log;
1110
import android.view.View;
1211
import android.widget.TextView;
@@ -26,8 +25,8 @@ public class GlideImageSpan extends VerticalImageSpan implements Drawable.Callba
2625
private Drawable mDrawable;
2726
private Animatable animatable;
2827

29-
public GlideImageSpan(Context context, String url, TextView textView, SpannableStringBuilder builder, int assumedSize, float scale, String backgroundColor) {
30-
this(context, url, textView, builder, assumedSize, scale);
28+
public GlideImageSpan(Context context, String url, TextView textView, int assumedSize, float scale, String backgroundColor) {
29+
this(context, url, textView, assumedSize, scale);
3130

3231
if (backgroundColor == null)
3332
return;
@@ -39,7 +38,7 @@ public GlideImageSpan(Context context, String url, TextView textView, SpannableS
3938
layerDrawable.setId(1, 1);
4039
}
4140

42-
public GlideImageSpan(Context context, String url, TextView textView, SpannableStringBuilder builder, int assumedSize, float scale) {
41+
public GlideImageSpan(Context context, String url, TextView textView, int assumedSize, float scale) {
4342
super(new BlankDrawable());
4443

4544
this.textView = textView;
@@ -80,7 +79,7 @@ public void onResourceReady(@NonNull Drawable resource, @Nullable Transition<? s
8079
mDrawable = resource;
8180

8281
if (resource.getIntrinsicWidth() != assumedSize) {
83-
textView.setText(builder);
82+
textView.setText(textView.getText());
8483
Log.d("EmoteShift", "Got " + resource.getIntrinsicWidth() + " but assumed " + assumedSize + " (" + url + ")");
8584
} else {
8685
textView.invalidate();

0 commit comments

Comments
 (0)