lib/screens/settings/components/shared/dialog_filter/widget_dialog_filter.dart

@@ -1,5 +1,4 @@
 import 'package:flutter/material.dart';
-import 'package:flutter/scheduler.dart';
 import 'package:lightmeter/generated/l10n.dart';
 import 'package:lightmeter/res/dimens.dart';
 
@@ -35,20 +34,18 @@ class _DialogFilterState<T> extends State<DialogFilter<T>> {
   bool get _hasAnySelected => checkboxValues.contains(true);
   bool get _hasAnyUnselected => checkboxValues.contains(false);
 
-  final ScrollController _scrollController = ScrollController();
+  late final ScrollController _scrollController;
 
   @override
   void initState() {
     super.initState();
-    SchedulerBinding.instance.addPostFrameCallback((_) {
-      int i = 0;
-      for (; i < checkboxValues.length; i++) {
-        if (checkboxValues[i]) {
-          break;
-        }
+    int i = 0;
+    for (; i < checkboxValues.length; i++) {
+      if (checkboxValues[i]) {
+        break;
       }
-      _scrollController.jumpTo((Dimens.grid56 * i).clamp(0, _scrollController.position.maxScrollExtent));
-    });
+    }
+    _scrollController = ScrollController(initialScrollOffset: Dimens.grid56 * i);
   }
 
   @override
@@ -64,80 +61,79 @@ class _DialogFilterState<T> extends State<DialogFilter<T>> {
       titlePadding: Dimens.dialogIconTitlePadding,
       title: Text(widget.title),
       contentPadding: EdgeInsets.zero,
-      content: SizedBox(
-        width: double.maxFinite,
-        child: Column(
-          children: [
-            Padding(
-              padding: Dimens.dialogIconTitlePadding,
-              child: Text(widget.description),
-            ),
-            const Divider(),
-            Expanded(
-              child: SingleChildScrollView(
-                controller: _scrollController,
-                child: Column(
-                  crossAxisAlignment: CrossAxisAlignment.stretch,
-                  mainAxisSize: MainAxisSize.min,
-                  children: List.generate(
-                    widget.values.length,
-                    (index) => CheckboxListTile(
-                      value: checkboxValues[index],
-                      controlAffinity: ListTileControlAffinity.leading,
-                      title: Text(
-                        widget.titleAdapter(context, widget.values[index]),
-                        style: Theme.of(context).textTheme.bodyLarge,
-                      ),
-                      onChanged: (value) {
-                        if (value != null) {
-                          setState(() {
-                            checkboxValues[index] = value;
-                          });
-                        }
-                      },
+      content: Column(
+        children: [
+          Padding(
+            padding: Dimens.dialogIconTitlePadding,
+            child: Text(widget.description),
+          ),
+          const Divider(),
+          Expanded(
+            child: SingleChildScrollView(
+              controller: _scrollController,
+              child: Column(
+                crossAxisAlignment: CrossAxisAlignment.stretch,
+                mainAxisSize: MainAxisSize.min,
+                children: List.generate(
+                  widget.values.length,
+                  (index) => CheckboxListTile(
+                    value: checkboxValues[index],
+                    controlAffinity: ListTileControlAffinity.leading,
+                    title: Text(
+                      widget.titleAdapter(context, widget.values[index]),
+                      style: Theme.of(context).textTheme.bodyLarge,
                     ),
+                    onChanged: (value) {
+                      if (value != null) {
+                        setState(() {
+                          checkboxValues[index] = value;
+                        });
+                      }
+                    },
                   ),
                 ),
               ),
             ),
-            const Divider(),
-            Padding(
-              padding: Dimens.dialogActionsPadding,
-              child: Row(
-                children: [
-                  SizedBox(
-                    width: 40,
-                    child: IconButton(
-                      padding: EdgeInsets.zero,
-                      icon: Icon(_hasAnyUnselected ? Icons.select_all : Icons.deselect),
-                      onPressed: _toggleAll,
-                      tooltip: _hasAnyUnselected ? S.of(context).tooltipSelectAll : S.of(context).tooltipDesecelectAll,
-                    ),
+          ),
+          const Divider(),
+          Padding(
+            padding: Dimens.dialogActionsPadding,
+            child: Row(
+              children: [
+                SizedBox(
+                  width: 40,
+                  child: IconButton(
+                    padding: EdgeInsets.zero,
+                    icon: Icon(_hasAnyUnselected ? Icons.select_all : Icons.deselect),
+                    onPressed: _toggleAll,
+                    tooltip: _hasAnyUnselected
+                        ? S.of(context).tooltipSelectAll
+                        : S.of(context).tooltipDesecelectAll,
                   ),
-                  const Spacer(),
-                  TextButton(
-                    onPressed: Navigator.of(context).pop,
-                    child: Text(S.of(context).cancel),
-                  ),
-                  TextButton(
-                    onPressed: _hasAnySelected
-                        ? () {
-                            final List<T> selectedValues = [];
-                            for (int i = 0; i < widget.values.length; i++) {
-                              if (checkboxValues[i]) {
-                                selectedValues.add(widget.values[i]);
-                              }
+                ),
+                const Spacer(),
+                TextButton(
+                  onPressed: Navigator.of(context).pop,
+                  child: Text(S.of(context).cancel),
+                ),
+                TextButton(
+                  onPressed: _hasAnySelected
+                      ? () {
+                          final List<T> selectedValues = [];
+                          for (int i = 0; i < widget.values.length; i++) {
+                            if (checkboxValues[i]) {
+                              selectedValues.add(widget.values[i]);
                             }
-                            Navigator.of(context).pop(selectedValues);
                           }
-                        : null,
-                    child: Text(S.of(context).save),
-                  ),
-                ],
-              ),
-            )
-          ],
-        ),
+                          Navigator.of(context).pop(selectedValues);
+                        }
+                      : null,
+                  child: Text(S.of(context).save),
+                ),
+              ],
+            ),
+          )
+        ],
       ),
     );
   }

lib/screens/settings/components/shared/dialog_picker/widget_dialog_picker.dart

@@ -32,28 +32,24 @@ class _DialogPickerState<T> extends State<DialogPicker<T>> {
       titlePadding: Dimens.dialogIconTitlePadding,
       title: Text(widget.title),
       contentPadding: EdgeInsets.zero,
-      content: SizedBox(
-        width: double.maxFinite,
-        child: Column(
-          crossAxisAlignment: CrossAxisAlignment.stretch,
-          mainAxisSize: MainAxisSize.min,
-          children: widget.values
-              .map(
-                (e) => RadioListTile(
-                  value: e,
-                  groupValue: _selected,
-                  title: Text(widget.titleAdapter(context, e)),
-                  onChanged: (T? value) {
-                    if (value != null) {
-                      setState(() {
-                        _selected = value;
-                      });
-                    }
-                  },
-                ),
-              )
-              .toList(),
-        ),
+      content: Column(
+        mainAxisSize: MainAxisSize.min,
+        children: widget.values
+            .map(
+              (e) => RadioListTile(
+                value: e,
+                groupValue: _selected,
+                title: Text(widget.titleAdapter(context, e)),
+                onChanged: (T? value) {
+                  if (value != null) {
+                    setState(() {
+                      _selected = value;
+                    });
+                  }
+                },
+              ),
+            )
+            .toList(),
       ),
       actionsPadding: Dimens.dialogActionsPadding,
       actions: [

lib/screens/settings/components/shared/dialog_range_picker/widget_dialog_picker_range.dart

@@ -36,50 +36,47 @@ class _DialogRangePickerState<T extends PhotographyValue> extends State<DialogRa
       titlePadding: Dimens.dialogIconTitlePadding,
       title: Text(widget.title),
       contentPadding: EdgeInsets.zero,
-      content: SizedBox(
-        width: double.maxFinite,
-        child: Column(
-          mainAxisSize: MainAxisSize.min,
-          children: [
-            Padding(
-              padding: Dimens.dialogIconTitlePadding,
-              child: Text(widget.description),
-            ),
-            Padding(
-              padding: const EdgeInsets.symmetric(horizontal: Dimens.paddingL),
-              child: DefaultTextStyle(
-                style: Theme.of(context).textTheme.bodyLarge!,
-                child: Row(
-                  mainAxisAlignment: MainAxisAlignment.spaceBetween,
-                  children: [
-                    Text(widget.values[_start].toString()),
-                    Text(widget.values[_end].toString()),
-                  ],
-                ),
+      content: Column(
+        mainAxisSize: MainAxisSize.min,
+        children: [
+          Padding(
+            padding: Dimens.dialogIconTitlePadding,
+            child: Text(widget.description),
+          ),
+          Padding(
+            padding: const EdgeInsets.symmetric(horizontal: Dimens.paddingL),
+            child: DefaultTextStyle(
+              style: Theme.of(context).textTheme.bodyLarge!,
+              child: Row(
+                mainAxisAlignment: MainAxisAlignment.spaceBetween,
+                children: [
+                  Text(widget.values[_start].toString()),
+                  Text(widget.values[_end].toString()),
+                ],
               ),
             ),
-            Row(
-              children: [
-                Expanded(
-                  child: RangeSlider(
-                    values: RangeValues(
-                      _start.toDouble(),
-                      _end.toDouble(),
-                    ),
-                    max: widget.values.length.toDouble() - 1,
-                    divisions: widget.values.length - 1,
-                    onChanged: (value) {
-                      setState(() {
-                        _start = value.start.round();
-                        _end = value.end.round();
-                      });
-                    },
+          ),
+          Row(
+            children: [
+              Expanded(
+                child: RangeSlider(
+                  values: RangeValues(
+                    _start.toDouble(),
+                    _end.toDouble(),
                   ),
+                  max: widget.values.length.toDouble() - 1,
+                  divisions: widget.values.length - 1,
+                  onChanged: (value) {
+                    setState(() {
+                      _start = value.start.round();
+                      _end = value.end.round();
+                    });
+                  },
                 ),
-              ],
-            ),
-          ],
-        ),
+              ),
+            ],
+          ),
+        ],
       ),
       actionsPadding: Dimens.dialogActionsPadding,
       actions: [

pubspec.yaml

@@ -1,7 +1,7 @@
 name: lightmeter
 description: Lightmeter app inspired by Material 3 design system.
 publish_to: "none"
-version: 0.16.0+45
+version: 0.15.3+44
 
 environment:
   sdk: ">=3.0.0 <4.0.0"