From daaea5aef4077b363529806449b3a30f1bba7175 Mon Sep 17 00:00:00 2001
From: Damyan Ivanov <dam+mobileledger@ktnx.net>
Date: Sat, 8 May 2021 22:33:11 +0300
Subject: [PATCH] improve account list with Flow helper widget

makes a difference with very deep account trees and/or long commodities
or big amounts and small screen or big font
---
 .../AccountSummaryAdapter.java                |  2 +-
 app/src/main/res/layout/account_list_row.xml  | 39 ++++++++++---------
 2 files changed, 22 insertions(+), 19 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 6b8d0930..2887058f 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
@@ -322,7 +322,7 @@ public class AccountSummaryAdapter extends RecyclerView.Adapter<AccountSummaryAd
 
             if (changes.has(Change.LEVEL)) {
                 ConstraintLayout.LayoutParams lp =
-                        (ConstraintLayout.LayoutParams) b.accountNameLayout.getLayoutParams();
+                        (ConstraintLayout.LayoutParams) b.flowWrapper.getLayoutParams();
                 lp.setMarginStart(
                         acc.getLevel() * rm.getDimensionPixelSize(R.dimen.thumb_row_height) / 3);
             }
diff --git a/app/src/main/res/layout/account_list_row.xml b/app/src/main/res/layout/account_list_row.xml
index 2add4b7c..a11d39b8 100644
--- a/app/src/main/res/layout/account_list_row.xml
+++ b/app/src/main/res/layout/account_list_row.xml
@@ -25,16 +25,14 @@
     android:layout_height="wrap_content"
     android:animateLayoutChanges="true"
     android:longClickable="true"
+    android:minHeight="@dimen/default_account_row_height"
     >
     <androidx.constraintlayout.widget.ConstraintLayout
         android:id="@+id/account_name_layout"
-        android:layout_width="0dp"
+        android:layout_width="wrap_content"
         android:layout_height="wrap_content"
-        app:layout_constraintEnd_toStartOf="@+id/account_row_acc_amounts"
         app:layout_constraintHorizontal_chainStyle="spread_inside"
-        app:layout_constraintHorizontal_weight="3"
-        app:layout_constraintStart_toStartOf="parent"
-        app:layout_constraintTop_toTopOf="parent"
+
         >
         <TextView
             android:id="@+id/account_row_acc_name"
@@ -43,18 +41,16 @@
             android:layout_marginStart="8dp"
             android:gravity="center_vertical"
             android:longClickable="true"
-            android:minHeight="@dimen/default_account_row_height"
             android:paddingStart="8dp"
             android:text="AccountName"
             android:textAppearance="@android:style/TextAppearance.Material.Medium"
             app:layout_constrainedWidth="true"
+            app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toStartOf="@id/account_expander_container"
             app:layout_constraintHorizontal_bias="0.0"
             app:layout_constraintHorizontal_chainStyle="packed"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toTopOf="parent"
-            app:layout_constraintWidth_max="wrap"
-            app:layout_constraintWidth_percent=".67"
             tools:ignore="HardcodedText"
             />
 
@@ -89,20 +85,14 @@
     <TextView
         android:id="@+id/account_row_acc_amounts"
         style="@style/account_summary_amounts"
-        android:layout_width="0dp"
+        android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_marginStart="12dp"
         android:layout_marginEnd="8dp"
         android:gravity="center_vertical"
-        android:minHeight="@dimen/default_account_row_height"
         android:text="USD 123,45\n678,90\nIRAUSD -17 000.00"
         android:textAppearance="@style/TextAppearance.AppCompat.Medium"
         app:layout_constrainedWidth="true"
-        app:layout_constraintBottom_toBottomOf="parent"
-        app:layout_constraintEnd_toEndOf="parent"
-        app:layout_constraintHorizontal_weight="2"
-        app:layout_constraintStart_toEndOf="@id/account_name_layout"
-        app:layout_constraintTop_toTopOf="parent"
         app:layout_constraintWidth_min="90sp"
         tools:ignore="HardcodedText"
         />
@@ -115,7 +105,20 @@
         app:layout_constraintBottom_toBottomOf="@id/account_row_acc_amounts"
         app:layout_constraintEnd_toEndOf="@id/account_row_acc_amounts"
         app:layout_constraintStart_toStartOf="@id/account_row_acc_amounts"
-        >
-
-    </FrameLayout>
+        />
+    <androidx.constraintlayout.helper.widget.Flow
+        android:id="@+id/flow_wrapper"
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:minHeight="@dimen/default_account_row_height"
+        app:constraint_referenced_ids="account_name_layout,account_row_acc_amounts"
+        app:flow_firstHorizontalBias="0"
+        app:flow_firstHorizontalStyle="spread_inside"
+        app:flow_horizontalBias="1"
+        app:flow_verticalStyle="spread_inside"
+        app:flow_wrapMode="chain"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent"
+        />
 </androidx.constraintlayout.widget.ConstraintLayout>
\ No newline at end of file
-- 
2.39.2