Просмотр исходного кода

fix: 采纳 PR 审阅意见

- 移除滑条拖动过程中对隐藏输入框的冗余 textFieldState.edit 更新,
  输入模式切换时 LaunchedEffect 会重新初始化文本,拖动期间的写入
  从不可见(Gemini)
- 字号缩放数值标签改用 context.getString 格式化,数字格式跟随应用
  语言而非系统默认 locale(Gemini)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
5151561 3 месяцев назад
Родитель
Сommit
6ad98083c7

+ 1 - 2
app/src/main/java/io/legado/app/ui/config/themeConfig/ThemeConfigScreen.kt

@@ -346,10 +346,9 @@ fun ThemeConfigScreen(
                             context.toastOnUi(R.string.restart_to_apply)
                         }
                     )
-                    val fontScaleSummary = stringResource(R.string.font_scale_summary)
                     SliderSettingItem(
                         title = stringResource(R.string.font_scale),
-                        valueLabel = { fontScaleSummary.format(it / 10f) },
+                        valueLabel = { context.getString(R.string.font_scale_summary, it / 10f) },
                         value = fontScaleValue.floatValue,
                         defaultValue = 10f,
                         valueRange = 8f..16f,

+ 0 - 6
app/src/main/java/io/legado/app/ui/widget/components/settingItem/SliderSettingItem.kt

@@ -147,9 +147,6 @@ fun SliderSettingItem(
                                 value = sliderValue,
                                 onValueChange = {
                                     sliderValue = snap(it)
-                                    textFieldState.edit {
-                                        replace(0, length, format(sliderValue))
-                                    }
                                 },
                                 onValueChangeFinished = {
                                     onValueChange(sliderValue.coerceIn(valueRange))
@@ -232,9 +229,6 @@ fun SliderSettingItem(
                                 value = sliderValue,
                                 onValueChange = {
                                     sliderValue = snap(it)
-                                    textFieldState.edit {
-                                        replace(0, length, format(sliderValue))
-                                    }
                                 },
                                 onValueChangeFinished = {
                                     onValueChange(sliderValue.coerceIn(valueRange))