Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
0b79d78
fix: align currency and calc widget with ios
piotr-iohk Apr 2, 2026
64e4eed
doc: changelog entry
piotr-iohk Apr 2, 2026
902e3f0
fix: sanitize calculator keyboard input
piotr-iohk Apr 3, 2026
1c9e0ee
Merge branch 'master' into fix/currency-widget-consistency-881
piotr-iohk Apr 3, 2026
c4d651e
fix: stabilize calculator widget input
piotr-iohk Apr 3, 2026
e57d0e8
Update app/src/main/java/to/bitkit/ui/screens/widgets/calculator/comp…
piotr-iohk Apr 3, 2026
ab7f801
Merge branch 'master' into fix/currency-widget-consistency-881
piotr-iohk Apr 7, 2026
d365996
Merge branch 'master' into fix/currency-widget-consistency-881
piotr-iohk Apr 7, 2026
044cd74
Merge branch 'master' into fix/currency-widget-consistency-881
jvsena42 Apr 22, 2026
6172c82
chore: lint
jvsena42 Apr 22, 2026
43eba3a
fix: use raw text for offset mapping wile still formatting from the s…
jvsena42 Apr 22, 2026
33eb984
fix: update fiat value on currency change
jvsena42 Apr 22, 2026
837ccf4
fix: remove duplicated changelog entry
jvsena42 Apr 23, 2026
3410fad
fix: clear cached input values on widget delete
jvsena42 Apr 23, 2026
0cdbc47
fix: add a guard for empty inputs
jvsena42 Apr 23, 2026
6cc693e
Merge branch 'master' into fix/currency-widget-consistency-881
jvsena42 Apr 27, 2026
230097e
Merge branch 'master' into fix/currency-widget-consistency-881
jvsena42 Apr 28, 2026
95f0beb
dix: fix changelog entry
jvsena42 Apr 29, 2026
a632a4e
Merge branch 'master' into fix/currency-widget-consistency-881
jvsena42 Apr 30, 2026
4efd824
Merge branch 'master' into fix/currency-widget-consistency-881
jvsena42 Apr 30, 2026
9805949
doc: migrate changelog entry to fragment changelog pattern
jvsena42 Apr 30, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion app/src/main/java/to/bitkit/data/WidgetsStore.kt
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,11 @@ class WidgetsStore @Inject constructor(
if (!store.data.first().widgets.map { it.type }.contains(type)) return

store.updateData { data ->
data.copy(widgets = data.widgets.filterNot { it.type == type })
val updated = data.copy(widgets = data.widgets.filterNot { it.type == type })
when (type) {
WidgetType.CALCULATOR -> updated.copy(calculatorValues = CalculatorValues())
else -> updated
}
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@ import kotlinx.serialization.Serializable

@Serializable
data class CalculatorValues(
val btcValue: String = "",
val btcValue: String = "10000",
val fiatValue: String = "",
)
Original file line number Diff line number Diff line change
Expand Up @@ -15,18 +15,19 @@ import androidx.compose.foundation.layout.width
import androidx.compose.material3.Icon
import androidx.compose.material3.MaterialTheme
import androidx.compose.runtime.Composable
import androidx.compose.runtime.LaunchedEffect
import androidx.compose.runtime.getValue
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.saveable.rememberSaveable
import androidx.compose.runtime.setValue
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier
import androidx.compose.ui.draw.clip
import androidx.compose.ui.focus.onFocusChanged
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.platform.testTag
import androidx.compose.ui.res.painterResource
import androidx.compose.ui.res.stringResource
import androidx.compose.ui.text.input.KeyboardType
import androidx.compose.ui.tooling.preview.Preview
import androidx.compose.ui.unit.dp
import androidx.hilt.lifecycle.viewmodel.compose.hiltViewModel
Expand All @@ -43,6 +44,7 @@ import to.bitkit.ui.utils.visualTransformation.BitcoinVisualTransformation
import to.bitkit.ui.utils.visualTransformation.CalculatorFormatter
import to.bitkit.ui.utils.visualTransformation.MonetaryVisualTransformation
import to.bitkit.viewmodels.CurrencyViewModel
import java.math.BigDecimal

@Composable
fun CalculatorCard(
Expand All @@ -55,34 +57,99 @@ fun CalculatorCard(
val calculatorValues by calculatorViewModel.calculatorValues.collectAsStateWithLifecycle()
var btcValue: String by rememberSaveable { mutableStateOf(calculatorValues.btcValue) }
var fiatValue: String by rememberSaveable { mutableStateOf(calculatorValues.fiatValue) }
val displayedBtcValue = btcValue.ifEmpty { calculatorValues.btcValue }
val displayedFiatValue = fiatValue

LaunchedEffect(
calculatorValues.btcValue,
calculatorValues.fiatValue,
currencyUiState.displayUnit,
currencyUiState.selectedCurrency,
Comment thread
jvsena42 marked this conversation as resolved.
) {
if (!shouldHydrateFiatFromStoredBtc(
storedBtcValue = calculatorValues.btcValue,
storedFiatValue = calculatorValues.fiatValue,
currentFiatValue = fiatValue,
displayUnit = currencyUiState.displayUnit,
)
) {
return@LaunchedEffect
}
val convertedFiat = CalculatorFormatter.convertBtcToFiat(
btcValue = calculatorValues.btcValue,
displayUnit = currencyUiState.displayUnit,
currencyViewModel = currencyViewModel,
).orEmpty()
if (convertedFiat.isEmpty()) {
return@LaunchedEffect
}
fiatValue = convertedFiat
calculatorViewModel.updateCalculatorValues(
fiatValue = convertedFiat,
btcValue = calculatorValues.btcValue,
)
}

LaunchedEffect(currencyUiState.selectedCurrency, currencyUiState.displayUnit) {
val sourceBtc = btcValue.ifEmpty { calculatorValues.btcValue }
if (sourceBtc.isEmpty() || isZeroBtcValue(sourceBtc, currencyUiState.displayUnit)) {
return@LaunchedEffect
}
val convertedFiat = CalculatorFormatter.convertBtcToFiat(
btcValue = sourceBtc,
displayUnit = currencyUiState.displayUnit,
currencyViewModel = currencyViewModel,
).orEmpty()
if (convertedFiat.isEmpty()) {
return@LaunchedEffect
}
fiatValue = convertedFiat
calculatorViewModel.updateCalculatorValues(
fiatValue = convertedFiat,
btcValue = sourceBtc,
)
}

CalculatorCardContent(
modifier = modifier,
showWidgetTitle = showWidgetTitle,
btcPrimaryDisplayUnit = currencyUiState.displayUnit,
btcValue = btcValue.ifEmpty { calculatorValues.btcValue },
onBtcChange = { newValue ->
btcValue = newValue
val convertedFiat = CalculatorFormatter.convertBtcToFiat(
btcValue = btcValue,
displayUnit = currencyUiState.displayUnit,
currencyViewModel = currencyViewModel
)
fiatValue = convertedFiat.orEmpty()
btcValue = displayedBtcValue,
onBtcChange = { rawValue ->
val sanitized = if (currencyUiState.displayUnit.isModern()) {
sanitizeIntegerInput(rawValue)
} else {
sanitizeDecimalInput(rawValue)
}
btcValue = sanitized
fiatValue = if (sanitized.isEmpty()) {
""
} else {
CalculatorFormatter.convertBtcToFiat(
btcValue = btcValue,
displayUnit = currencyUiState.displayUnit,
currencyViewModel = currencyViewModel,
).orEmpty()
}
calculatorViewModel.updateCalculatorValues(fiatValue = fiatValue, btcValue = btcValue)
},
fiatSymbol = currencyUiState.currencySymbol,
fiatName = currencyUiState.selectedCurrency,
fiatValue = fiatValue.ifEmpty { calculatorValues.fiatValue },
onFiatChange = { newValue ->
fiatValue = newValue
btcValue = CalculatorFormatter.convertFiatToBtc(
fiatValue = fiatValue,
displayUnit = currencyUiState.displayUnit,
currencyViewModel = currencyViewModel
)
fiatValue = displayedFiatValue,
onFiatChange = { rawValue ->
val sanitized = sanitizeDecimalInput(rawValue)
fiatValue = sanitized
btcValue = if (sanitized.isEmpty()) {
""
} else {
CalculatorFormatter.convertFiatToBtc(
fiatValue = fiatValue,
displayUnit = currencyUiState.displayUnit,
currencyViewModel = currencyViewModel,
)
}
calculatorViewModel.updateCalculatorValues(fiatValue = fiatValue, btcValue = btcValue)
}
},
)
}

Expand Down Expand Up @@ -115,14 +182,13 @@ fun CalculatorCardContent(

// Bitcoin input with visual transformation
CalculatorInput(
modifier = Modifier
.fillMaxWidth()
.onFocusChanged { focusState -> if (focusState.hasFocus) onBtcChange("") },
value = btcValue,
onValueChange = onBtcChange,
currencySymbol = BITCOIN_SYMBOL,
currencyName = stringResource(R.string.settings__general__unit_bitcoin),
visualTransformation = BitcoinVisualTransformation(btcPrimaryDisplayUnit)
keyboardType = if (btcPrimaryDisplayUnit.isModern()) KeyboardType.Number else KeyboardType.Decimal,
visualTransformation = BitcoinVisualTransformation(btcPrimaryDisplayUnit),
modifier = Modifier.fillMaxWidth()
)

VerticalSpacer(16.dp)
Expand All @@ -133,15 +199,40 @@ fun CalculatorCardContent(
onValueChange = onFiatChange,
currencySymbol = fiatSymbol,
currencyName = fiatName,
keyboardType = KeyboardType.Decimal,
visualTransformation = MonetaryVisualTransformation(decimalPlaces = 2),
modifier = Modifier
.fillMaxWidth()
.onFocusChanged { focusState -> if (focusState.hasFocus) onFiatChange("") }
modifier = Modifier.fillMaxWidth()
)
}
}
}

internal fun shouldHydrateFiatFromStoredBtc(
storedBtcValue: String,
storedFiatValue: String,
currentFiatValue: String,
displayUnit: BitcoinDisplayUnit,
): Boolean {
if (storedBtcValue.isEmpty()) {
return false
}
if (isZeroBtcValue(storedBtcValue, displayUnit)) {
return false
}
if (storedFiatValue.isNotEmpty()) {
return false
}
return currentFiatValue.isEmpty()
}

internal fun isZeroBtcValue(
btcValue: String,
displayUnit: BitcoinDisplayUnit,
): Boolean = when (displayUnit) {
BitcoinDisplayUnit.MODERN -> btcValue == "0"
BitcoinDisplayUnit.CLASSIC -> btcValue.toBigDecimalOrNull()?.compareTo(BigDecimal.ZERO) == 0
}

@Composable
private fun WidgetTitleRow() {
Row(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,11 @@ fun CalculatorInput(
currencySymbol: String,
currencyName: String,
modifier: Modifier = Modifier,
keyboardType: KeyboardType = KeyboardType.Number,
visualTransformation: VisualTransformation = VisualTransformation.None,
) {
val displayCurrencySymbol = currencySymbol.toCalculatorDisplaySymbol()

TextInput(
value = value,
singleLine = true,
Expand All @@ -44,11 +47,11 @@ fun CalculatorInput(
.background(color = Colors.Gray6, shape = CircleShape)
.size(32.dp)
) {
BodyMSB(currencySymbol, color = Colors.Brand)
BodyMSB(displayCurrencySymbol, color = Colors.Brand)
}
},
keyboardOptions = KeyboardOptions(
keyboardType = KeyboardType.Number
keyboardType = keyboardType,
),
suffix = { CaptionB(currencyName.uppercase(), color = Colors.Gray1) },
colors = AppTextFieldDefaults.noIndicatorColors.copy(
Expand All @@ -60,6 +63,26 @@ fun CalculatorInput(
)
}

internal fun sanitizeIntegerInput(raw: String): String =
raw.filter { it.isDigit() }

internal fun sanitizeDecimalInput(raw: String): String {
val filtered = raw.filter { it.isDigit() || it == '.' }
val dotIndex = filtered.indexOf('.')
if (dotIndex == -1) return filtered
return filtered.substring(0, dotIndex + 1) +
filtered.substring(dotIndex + 1).replace(".", "")
}

internal fun String.toCalculatorDisplaySymbol(): String {
val symbol = trim()
return if (symbol.length >= 3) {
symbol.take(1)
} else {
symbol
}
}

@Preview(showBackground = true)
@Composable
private fun Preview() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ fun LocalCurrencySettingsContent(
}
items(mostUsedRates) { rate ->
SettingsButtonRow(
title = "${rate.quote} (${rate.currencySymbol})",
title = formatCurrencyTitle(rate),
value = SettingsButtonValue.BooleanValue(selectedCurrency == rate.quote),
onClick = { onCurrencyClick(rate.quote) },
)
Expand All @@ -135,7 +135,7 @@ fun LocalCurrencySettingsContent(

items(otherCurrencies) { rate ->
SettingsButtonRow(
title = rate.quote,
title = formatCurrencyTitle(rate),
Comment thread
jvsena42 marked this conversation as resolved.
value = SettingsButtonValue.BooleanValue(selectedCurrency == rate.quote),
onClick = { onCurrencyClick(rate.quote) },
)
Expand All @@ -150,6 +150,11 @@ fun LocalCurrencySettingsContent(
}
}

private fun formatCurrencyTitle(rate: FxRate): String {
val symbol = rate.currencySymbol.trim()
return if (symbol.isNotEmpty()) "${rate.quote} ($symbol)" else rate.quote
}

@Preview(showSystemUi = true)
@Composable
private fun Preview() {
Expand Down
Loading
Loading