fix: update theme

nank1ro · Nov 25, 2024 · 9e22fbc · 9e22fbc
1 parent 23c55f2
commit 9e22fbc
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 34 deletions.
diff --git a/lib/src/components/time_picker.dart b/lib/src/components/time_picker.dart
@@ -710,8 +710,9 @@ class _ShadTimePickerFieldState extends State<ShadTimePickerField> {
       height: 24 / 16,
     );
 
-    final effectivePlaceholderStyle =
-        defaultPlaceholderStyle.merge(controller.placeholderStyle);
+    final effectivePlaceholderStyle = defaultPlaceholderStyle
+        .merge(theme.timePickerTheme.placeholderStyle)
+        .merge(controller.placeholderStyle);
 
     final defaultLabelStyle = theme.textTheme.small.copyWith(fontSize: 12);
     final effectiveLabelStyle = defaultLabelStyle.merge(widget.labelStyle);
@@ -798,11 +799,6 @@ class ShadTimePickerTextEditingController extends TextEditingController {
     assert(
       !value.composing.isValid || !withComposing || value.isComposingRangeValid,
     );
-    // If the composing range is out of range for the current text, ignore it to
-    // preserve the tree integrity, otherwise in release mode a RangeError will
-    // be thrown and this EditableText will be built with a broken subtree.
-    final composingRegionOutOfRange =
-        !value.isComposingRangeValid || !withComposing;
 
     final theme = ShadTheme.of(context);
 
@@ -817,29 +813,12 @@ class ShadTimePickerTextEditingController extends TextEditingController {
     final intValue = int.tryParse(value.text);
     if (intValue == null) return const TextSpan();
 
-    if (composingRegionOutOfRange) {
-      return TextSpan(
-        style: style,
-        children: [
-          if (value.text.length == 1 && intValue < 10)
-            TextSpan(text: '0', style: effectivePlaceholderStyle),
-          TextSpan(text: text),
-        ],
-      );
-    }
-
-    final composingStyle =
-        style?.merge(const TextStyle(decoration: TextDecoration.underline)) ??
-            const TextStyle(decoration: TextDecoration.underline);
     return TextSpan(
       style: style,
-      children: <TextSpan>[
-        TextSpan(text: value.composing.textBefore(value.text)),
-        TextSpan(
-          style: composingStyle,
-          text: value.composing.textInside(value.text),
-        ),
-        TextSpan(text: value.composing.textAfter(value.text)),
+      children: [
+        if (value.text.length == 1 && intValue < 10)
+          TextSpan(text: '0', style: effectivePlaceholderStyle),
+        TextSpan(text: text),
       ],
     );
   }

diff --git a/lib/src/theme/themes/default_theme_no_secondary_border_variant.dart b/lib/src/theme/themes/default_theme_no_secondary_border_variant.dart
@@ -885,9 +885,9 @@ class ShadDefaultThemeNoSecondaryBorderVariant extends ShadThemeVariant {
         fontSize: 16,
         height: 24 / 16,
       ),
-      placeholderStyle: defaultTextTheme.large.copyWith(
-        fontWeight: FontWeight.w500,
-        color: colorScheme.foreground,
+      placeholderStyle: effectiveTextTheme.muted.copyWith(
+        fontSize: 16,
+        height: 24 / 16,
       ),
       labelStyle: defaultTextTheme.small.copyWith(fontSize: 12),
       fieldWidth: 58,

diff --git a/lib/src/theme/themes/default_theme_variant.dart b/lib/src/theme/themes/default_theme_variant.dart
@@ -858,9 +858,9 @@ class ShadDefaultThemeVariant extends ShadThemeVariant {
         fontSize: 16,
         height: 24 / 16,
       ),
-      placeholderStyle: defaultTextTheme.large.copyWith(
-        fontWeight: FontWeight.w500,
-        color: colorScheme.foreground,
+      placeholderStyle: effectiveTextTheme.muted.copyWith(
+        fontSize: 16,
+        height: 24 / 16,
       ),
       labelStyle: defaultTextTheme.small.copyWith(fontSize: 12),
       fieldWidth: 58,