Kaydet (Commit) 16be8279 authored tarafından Christian Lohmaier's avatar Christian Lohmaier

android: position in RecyclerView must not be treated as fixed

Change-Id: Id7f88b2b1b9913a006b0b73ceb4421b47fccc495
üst 1a785721
...@@ -38,7 +38,7 @@ public class ColorPaletteAdapter extends RecyclerView.Adapter<ColorPaletteAdapte ...@@ -38,7 +38,7 @@ public class ColorPaletteAdapter extends RecyclerView.Adapter<ColorPaletteAdapte
} }
@Override @Override
public void onBindViewHolder(final ColorPaletteViewHolder holder, final int position) { public void onBindViewHolder(final ColorPaletteViewHolder holder, int position) {
holder.colorBox.setBackgroundColor(color_palette[upperSelectedBox][position]); holder.colorBox.setBackgroundColor(color_palette[upperSelectedBox][position]);
if (selectedBox == position) { if (selectedBox == position) {
...@@ -50,7 +50,7 @@ public class ColorPaletteAdapter extends RecyclerView.Adapter<ColorPaletteAdapte ...@@ -50,7 +50,7 @@ public class ColorPaletteAdapter extends RecyclerView.Adapter<ColorPaletteAdapte
holder.colorBox.setOnClickListener(new View.OnClickListener() { holder.colorBox.setOnClickListener(new View.OnClickListener() {
@Override @Override
public void onClick(View view) { public void onClick(View view) {
setPosition(position); setPosition(holder.getAdapterPosition());
} }
}); });
if (animate) //it will only animate when the upper color box is selected if (animate) //it will only animate when the upper color box is selected
......
...@@ -39,7 +39,7 @@ public class ColorPickerAdapter extends RecyclerView.Adapter<ColorPickerAdapter. ...@@ -39,7 +39,7 @@ public class ColorPickerAdapter extends RecyclerView.Adapter<ColorPickerAdapter.
} }
@Override @Override
public void onBindViewHolder(final ColorPickerViewHolder holder, final int position) { public void onBindViewHolder(final ColorPickerViewHolder holder, int position) {
holder.colorBox.setBackgroundColor(colorList[position]); holder.colorBox.setBackgroundColor(colorList[position]);
if (selectedBox != position) if (selectedBox != position)
...@@ -51,12 +51,10 @@ public class ColorPickerAdapter extends RecyclerView.Adapter<ColorPickerAdapter. ...@@ -51,12 +51,10 @@ public class ColorPickerAdapter extends RecyclerView.Adapter<ColorPickerAdapter.
holder.colorBox.setOnClickListener(new View.OnClickListener() { holder.colorBox.setOnClickListener(new View.OnClickListener() {
@Override @Override
public void onClick(View view) { public void onClick(View view) {
setPosition(position); setPosition(holder.getAdapterPosition());
colorPaletteListener.applyColor(colorList[position]); colorPaletteListener.applyColor(colorList[holder.getAdapterPosition()]);
} }
}); });
} }
@Override @Override
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment