Tom Kuschka Tom Kuschka - 3 months ago 17
Android Question

Changing color of single drawable in RecyclerView will change all drawables

I just tried to change the color of my drawable inside my row depending on a value but instead of one drawable the adapter changed all of them.

Here is my Adapter:

public class ReportAdapter extends RecyclerView.Adapter<ReportAdapter.ReportViewHolder> {

DataBaseHelper dataBase;

private LayoutInflater inflater;
List<ChoosedSubject> data = Collections.emptyList();
Context context;
OnItemClickListener itemClickListener;

public ReportAdapter(Context context, List<ChoosedSubject> data, OnItemClickListener itemClickListener) {
inflater = LayoutInflater.from(context); = data;
this.context = context;
this.itemClickListener = itemClickListener;

public ReportViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
View view = inflater.inflate(R.layout.report_cell, parent, false);
ReportViewHolder holder = new ReportViewHolder(view);

dataBase = new DataBaseHelper(context);

return holder;

//Set Data inside RecyclerView
public void onBindViewHolder(ReportViewHolder holder, int position) {
ChoosedSubject current = data.get(position);
Grades grades = new Grades(context);

Resources resources = context.getResources();
int iconColor;
Drawable icon;

icon = ContextCompat.getDrawable(context, dataBase.getSpecificChoosedSubjectAppendingToName(current.getName()).get(0).getChoosedIcon());

if (dataBase.getSpecificChoosedSubjectAppendingToName(current.getName()).get(0).getChoosedIcon() != R.drawable.subject_default) {
iconColor = resources.getColor(dataBase.getSpecificChoosedSubjectAppendingToName(current.getName()).get(0).getChoosedColor());
icon.setColorFilter(iconColor, PorterDuff.Mode.SRC_IN);
} else {


NumberFormat formatter = NumberFormat.getNumberInstance();
String output = formatter.format(dataBase.getSpecificChoosedSubjectAppendingToName(current.getName()).get(0).getAverage());

int formattedValue = Integer.valueOf(output);

int boxColor = 0;
Drawable box = ContextCompat.getDrawable(context, R.drawable.markbox);
Drawable boxBorder = ContextCompat.getDrawable(context, R.drawable.markbox_border);

if (formattedValue >= 10) {
boxColor = resources.getColor(R.color.positive);
} else if (formattedValue >= 4 && formattedValue <= 9) {
boxColor = resources.getColor(R.color.neutral);
} else if (formattedValue < 4) {
boxColor = resources.getColor(R.color.negative);


box.setColorFilter(boxColor, PorterDuff.Mode.SRC_IN);
boxBorder.setColorFilter(boxColor, PorterDuff.Mode.SRC_IN);



public int getItemCount() {
return data.size();

public class ReportViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {

TextView subject;
ImageView icon;
ImageView markbox;
ImageView markboxBorder;
TextView average;

public ReportViewHolder(View itemView) {
subject = (TextView) itemView.findViewById(;
icon = (ImageView) itemView.findViewById(;
markbox = (ImageView) itemView.findViewById(;
markboxBorder = (ImageView) itemView.findViewById(;
average = (TextView) itemView.findViewById(;


public void onClick(View v) {
itemClickListener.onItemClick(v, this.getAdapterPosition());



Knows anybody what to do? Thank you for your help!!!


It's sort of caching. From the Android docs:

if you instantiate two Drawable objects from the same image resource, then change a property (such as the alpha) for one of the Drawables, then it will also affect the other. So when dealing with multiple instances of an image resource, instead of directly transforming the Drawable, you should perform a tween animation.

Drawable.mutate() after creating should fix the issue.

A mutable drawable is guaranteed to not share its state with any other drawable. This is especially useful when you need to modify properties of drawables loaded from resources. By default, all drawables instances loaded from the same resource share a common state; if you modify the state of one instance, all the other instances will receive the same modification.

Something like this:

Drawable box = ContextCompat.getDrawable(context, R.drawable.markbox).mutate();
Drawable boxBorder = ContextCompat.getDrawable(context, R.drawable.markbox_border).mutate();