add FIXME about account list trailer divider

[mobile-ledger.git] / app / src / main / java / net / ktnx / mobileledger / ui / account_summary / AccountSummaryAdapter.java

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 060828e746424e69f90f37704d469d30209cfbd8..3384fe8184f9e5ad1bb180cbc36db7f4be1e7d79 100644 (file)
--- 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
@@ -84,6 +84,8 @@ public class AccountSummaryAdapter
                 holder.row.setTag(R.id.POS, position);
             }
             else {
+                // FIXME trailer's divider looks bad
+                // perhaps replace the trailer with bottom padding for the last item?
                 holder.vTrailer.setVisibility(View.VISIBLE);
                 holder.row.setVisibility(View.GONE);
             }
@@ -121,17 +123,6 @@ public class AccountSummaryAdapter
             this.accountExpanderContainer =
                     itemView.findViewById(R.id.account_row_amounts_expander_container);
 
-            expanderContainer.addOnLayoutChangeListener(
-                    (v, left, top, right, bottom, oldLeft, oldTop, oldRight, oldBottom) -> {
-                        int w = right - left;
-                        int h = bottom - top;
-                        if (h > w) {
-                            int p = (h - w) / 2;
-                            v.setPadding(0, p, 0, p);
-                        }
-                        else v.setPadding(0, 0, 0, 0);
-                    });
-
             itemView.setOnLongClickListener(this::onItemLongClick);
             tvAccountName.setOnLongClickListener(this::onItemLongClick);
             tvAccountAmounts.setOnLongClickListener(this::onItemLongClick);