Skip to content

Commit

Permalink
fix currency field display
Browse files Browse the repository at this point in the history
On a Nothing Phone 1, on the Add transaction screen the currency were
not visible because the left and right paddings added too much empty
space to the text field and it just scrolled out from the view.

Removed the the padding from the currency field as the currency is
already aligend to the center. The padding is not needed, the spacing
will be evenly distributed before and after the text.
  • Loading branch information
Ajnasz committed Dec 13, 2023
1 parent 1de5964 commit ae7c1fe
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/src/main/java/be/chvp/nanoledger/ui/add/AddActivity.kt
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,7 @@ fun PostingRow(
modifier = Modifier.weight(0.57f).padding(start = 4.dp, end = 2.dp),
)
if (currencyBeforeAmount ?: true) {
CurrencyField(index, posting, Modifier.weight(0.18f).padding(horizontal = 2.dp))
CurrencyField(index, posting, Modifier.weight(0.18f))
AmountField(
index,
posting,
Expand All @@ -385,7 +385,7 @@ fun PostingRow(
firstEmptyAmount,
Modifier.weight(0.25f).padding(horizontal = 2.dp),
)
CurrencyField(index, posting, Modifier.weight(0.18f).padding(start = 2.dp, end = 4.dp))
CurrencyField(index, posting, Modifier.weight(0.18f))
}
}
}
Expand Down

0 comments on commit ae7c1fe

Please sign in to comment.