#396 Filter cards

RecyclerView with selection - working draft

2 files changed, 62 insertions, 61 deletions

diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/FilterDialogFragment.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/FilterDialogFragment.java
index 38220c725..d25f4d28f 100644
--- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/FilterDialogFragment.java
+++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/FilterDialogFragment.java
@@ -76,11 +76,9 @@ public class FilterDialogFragment extends BrandedDialogFragment {
         final DialogFilterBinding binding = DialogFilterBinding.inflate(requireActivity().getLayoutInflater());
 
         overdueAdapter = new OverdueFilterAdapter(requireContext());
-        labelAdapter = new LabelFilterAdapter(requireContext());
         userAdapter = new UserFilterAdapter(requireContext());
 
         binding.overdue.setAdapter(overdueAdapter);
-        binding.labels.setAdapter(labelAdapter);
         binding.people.setAdapter(userAdapter);
 
         binding.overdue.setSelection(overdueAdapter.getPosition(this.filterInformation.getDueType()));
@@ -99,23 +97,9 @@ public class FilterDialogFragment extends BrandedDialogFragment {
         SyncManager syncManager = new SyncManager(requireActivity());
 
         observeOnce(syncManager.findProposalsForLabelsToAssign(account.getId(), boardId), requireActivity(), (labels) -> {
-            labelAdapter.addAll(labels);
-            labelAdapter.notifyDataSetChanged();
-            for (long labelId : this.filterInformation.getLabelIDs()) {
-                binding.labels.setSelection(labelAdapter.getPosition(labelId));
-            }
-            binding.labels.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
-                @Override
-                public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
-                    DeckLog.info("clicked position: " + position);
-                    filterInformation.addLabelId(labelAdapter.getItemId(position));
-                }
-
-                @Override
-                public void onNothingSelected(AdapterView<?> parent) {
-                    // Nothing to do
-                }
-            });
+            labelAdapter = new LabelFilterAdapter(requireContext(), labels, this.filterInformation.getLabelIDs());
+            binding.labels.setNestedScrollingEnabled(false);
+            binding.labels.setAdapter(labelAdapter);
         });
 
         observeOnce(syncManager.findProposalsForUsersToAssign(account.getId(), boardId), requireActivity(), (users) -> {
@@ -128,7 +112,7 @@ public class FilterDialogFragment extends BrandedDialogFragment {
                 @Override
                 public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
                     DeckLog.info("clicked position: " + position);
-                    filterInformation.addLabelId(userAdapter.getItemId(position));
+                    filterInformation.addUserId(userAdapter.getItemId(position));
                 }
 
                 @Override
@@ -143,7 +127,11 @@ public class FilterDialogFragment extends BrandedDialogFragment {
                 .setView(binding.getRoot())
                 .setNeutralButton(android.R.string.cancel, null)
                 .setNegativeButton(R.string.simple_clear, (a, b) -> viewModel.postFilterInformation(null))
-                .setPositiveButton(R.string.simple_filter, (a, b) -> viewModel.postFilterInformation(filterInformation))
+                .setPositiveButton(R.string.simple_filter, (a, b) -> {
+                    filterInformation.clearLabelIds();
+                    filterInformation.addAllLabelIds(labelAdapter.getSelected());
+                    viewModel.postFilterInformation(filterInformation);
+                })
                 .create();
     }
 
diff --git a/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/LabelFilterAdapter.java b/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/LabelFilterAdapter.java
index 8fed859a5..1105de814 100644
--- a/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/LabelFilterAdapter.java
+++ b/app/src/main/java/it/niedermann/nextcloud/deck/ui/filter/LabelFilterAdapter.java
@@ -4,68 +4,81 @@ import android.content.Context;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
-import android.widget.ArrayAdapter;
+import android.widget.ImageView;
 import android.widget.TextView;
 
 import androidx.annotation.NonNull;
-import androidx.annotation.Nullable;
+import androidx.recyclerview.widget.RecyclerView;
 
-import org.jetbrains.annotations.NotNull;
-
-import java.util.NoSuchElementException;
+import java.util.List;
 
+import it.niedermann.nextcloud.deck.R;
 import it.niedermann.nextcloud.deck.model.Label;
 
-public class LabelFilterAdapter extends ArrayAdapter<Label> {
+public class LabelFilterAdapter extends RecyclerView.Adapter<LabelFilterAdapter.LabelViewHolder> {
+    private Context context;
+    private List<Label> labels;
+    private List<Long> selectedLabelIds;
 
-    @NonNull
-    private final LayoutInflater inflater;
+    public LabelFilterAdapter(@NonNull Context context, @NonNull List<Label> labels, @NonNull List<Long> selectedLabelIds) {
+        super();
+        this.context = context;
+        this.labels = labels;
+        this.selectedLabelIds = selectedLabelIds;
+        setHasStableIds(true);
+        notifyDataSetChanged();
+    }
 
-    @SuppressWarnings("WeakerAccess")
-    public LabelFilterAdapter(@NonNull Context context) {
-        super(context, android.R.layout.simple_list_item_multiple_choice, android.R.id.text1);
-        inflater = LayoutInflater.from(context);
+    @Override
+    public long getItemId(int position) {
+        return labels.get(position).getLocalId();
     }
 
+    @NonNull
     @Override
-    public boolean hasStableIds() {
-        return true;
+    public LabelViewHolder onCreateViewHolder(@NonNull ViewGroup viewGroup, int i) {
+        View view = LayoutInflater.from(context).inflate(R.layout.item_user, viewGroup, false);
+        return new LabelViewHolder(view);
     }
 
     @Override
-    public long getItemId(int position) {
-        final Label label = getItem(position);
-        if (label == null) {
-            throw new NoSuchElementException();
-        }
-        return label.getLocalId();
+    public void onBindViewHolder(@NonNull LabelViewHolder multiViewHolder, int position) {
+        multiViewHolder.bind(labels.get(position));
     }
 
     @Override
-    public View getDropDownView(int position, @Nullable View convertView, @NonNull ViewGroup parent) {
-        return getView(position, convertView, parent);
+    public int getItemCount() {
+        return labels.size();
     }
 
-    @SuppressWarnings("WeakerAccess")
-    public int getPosition(long labelId) {
-        for (int i = 0; i < getCount(); i++) {
-            if (getItemId(i) == labelId) {
-                return i;
-            }
-        }
-        throw new NoSuchElementException();
+    public List<Long> getSelected() {
+        return selectedLabelIds;
     }
 
-    @NotNull
-    @Override
-    public View getView(int position, View convertView, @NotNull ViewGroup parent) {
-        final View view;
-        if (convertView == null) {
-            view = inflater.inflate(android.R.layout.simple_list_item_multiple_choice, parent, false);
-        } else {
-            view = convertView;
+    class LabelViewHolder extends RecyclerView.ViewHolder {
+
+        private TextView textView;
+        private ImageView imageView;
+
+        LabelViewHolder(@NonNull View itemView) {
+            super(itemView);
+            textView = itemView.findViewById(R.id.displayname);
+            imageView = itemView.findViewById(R.id.avatar);
+        }
+
+        void bind(final Label label) {
+            imageView.setVisibility(selectedLabelIds.contains(label.getLocalId()) ? View.VISIBLE : View.GONE);
+            textView.setText(label.getTitle());
+
+            itemView.setOnClickListener(view -> {
+                if (selectedLabelIds.contains(label.getLocalId())) {
+                    selectedLabelIds.remove(label.getLocalId());
+                    itemView.setSelected(false);
+                } else {
+                    selectedLabelIds.add(label.getLocalId());
+                    itemView.setSelected(true);
+                }
+            });
         }
-        ((TextView) view.findViewById(android.R.id.text1)).setText(getItem(position).getTitle());
-        return view;
     }
 }