11

I have this function that works fine on Android 4.4.1, but breaks on 5.0+.

  public static SpannableStringBuilder prependImage(Drawable drawable, String text) {
    SpannableStringBuilder builder = new SpannableStringBuilder("  " + text);
    builder.setSpan(new ImageSpan(drawable), 0, 1, Spanned.SPAN_EXCLUSIVE_EXCLUSIVE);
    return builder;
  }

And I use it like this:

class MyButton extends Button {

    // ... snip ...

    setText(
        prependImage(
            getDrawable(imageResource, color),                     
            getContext().getString(stringResource)),
        BufferType.SPANNABLE);

Here is the getDrawable() method referenced above:

 private Drawable getDrawable(int resource, int color) {
    final Resources resources = getContext().getResources();
    Drawable drawable = resources.getDrawable(resource);
    if (drawable != null) {
      drawable.setColorFilter(color, PorterDuff.Mode.SRC_IN);
      drawable.setBounds(0, 0, drawable.getIntrinsicWidth(), drawable.getIntrinsicHeight());
    }
    return drawable;
  }

When I debug, everything seems to succeed, but no image is drawn. Any ideas what I might be doing wrong?

i_am_jorf
  • 53,608
  • 15
  • 131
  • 222
  • I created a class that `extends android.widget.Button`, tried using your `prependImage(Drawable, String)` method on the emulator running KitKat and the image was *not* drawn. – Jared Rummler Sep 30 '15 at 05:55
  • Well, there should be a way to make it draw on both. I'm perfectly happy to throw this away and do something else. Did you make sure you set the bounds of the drawable to be something other than 0,0,0,0? – i_am_jorf Sep 30 '15 at 05:59
  • @i_am_jorf You could use `TextView` instead of `Button`. It seems to be working fine with `TextView` in all the android versions. – Abhishek V Sep 30 '15 at 09:54
  • on a Nexus 5 with Android 5.1.1 is working. Where are you testing it ? – Blackbelt Sep 30 '15 at 12:31
  • I am testing on a Samsung S6 (5.1), and S4 (4.4). – i_am_jorf Sep 30 '15 at 16:03

5 Answers5

23

By default, in Material buttons are styled to show text in all-caps. However, there is a bug in the AllCapsTransformationMethod used for capitalization that causes it to discard Spannable data.

You can override the default button styling and disable all-caps by specifying android:textAllCaps="false" on your Button.

<Button
    ...
    android:textAllCaps="false" />

have a look here

Community
  • 1
  • 1
Shishram
  • 1,514
  • 11
  • 20
4

Your code related to working with Spannables is ok. You can check it by setting text for TextView.

The problem is in material design of button on Android 5.0.

<style name="Widget.Material.Button">
    <item name="background">@drawable/btn_default_material</item>
    <item name="textAppearance">?attr/textAppearanceButton</item>
    <item name="minHeight">48dip</item>
    <item name="minWidth">88dip</item>
    <item name="stateListAnimator">@anim/button_state_list_anim_material</item>
    <item name="focusable">true</item>
    <item name="clickable">true</item>
    <item name="gravity">center_vertical|center_horizontal</item>
</style>

There are two solution.

The first one is just use TextView as your button and setText with image to it.

For another (and may be more correct) you need to extend button style (Widget.Material.Button) in next way:

<style name="BtnStyle" parent="android:Widget.Material.Button">
    <item name="android:textAppearance">@null</item>
</style>

Then in your layout:

<Button
    android:id="@+id/test2"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:text="Test"
    style="@style/BtnStyle"/>

After you'll do it you should see the images in the button.

Don't forget for Android version that is lower than 5.0 you should create BtnStyle too, but in other resource directory (res/values-v14/style.xml).

Ilya Tretyakov
  • 6,848
  • 3
  • 28
  • 45
1

One thing to note, drawable vectors won't work, you must either have a drawable which is a png or jpeg or pass to ImageSpan bitmap instead

Ben Levi
  • 177
  • 1
  • 5
0

Maybe you need to check the content of that Drawable object, you use the getDrawable() to get the Drawable object, but the API definition seems not match your calling parameters.

For Android 5.0+

Drawable getDrawable(int id) This method was deprecated in API level 22. Use getDrawable(int, Theme) instead.

Drawable getDrawable(int id, Resources.Theme theme) Return a drawable object associated with a particular resource ID and styled for the specified theme.

The second parameter looks like to be a Theme, not a color. right ?

Zephyr
  • 6,123
  • 34
  • 33
  • Ah, sorry. I have a private method in `MyButton` called `getDrawable(int id, int color)` that wraps the call to `getContext().getResources().getDrawable(int id)` and also calls `drawable.setColorFilter(color, PorterDuff.Mode.SRC_IN);`. This returns, what looks like, a perfectly reasonable drawable. – i_am_jorf Sep 30 '15 at 05:11
0

try this

 if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
      getResources().getDrawable(R.drawable.your_drawable, getTheme());
            } else {
                getResources().
                        getDrawable(R.drawable.your_drawable);
            }
Amit Kumar
  • 208
  • 1
  • 9