Improve widget handling

Author: theEvilReaper

lib/feature/sound/modal/section/base_integer_section_field.dart

@@ -0,0 +1,61 @@
+import 'package:flutter/material.dart';
+import 'package:stelaris/util/formatter/min_value_fomatter.dart';
+
+class BaseIntegerField extends StatefulWidget {
+  final String label;
+  final int initialValue;
+  final int minValue;
+  final ValueChanged<int> onChanged;
+
+  const BaseIntegerField({
+    required this.label,
+    required this.initialValue,
+    required this.minValue,
+    required this.onChanged,
+    super.key,
+  });
+
+  @override
+  State<BaseIntegerField> createState() => _BaseIntegerFieldState();
+}
+
+class _BaseIntegerFieldState extends State<BaseIntegerField> {
+  late final TextEditingController _controller;
+
+  @override
+  void initState() {
+    super.initState();
+    _controller = TextEditingController(text: widget.initialValue.toString());
+  }
+
+  @override
+  void dispose() {
+    _controller.dispose();
+    super.dispose();
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    return TextFormField(
+      controller: _controller,
+      keyboardType: TextInputType.number,
+      decoration: InputDecoration(
+        labelText: widget.label,
+        border: const OutlineInputBorder(),
+      ),
+      inputFormatters: [
+        MinValueFormatter(widget.minValue),
+      ],
+      validator: (v) {
+        if (v == null || v.isEmpty) return 'Enter a ${widget.label}';
+        final val = int.tryParse(v);
+        if (val == null) return 'Enter a valid integer';
+        return null;
+      },
+      onChanged: (v) {
+        final parsed = int.tryParse(v);
+        if (parsed != null) widget.onChanged(parsed);
+      },
+    );
+  }
+}

lib/feature/sound/modal/section/integer_fields_section.dart

@@ -0,0 +1,43 @@
+import 'package:flutter/material.dart';
+import 'package:stelaris/feature/sound/modal/section/base_section.dart';
+import 'package:stelaris/feature/sound/modal/section/base_integer_section_field.dart';
+
+class IntegerFieldsSection extends StatelessWidget {
+  final int weight;
+  final int attenuation;
+  final int minValue;
+  final ValueChanged<int> onWeightChanged;
+  final ValueChanged<int> onAttenuationChanged;
+
+  const IntegerFieldsSection({
+    required this.weight,
+    required this.attenuation,
+    required this.onWeightChanged,
+    required this.onAttenuationChanged,
+    this.minValue = 1,
+    super.key,
+  });
+
+  @override
+  Widget build(BuildContext context) {
+    return BaseSection(
+      child: Column(
+        children: [
+          BaseIntegerField(
+            label: 'Weight',
+            initialValue: weight,
+            minValue: minValue,
+            onChanged: onWeightChanged,
+          ),
+          const SizedBox(height: 12),
+          BaseIntegerField(
+            label: 'Attenuation Distance',
+            initialValue: attenuation,
+            minValue: minValue,
+            onChanged: onAttenuationChanged,
+          ),
+        ],
+      ),
+    );
+  }
+}

lib/feature/sound/modal/type/integer_fields_section.dart

@@ -20,7 +20,8 @@ class SwitchesSection extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     return BaseSection(
-      child: Column(
+      child: Row(
+        mainAxisSize: MainAxisSize.min,
         children: [
           _buildSwitchRow(
             context: context,
@@ -47,12 +48,11 @@ class SwitchesSection extends StatelessWidget {
     required ValueChanged<bool> onChanged,
   }) {
     return Row(
+      mainAxisSize: MainAxisSize.min,
       children: [
-        Expanded(
-          child: Text(
-            label,
-            style: Theme.of(context).textTheme.bodyLarge,
-          ),
+        Text(
+          label,
+          style: Theme.of(context).textTheme.bodyLarge,
         ),
         Switch(
           value: value,

lib/feature/sound/modal/section/sound_switch_section.dart renamed to lib/feature/sound/modal/type/sound_switch_section.dart

@@ -56,10 +56,9 @@ class _VolumeSectionState extends State<VolumeSection> {
                 _currentVolume = newVolume;
               });
             },
-            onChangeEnd:
-                widget.onVolumeFinalized, // Pass through the finalized callback
+            onChangeEnd: widget.onVolumeFinalized,
           ),
-          const SizedBox(height: 12), // Your existing spacer
+          const SizedBox(height: 12),
           SoundSliderRow(
             label: 'Pitch',
             value: _currentPitch,
@@ -68,8 +67,7 @@ class _VolumeSectionState extends State<VolumeSection> {
                 _currentPitch = newPitch;
               });
             },
-            onChangeEnd:
-                widget.onPitchFinalized, // Pass through the finalized callback
+            onChangeEnd: widget.onPitchFinalized,
           ),
         ],
       ),