Skip to content

Commit a480ae3

Browse files
committed
Change background colour of item on being selected
1 parent e1e195b commit a480ae3

File tree

1 file changed

+14
-3
lines changed

1 file changed

+14
-3
lines changed

app/src/main/java/fr/free/nrw/commons/nearby/NearbyFilterSearchRecyclerViewAdapter.java

+14-3
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,12 @@ public void onBindViewHolder(@NonNull RecyclerViewHolder holder, int position) {
8080
holder.placeTypeIcon.setImageResource(label.getIcon());
8181
holder.placeTypeLabel.setText(label.toString());
8282

83-
holder.placeTypeLayout.setBackgroundColor(label.isSelected() ? ContextCompat.getColor(context, R.color.divider_grey) : callback.isDarkTheme() ? Color.BLACK : Color.WHITE);
83+
if (callback.isDarkTheme()) {
84+
holder.placeTypeLayout.setBackgroundColor(label.isSelected() ? ContextCompat.getColor(context, R.color.opak_middle_grey) : ContextCompat.getColor(context, R.color.black));
85+
} else {
86+
holder.placeTypeLayout.setBackgroundColor(label.isSelected() ? ContextCompat.getColor(context, R.color.divider_grey) : ContextCompat.getColor(context, R.color.white));
87+
}
88+
8489
holder.placeTypeLayout.setOnClickListener(view -> {
8590
callback.setCheckboxUnknown();
8691
if (label.isSelected()) {
@@ -89,7 +94,13 @@ public void onBindViewHolder(@NonNull RecyclerViewHolder holder, int position) {
8994
selectedLabels.add(label);
9095
}
9196
label.setSelected(!label.isSelected());
92-
holder.placeTypeLayout.setBackgroundColor(label.isSelected() ? ContextCompat.getColor(context, R.color.divider_grey) : callback.isDarkTheme() ? Color.BLACK : Color.WHITE);
97+
98+
if (callback.isDarkTheme()) {
99+
holder.placeTypeLayout.setBackgroundColor(label.isSelected() ? ContextCompat.getColor(context, R.color.opak_middle_grey) : ContextCompat.getColor(context, R.color.black));
100+
} else {
101+
holder.placeTypeLayout.setBackgroundColor(label.isSelected() ? ContextCompat.getColor(context, R.color.divider_grey) : ContextCompat.getColor(context, R.color.white));
102+
}
103+
93104
callback.filterByMarkerType(selectedLabels, 0, false, false);
94105
});
95106
}
@@ -165,7 +176,7 @@ public void setRecyclerViewAdapterAllSelected() {
165176
notifyDataSetChanged();
166177
}
167178

168-
public interface Callback{
179+
public interface Callback {
169180

170181
void setCheckboxUnknown();
171182

0 commit comments

Comments
 (0)