From 1e5dde855621bd03da6b68f52dc6696c1fd24680 Mon Sep 17 00:00:00 2001
From: Damyan Ivanov <dam+mobileledger@ktnx.net>
Date: Thu, 21 May 2020 18:23:40 +0300
Subject: [PATCH] remove long unused items from account list

no "favourite" accounts anymore
---
 .../AccountSummaryAdapter.java                | 31 +------------------
 .../main/res/layout/account_summary_row.xml   | 19 +-----------
 2 files changed, 2 insertions(+), 48 deletions(-)

diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java b/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java
index 3ceccc15..a6566f17 100644
--- a/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java
+++ b/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java
@@ -24,7 +24,6 @@ import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
-import android.widget.CheckBox;
 import android.widget.FrameLayout;
 import android.widget.ImageView;
 import android.widget.TextView;
@@ -43,11 +42,8 @@ import androidx.recyclerview.widget.RecyclerView;
 public class AccountSummaryAdapter
         extends RecyclerView.Adapter<AccountSummaryAdapter.LedgerRowHolder> {
     public static final int AMOUNT_LIMIT = 3;
-    private boolean selectionActive;
 
-    AccountSummaryAdapter() {
-        this.selectionActive = false;
-    }
+    AccountSummaryAdapter() { }
 
     public void onBindViewHolder(@NonNull LedgerRowHolder holder, int position) {
         try (LockHolder lh = Data.accounts.lockForReading()) {
@@ -86,9 +82,6 @@ public class AccountSummaryAdapter
                     holder.tvAccountAmounts.setTypeface(null, Typeface.NORMAL);
                 }
 
-                holder.selectionCb.setVisibility(selectionActive ? View.VISIBLE : View.GONE);
-                holder.selectionCb.setChecked(!acc.isHiddenByStarToBe());
-
                 holder.row.setTag(R.id.POS, position);
             }
             else {
@@ -110,27 +103,6 @@ public class AccountSummaryAdapter
     public int getItemCount() {
         return Data.accounts.size();
     }
-    public void startSelection() {
-        try (LockHolder lh = Data.accounts.lockForWriting()) {
-            for (int i = 0; i < Data.accounts.size(); i++) {
-                LedgerAccount acc = Data.accounts.get(i);
-                acc.setHiddenByStarToBe(acc.isHiddenByStar());
-            }
-            this.selectionActive = true;
-            lh.downgrade();
-            notifyDataSetChanged();
-        }
-    }
-
-    public void stopSelection() {
-        this.selectionActive = false;
-        notifyDataSetChanged();
-    }
-
-    public boolean isSelectionActive() {
-        return selectionActive;
-    }
-
     public void selectItem(int position) {
         try (LockHolder lh = Data.accounts.lockForWriting()) {
             LedgerAccount acc = Data.accounts.get(position);
@@ -167,7 +139,6 @@ public class AccountSummaryAdapter
             this.row = itemView.findViewById(R.id.account_summary_row);
             this.tvAccountName = itemView.findViewById(R.id.account_row_acc_name);
             this.tvAccountAmounts = itemView.findViewById(R.id.account_row_acc_amounts);
-            this.selectionCb = itemView.findViewById(R.id.account_row_check);
             this.vTrailer = itemView.findViewById(R.id.account_summary_trailer);
             this.expanderContainer = itemView.findViewById(R.id.account_expander_container);
             this.expander = itemView.findViewById(R.id.account_expander);
diff --git a/app/src/main/res/layout/account_summary_row.xml b/app/src/main/res/layout/account_summary_row.xml
index 08d18b98..0dc4816e 100644
--- a/app/src/main/res/layout/account_summary_row.xml
+++ b/app/src/main/res/layout/account_summary_row.xml
@@ -26,16 +26,6 @@
     android:longClickable="true"
     android:layout_height="wrap_content">
 
-    <CheckBox
-        android:id="@+id/account_row_check"
-        android:layout_width="wrap_content"
-        android:layout_height="match_parent"
-        android:button="@drawable/checkbox_star_black"
-        android:onClick="onAccountSummaryRowViewClicked"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent" />
-
     <TextView
         android:id="@+id/account_row_acc_name"
         style="@style/account_summary_account_name"
@@ -49,7 +39,7 @@
         android:text="Account name, a really long one. A very very very long one. It may even spawn on more than two lines -- three, four or more."
         android:textAppearance="@android:style/TextAppearance.Material.Medium"
         app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintStart_toEndOf="@id/account_row_check"
+        app:layout_constraintStart_toStartOf="parent"
         app:layout_constraintTop_toTopOf="parent"
         tools:ignore="HardcodedText" />
 
@@ -109,13 +99,6 @@
         app:layout_constraintBottom_toBottomOf="@id/account_row_acc_amounts"
         android:background="@drawable/fade_down_white">
 
-        <!--<ImageView-->
-        <!--android:layout_gravity="center_vertical|end"-->
-        <!--android:id="@+id/account_row_amounts_expander"-->
-        <!--android:layout_width="20dp"-->
-        <!--android:layout_height="20dp"-->
-        <!--android:background="@drawable/ic_expand_more_black_24dp" />-->
-
     </FrameLayout>
 
     <View
-- 
2.39.2