update device_gesture_wrapper.dart (#227)

components/in_app_review_platform_interface/lib/in_app_review_platform_interface.dart

@@ -12,8 +12,7 @@ import 'package:plugin_platform_interface/plugin_platform_interface.dart';
 abstract class InAppReviewPlatform extends PlatformInterface {
   InAppReviewPlatform() : super(token: _token);
 
-  static InAppReviewPlatform _instance =
-      MethodChannelInAppReview() as InAppReviewPlatform;
+  static InAppReviewPlatform _instance = MethodChannelInAppReview();
 
   static final Object _token = Object();
 

ios/Podfile.lock

@@ -156,4 +156,4 @@ SPEC CHECKSUMS:
 
 PODFILE CHECKSUM: d28e9a1c7bee335d05ddd795703aad5bf05bb937
 
-COCOAPODS: 1.12.0
+COCOAPODS: 1.11.3

lib/config/constants.dart

@@ -78,3 +78,15 @@ abstract class RegExpConstants {
   static const String linkSuffix = r'(\)|]|,|\*)(.)*$';
   static const String number = '[0-9]+';
 }
+
+abstract class Durations {
+  static const Duration ms100 = Duration(milliseconds: 100);
+  static const Duration ms200 = Duration(milliseconds: 200);
+  static const Duration ms300 = Duration(milliseconds: 300);
+  static const Duration ms400 = Duration(milliseconds: 400);
+  static const Duration ms500 = Duration(milliseconds: 500);
+  static const Duration ms600 = Duration(milliseconds: 600);
+  static const Duration oneSecond = Duration(seconds: 1);
+  static const Duration twoSeconds = Duration(seconds: 2);
+  static const Duration tenSeconds = Duration(seconds: 10);
+}

lib/cubits/comments/comments_cubit.dart

@@ -5,6 +5,7 @@ import 'package:bloc/bloc.dart';
 import 'package:collection/collection.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/foundation.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/config/locator.dart';
 import 'package:hacki/cubits/cubits.dart';
 import 'package:hacki/main.dart';
@@ -348,6 +349,7 @@ class CommentsCubit extends Cubit<CommentsState> {
     init(useCommentCache: true);
   }
 
+  /// Jump to next root level comment.
   void jump(
     ItemScrollController itemScrollController,
     ItemPositionsListener itemPositionsListener,
@@ -378,13 +380,14 @@ class CommentsCubit extends Cubit<CommentsState> {
         itemScrollController.scrollTo(
           index: i + 1,
           alignment: 0.15,
-          duration: const Duration(milliseconds: 400),
+          duration: Durations.ms400,
         );
         return;
       }
     }
   }
 
+  /// Jump to previous root level comment.
   void jumpUp(
     ItemScrollController itemScrollController,
     ItemPositionsListener itemPositionsListener,
@@ -416,7 +419,7 @@ class CommentsCubit extends Cubit<CommentsState> {
         itemScrollController.scrollTo(
           index: i + 1,
           alignment: 0.15,
-          duration: const Duration(milliseconds: 400),
+          duration: Durations.ms400,
         );
         return;
       }

lib/cubits/edit/edit_cubit.dart

@@ -1,4 +1,5 @@
 import 'package:equatable/equatable.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/config/locator.dart';
 import 'package:hacki/extensions/extensions.dart';
 import 'package:hacki/models/models.dart';
@@ -11,7 +12,7 @@ part 'edit_state.dart';
 class EditCubit extends HydratedCubit<EditState> {
   EditCubit({DraftCache? draftCache})
       : _draftCache = draftCache ?? locator.get<DraftCache>(),
-        _debouncer = Debouncer(delay: const Duration(seconds: 1)),
+        _debouncer = Debouncer(delay: Durations.oneSecond),
         super(const EditState.init());
 
   final DraftCache _draftCache;

lib/cubits/notification/notification_cubit.dart

@@ -4,6 +4,7 @@ import 'dart:math';
 import 'package:bloc/bloc.dart';
 import 'package:equatable/equatable.dart';
 import 'package:hacki/blocs/blocs.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/config/locator.dart';
 import 'package:hacki/cubits/cubits.dart';
 import 'package:hacki/models/models.dart';
@@ -31,7 +32,7 @@ class NotificationCubit extends Cubit<NotificationState> {
       if (authState.isLoggedIn && authState.username != _username) {
         // Get the user setting.
         if (_preferenceCubit.state.notificationEnabled) {
-          Future<void>.delayed(const Duration(seconds: 2), init);
+          Future<void>.delayed(Durations.twoSeconds, init);
         }
 
         // Listen for setting changes in the future.

lib/cubits/post/post_cubit.dart

@@ -20,9 +20,6 @@ class PostCubit extends Cubit<PostState> {
       text: text,
     );
 
-    // final successful =
-    //     await Future<bool>.delayed(const Duration(seconds: 2), () => true);
-
     if (successful) {
       emit(state.copyWith(status: PostStatus.successful));
     } else {

lib/cubits/preference/preference_state.dart

@@ -68,6 +68,8 @@ class PreferenceState extends Equatable {
 
   bool get swipeGestureEnabled => _isOn<SwipeGesturePreference>();
 
+  bool get autoScrollEnabled => _isOn<AutoScrollModePreference>();
+
   List<StoryType> get tabs {
     final String result =
         preferences.singleWhereType<TabOrderPreference>().val.toString();

lib/cubits/search/search_state.dart

@@ -48,3 +48,11 @@ class SearchState extends Equatable {
         params,
       ];
 }
+
+extension SearchStateExtension on SearchState {
+  bool get showDateRangeShortcutChips {
+    return hasDateFilter &&
+        dateFilter?.startTime != null &&
+        dateFilter?.endTime != null;
+  }
+}

lib/main.dart

@@ -7,6 +7,7 @@ import 'package:equatable/equatable.dart';
 import 'package:feature_discovery/feature_discovery.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
+import 'package:flutter/scheduler.dart';
 import 'package:flutter/services.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_local_notifications/flutter_local_notifications.dart';
@@ -268,8 +269,8 @@ class HackiApp extends StatelessWidget {
               AsyncSnapshot<AdaptiveThemeMode?> snapshot,
             ) {
               final AdaptiveThemeMode? mode = snapshot.data;
-              ThemeUtil.updateAndroidStatusBarSetting(
-                Theme.of(context).brightness,
+              ThemeUtil.updateStatusBarSetting(
+                SchedulerBinding.instance.platformDispatcher.platformBrightness,
                 mode,
               );
               return BlocBuilder<PreferenceCubit, PreferenceState>(

lib/models/preference.dart

@@ -30,6 +30,7 @@ abstract class Preference<T> extends Equatable with SettingsDisplayable {
       const StoryUrlModePreference(),
       const NotificationModePreference(),
       const SwipeGesturePreference(),
+      const AutoScrollModePreference(),
       const CollapseModePreference(),
       const ReaderModePreference(),
       const MarkReadStoriesModePreference(),
@@ -54,12 +55,13 @@ const bool _notificationModeDefaultValue = true;
 const bool _swipeGestureModeDefaultValue = false;
 const bool _displayModeDefaultValue = true;
 const bool _eyeCandyModeDefaultValue = false;
-const bool _trueDarkModeDefaultValue = false;
+const bool _trueDarkModeDefaultValue = true;
 const bool _readerModeDefaultValue = true;
 const bool _markReadStoriesModeDefaultValue = true;
 const bool _metadataModeDefaultValue = true;
 const bool _storyUrlModeDefaultValue = true;
 const bool _collapseModeDefaultValue = true;
+const bool _autoScrollModeDefaultValue = true;
 final int _fetchModeDefaultValue = FetchMode.eager.index;
 final int _commentsOrderDefaultValue = CommentsOrder.natural.index;
 final int _fontSizeDefaultValue = FontSize.regular.index;
@@ -127,6 +129,26 @@ class CollapseModePreference extends BooleanPreference {
       '''if disabled, tap on the top of comment tile to collapse.''';
 }
 
+class AutoScrollModePreference extends BooleanPreference {
+  const AutoScrollModePreference({bool? val})
+      : super(val: val ?? _autoScrollModeDefaultValue);
+
+  @override
+  AutoScrollModePreference copyWith({required bool? val}) {
+    return AutoScrollModePreference(val: val);
+  }
+
+  @override
+  String get key => 'autoScrollMode';
+
+  @override
+  String get title => 'Auto-scroll on collapsing';
+
+  @override
+  String get subtitle =>
+      '''automatically scroll to next comment when you collapse a comment.''';
+}
+
 /// The value deciding whether or not the story
 /// tile should display link preview. Defaults to true.
 class DisplayModePreference extends BooleanPreference {

lib/screens/home/home_screen.dart

@@ -57,7 +57,7 @@ class _HomeScreenState extends State<HomeScreen>
         DeviceScreenType.mobile) {
       locator.get<Logger>().i('resetting comments in CommentCache');
       Future<void>.delayed(
-        const Duration(milliseconds: 500),
+        Durations.ms500,
         locator.get<CommentCache>().resetComments,
       );
     }

lib/screens/home/widgets/tablet_home_screen.dart

@@ -1,5 +1,6 @@
 import 'package:flutter/material.dart' hide Badge;
 import 'package:flutter_bloc/flutter_bloc.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/cubits/cubits.dart';
 import 'package:hacki/screens/screens.dart';
 import 'package:hacki/screens/widgets/widgets.dart';
@@ -36,7 +37,7 @@ class TabletHomeScreen extends StatelessWidget {
                   top: Dimens.zero,
                   bottom: Dimens.zero,
                   width: homeScreenWidth,
-                  duration: const Duration(milliseconds: 300),
+                  duration: Durations.ms300,
                   curve: Curves.elasticOut,
                   child: homeScreen,
                 ),
@@ -52,7 +53,7 @@ class TabletHomeScreen extends StatelessWidget {
                   top: Dimens.zero,
                   bottom: Dimens.zero,
                   left: state.expanded ? Dimens.zero : homeScreenWidth,
-                  duration: const Duration(milliseconds: 300),
+                  duration: Durations.ms300,
                   curve: Curves.elasticOut,
                   child: const _TabletStoryView(),
                 ),

lib/screens/item/item_screen.dart

@@ -153,9 +153,9 @@ class _ItemScreenState extends State<ItemScreen> with RouteAware {
   );
   final GlobalKey fontSizeIconButtonKey = GlobalKey();
 
-  static const Duration _storyLinkTapThrottleDelay = Duration(seconds: 2);
+  static const Duration _storyLinkTapThrottleDelay = Durations.twoSeconds;
   static const Duration _featureDiscoveryDismissThrottleDelay =
-      Duration(seconds: 1);
+      Durations.oneSecond;
 
   @override
   void didPop() {

lib/screens/item/widgets/main_view.dart

@@ -139,6 +139,7 @@ class MainView extends StatelessWidget {
                           },
                           onMoreTapped: onMoreTapped,
                           onRightMoreTapped: onRightMoreTapped,
+                          itemScrollController: itemScrollController,
                         ),
                       );
                     },

lib/screens/item/widgets/reply_box.dart

@@ -2,6 +2,7 @@ import 'package:clipboard/clipboard.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_feather_icons/flutter_feather_icons.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/cubits/cubits.dart';
 import 'package:hacki/extensions/extensions.dart';
 import 'package:hacki/models/item/item.dart';
@@ -60,7 +61,7 @@ class _ReplyBoxState extends State<ReplyBox> {
               ),
               child: AnimatedContainer(
                 height: expanded ? expandedHeight : _collapsedHeight,
-                duration: const Duration(milliseconds: 200),
+                duration: Durations.ms200,
                 decoration: BoxDecoration(
                   boxShadow: <BoxShadow>[
                     if (!context.read<SplitViewCubit>().state.enabled)
@@ -79,7 +80,7 @@ class _ReplyBoxState extends State<ReplyBox> {
                         ),
                       AnimatedContainer(
                         height: expanded ? Dimens.pt36 : Dimens.zero,
-                        duration: const Duration(milliseconds: 200),
+                        duration: Durations.ms200,
                       ),
                       Row(
                         children: <Widget>[
@@ -107,7 +108,7 @@ class _ReplyBoxState extends State<ReplyBox> {
                                 AnimatedOpacity(
                                   opacity:
                                       expanded ? NumSwitch.on : NumSwitch.off,
-                                  duration: const Duration(milliseconds: 300),
+                                  duration: Durations.ms300,
                                   child: IconButton(
                                     key: const Key('quote'),
                                     icon: const Icon(

lib/screens/profile/profile_screen.dart

@@ -29,7 +29,7 @@ class _ProfileScreenState extends State<ProfileScreen>
   final RefreshController refreshControllerFav = RefreshController();
   final RefreshController refreshControllerNotification = RefreshController();
   final ScrollController scrollController = ScrollController();
-  final Throttle throttle = Throttle(delay: const Duration(seconds: 2));
+  final Throttle throttle = Throttle(delay: Durations.twoSeconds);
 
   PageType pageType = PageType.notification;
 

lib/screens/profile/widgets/settings.dart

@@ -3,6 +3,7 @@ import 'dart:io';
 import 'package:adaptive_theme/adaptive_theme.dart';
 import 'package:clipboard/clipboard.dart';
 import 'package:flutter/material.dart';
+import 'package:flutter/scheduler.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_cache_manager/flutter_cache_manager.dart';
 import 'package:flutter_email_sender/flutter_email_sender.dart';
@@ -404,8 +405,9 @@ class _SettingsState extends State<Settings> {
         AdaptiveTheme.of(context).setSystem();
     }
 
-    final Brightness brightness = Theme.of(context).brightness;
-    ThemeUtil.updateAndroidStatusBarSetting(brightness, val);
+    final Brightness brightness =
+        SchedulerBinding.instance.platformDispatcher.platformBrightness;
+    ThemeUtil.updateStatusBarSetting(brightness, val);
   }
 
   void showClearCacheDialog() {

lib/screens/search/search_screen.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
-import 'package:flutter/rendering.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_fadein/flutter_fadein.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/cubits/cubits.dart';
 import 'package:hacki/extensions/extensions.dart';
 import 'package:hacki/models/models.dart';
@@ -30,31 +30,9 @@ class SearchScreen extends StatefulWidget {
 class _SearchScreenState extends State<SearchScreen> {
   final RefreshController refreshController = RefreshController();
   final ScrollController scrollController = ScrollController();
-  final Debouncer debouncer = Debouncer(delay: const Duration(seconds: 1));
-  bool showChips = true;
-  bool shouldOffStageChips = false;
+  final Debouncer debouncer = Debouncer(delay: Durations.oneSecond);
 
-  static const Duration chipsAnimationDuration = Duration(milliseconds: 300);
-
-  @override
-  void initState() {
-    super.initState();
-    scrollController.addListener(() {
-      if (scrollController.position.userScrollDirection ==
-              ScrollDirection.reverse &&
-          showChips) {
-        setState(() {
-          showChips = false;
-        });
-      } else if (scrollController.position.userScrollDirection ==
-              ScrollDirection.forward &&
-          !showChips) {
-        setState(() {
-          showChips = true;
-        });
-      }
-    });
-  }
+  static const Duration chipsAnimationDuration = Durations.ms300;
 
   @override
   void dispose() {
@@ -107,83 +85,13 @@ class _SearchScreenState extends State<SearchScreen> {
                   ),
                   AnimatedCrossFade(
                     duration: chipsAnimationDuration,
-                    crossFadeState: showChips
+                    crossFadeState: state.showDateRangeShortcutChips
                         ? CrossFadeState.showSecond
                         : CrossFadeState.showFirst,
                     firstChild: SizedBox.fromSize(),
                     secondChild: Column(
                       mainAxisSize: MainAxisSize.min,
                       children: <Widget>[
-                        if (state.hasDateFilter &&
-                            state.dateFilter?.startTime != null &&
-                            state.dateFilter?.endTime != null)
-                          SingleChildScrollView(
-                            scrollDirection: Axis.horizontal,
-                            child: Row(
-                              children: <Widget>[
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                DateTimeShortcutChip.dayBefore(
-                                  onDateTimeRangeUpdated: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                  startDate: state.dateFilter!.startTime!,
-                                  endDate: state.dateFilter!.endTime!,
-                                ),
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                DateTimeShortcutChip.dayAfter(
-                                  onDateTimeRangeUpdated: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                  startDate: state.dateFilter!.startTime!,
-                                  endDate: state.dateFilter!.endTime!,
-                                ),
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                DateTimeShortcutChip.weekBefore(
-                                  onDateTimeRangeUpdated: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                  startDate: state.dateFilter!.startTime!,
-                                  endDate: state.dateFilter!.endTime!,
-                                ),
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                DateTimeShortcutChip.weekAfter(
-                                  onDateTimeRangeUpdated: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                  startDate: state.dateFilter!.startTime!,
-                                  endDate: state.dateFilter!.endTime!,
-                                ),
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                DateTimeShortcutChip.monthBefore(
-                                  onDateTimeRangeUpdated: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                  startDate: state.dateFilter!.startTime!,
-                                  endDate: state.dateFilter!.endTime!,
-                                ),
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                DateTimeShortcutChip.monthAfter(
-                                  onDateTimeRangeUpdated: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                  startDate: state.dateFilter!.startTime!,
-                                  endDate: state.dateFilter!.endTime!,
-                                ),
-                              ],
-                            ),
-                          ),
                         SingleChildScrollView(
                           scrollDirection: Axis.horizontal,
                           child: Row(
@@ -191,78 +99,143 @@ class _SearchScreenState extends State<SearchScreen> {
                               const SizedBox(
                                 width: Dimens.pt8,
                               ),
-                              DateTimeRangeFilterChip(
-                                filter: state.dateFilter,
-                                initialStartDate: state.dateFilter?.startTime,
-                                initialEndDate: state.dateFilter?.endTime,
+                              DateTimeShortcutChip.dayBefore(
                                 onDateTimeRangeUpdated: context
                                     .read<SearchCubit>()
                                     .onDateTimeRangeUpdated,
-                                onDateTimeRangeRemoved: context
+                                startDate: state.dateFilter?.startTime,
+                                endDate: state.dateFilter?.endTime,
+                              ),
+                              const SizedBox(
+                                width: Dimens.pt8,
+                              ),
+                              DateTimeShortcutChip.dayAfter(
+                                onDateTimeRangeUpdated: context
                                     .read<SearchCubit>()
-                                    .removeFilter<DateTimeRangeFilter>,
+                                    .onDateTimeRangeUpdated,
+                                startDate: state.dateFilter?.startTime,
+                                endDate: state.dateFilter?.endTime,
                               ),
                               const SizedBox(
                                 width: Dimens.pt8,
                               ),
-                              PostedByFilterChip(
-                                filter: state.params.get<PostedByFilter>(),
-                                onChanged: context
+                              DateTimeShortcutChip.weekBefore(
+                                onDateTimeRangeUpdated: context
                                     .read<SearchCubit>()
-                                    .onPostedByChanged,
+                                    .onDateTimeRangeUpdated,
+                                startDate: state.dateFilter?.startTime,
+                                endDate: state.dateFilter?.endTime,
                               ),
                               const SizedBox(
                                 width: Dimens.pt8,
                               ),
-                              CustomChip(
-                                onSelected: (_) =>
-                                    context.read<SearchCubit>().onSortToggled(),
-                                selected: state.params.sorted,
-                                label: '''newest first''',
+                              DateTimeShortcutChip.weekAfter(
+                                onDateTimeRangeUpdated: context
+                                    .read<SearchCubit>()
+                                    .onDateTimeRangeUpdated,
+                                startDate: state.dateFilter?.startTime,
+                                endDate: state.dateFilter?.endTime,
                               ),
                               const SizedBox(
                                 width: Dimens.pt8,
                               ),
-                              for (final CustomDateTimeRange range
-                                  in CustomDateTimeRange.values) ...<Widget>[
-                                CustomRangeFilterChip(
-                                  range: range,
-                                  onTap: context
-                                      .read<SearchCubit>()
-                                      .onDateTimeRangeUpdated,
-                                ),
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                              ],
+                              DateTimeShortcutChip.monthBefore(
+                                onDateTimeRangeUpdated: context
+                                    .read<SearchCubit>()
+                                    .onDateTimeRangeUpdated,
+                                startDate: state.dateFilter?.startTime,
+                                endDate: state.dateFilter?.endTime,
+                              ),
+                              const SizedBox(
+                                width: Dimens.pt8,
+                              ),
+                              DateTimeShortcutChip.monthAfter(
+                                onDateTimeRangeUpdated: context
+                                    .read<SearchCubit>()
+                                    .onDateTimeRangeUpdated,
+                                startDate: state.dateFilter?.startTime,
+                                endDate: state.dateFilter?.endTime,
+                              ),
                             ],
                           ),
                         ),
-                        SingleChildScrollView(
-                          scrollDirection: Axis.horizontal,
-                          child: Row(
-                            children: <Widget>[
-                              for (final TypeTagFilter filter
-                                  in TypeTagFilter.all) ...<Widget>[
-                                const SizedBox(
-                                  width: Dimens.pt8,
-                                ),
-                                CustomChip(
-                                  onSelected: (_) => context
-                                      .read<SearchCubit>()
-                                      .onToggled(filter),
-                                  selected: context
-                                          .read<SearchCubit>()
-                                          .state
-                                          .params
-                                          .get<TypeTagFilter>() ==
-                                      filter,
-                                  label: filter.query,
-                                ),
-                              ],
-                            ],
-                          ),
+                      ],
+                    ),
+                  ),
+                  SingleChildScrollView(
+                    scrollDirection: Axis.horizontal,
+                    child: Row(
+                      children: <Widget>[
+                        const SizedBox(
+                          width: Dimens.pt8,
                         ),
+                        DateTimeRangeFilterChip(
+                          filter: state.dateFilter,
+                          initialStartDate: state.dateFilter?.startTime,
+                          initialEndDate: state.dateFilter?.endTime,
+                          onDateTimeRangeUpdated: context
+                              .read<SearchCubit>()
+                              .onDateTimeRangeUpdated,
+                          onDateTimeRangeRemoved: context
+                              .read<SearchCubit>()
+                              .removeFilter<DateTimeRangeFilter>,
+                        ),
+                        const SizedBox(
+                          width: Dimens.pt8,
+                        ),
+                        PostedByFilterChip(
+                          filter: state.params.get<PostedByFilter>(),
+                          onChanged:
+                              context.read<SearchCubit>().onPostedByChanged,
+                        ),
+                        const SizedBox(
+                          width: Dimens.pt8,
+                        ),
+                        CustomChip(
+                          onSelected: (_) =>
+                              context.read<SearchCubit>().onSortToggled(),
+                          selected: state.params.sorted,
+                          label: '''newest first''',
+                        ),
+                        const SizedBox(
+                          width: Dimens.pt8,
+                        ),
+                        for (final CustomDateTimeRange range
+                            in CustomDateTimeRange.values) ...<Widget>[
+                          CustomRangeFilterChip(
+                            range: range,
+                            onTap: context
+                                .read<SearchCubit>()
+                                .onDateTimeRangeUpdated,
+                          ),
+                          const SizedBox(
+                            width: Dimens.pt8,
+                          ),
+                        ],
+                      ],
+                    ),
+                  ),
+                  SingleChildScrollView(
+                    scrollDirection: Axis.horizontal,
+                    child: Row(
+                      children: <Widget>[
+                        for (final TypeTagFilter filter
+                            in TypeTagFilter.all) ...<Widget>[
+                          const SizedBox(
+                            width: Dimens.pt8,
+                          ),
+                          CustomChip(
+                            onSelected: (_) =>
+                                context.read<SearchCubit>().onToggled(filter),
+                            selected: context
+                                    .read<SearchCubit>()
+                                    .state
+                                    .params
+                                    .get<TypeTagFilter>() ==
+                                filter,
+                            label: filter.query,
+                          ),
+                        ],
                       ],
                     ),
                   ),

lib/screens/search/widgets/date_time_shortcut_chip.dart

@@ -68,8 +68,8 @@ class DateTimeShortcutChip extends StatelessWidget {
         _calculator = ((DateTime date) => date.add(const Duration(days: 30)));
 
   final void Function(DateTime, DateTime) onDateTimeRangeUpdated;
-  final DateTime startDate;
-  final DateTime endDate;
+  final DateTime? startDate;
+  final DateTime? endDate;
   final String label;
   final Calculator _calculator;
 
@@ -77,8 +77,9 @@ class DateTimeShortcutChip extends StatelessWidget {
   Widget build(BuildContext context) {
     return CustomChip(
       onSelected: (bool value) {
-        final DateTime updatedStartDate = _calculator(startDate);
-        final DateTime updatedEndDate = _calculator(endDate);
+        if (startDate == null || endDate == null) return;
+        final DateTime updatedStartDate = _calculator(startDate!);
+        final DateTime updatedEndDate = _calculator(endDate!);
         onDateTimeRangeUpdated(updatedStartDate, updatedEndDate);
       },
       selected: false,

lib/screens/widgets/comment_tile.dart

@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:flutter_slidable/flutter_slidable.dart';
 import 'package:hacki/blocs/blocs.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/cubits/cubits.dart';
 import 'package:hacki/extensions/extensions.dart';
 import 'package:hacki/models/models.dart';
@@ -9,6 +10,7 @@ import 'package:hacki/screens/widgets/widgets.dart';
 import 'package:hacki/services/services.dart';
 import 'package:hacki/styles/styles.dart';
 import 'package:hacki/utils/utils.dart';
+import 'package:scrollable_positioned_list/scrollable_positioned_list.dart';
 
 class CommentTile extends StatelessWidget {
   const CommentTile({
@@ -23,6 +25,7 @@ class CommentTile extends StatelessWidget {
     this.actionable = true,
     this.level = 0,
     this.onTap,
+    this.itemScrollController,
   });
 
   final String? opUsername;
@@ -30,6 +33,7 @@ class CommentTile extends StatelessWidget {
   final int level;
   final bool actionable;
   final FetchMode fetchMode;
+  final ItemScrollController? itemScrollController;
 
   final void Function(Comment)? onReplyTapped;
   final void Function(Comment, Rect?)? onMoreTapped;
@@ -116,8 +120,7 @@ class CommentTile extends StatelessWidget {
                   child: InkWell(
                     onTap: () {
                       if (actionable) {
-                        HapticFeedbackUtil.selection();
-                        context.read<CollapseCubit>().collapse();
+                        _collapse(context);
                       } else {
                         onTap?.call();
                       }
@@ -159,7 +162,7 @@ class CommentTile extends StatelessWidget {
                           ),
                         ),
                         AnimatedSize(
-                          duration: const Duration(milliseconds: 200),
+                          duration: Durations.ms200,
                           child: Column(
                             crossAxisAlignment: CrossAxisAlignment.start,
                             children: <Widget>[
@@ -339,8 +342,27 @@ class CommentTile extends StatelessWidget {
 
   void _onTextTapped(BuildContext context) {
     if (context.read<PreferenceCubit>().state.tapAnywhereToCollapseEnabled) {
-      HapticFeedbackUtil.selection();
-      context.read<CollapseCubit>().collapse();
+      _collapse(context);
+    }
+  }
+
+  void _collapse(BuildContext context) {
+    HapticFeedbackUtil.selection();
+    context.read<CollapseCubit>().collapse();
+    if (context.read<CollapseCubit>().state.collapsed &&
+        context.read<PreferenceCubit>().state.autoScrollEnabled) {
+      Future<void>.delayed(
+        Durations.ms300,
+        () {
+          itemScrollController?.scrollTo(
+            index:
+                context.read<CommentsCubit>().state.comments.indexOf(comment) +
+                    1,
+            alignment: 0.1,
+            duration: Durations.ms300,
+          );
+        },
+      );
     }
   }
 }

lib/screens/widgets/custom_tab_bar.dart

@@ -66,7 +66,7 @@ class _CustomTabBarState extends State<CustomTabBar> {
                         currentIndex == i ? TextDimens.pt14 : TextDimens.pt10,
                     color: currentIndex == i ? Palette.orange : Palette.grey,
                   ),
-                  duration: const Duration(milliseconds: 200),
+                  duration: Durations.ms200,
                   child: Text(
                     state.tabs.elementAt(i).label,
                     key: ValueKey<String>(

lib/screens/widgets/device_gesture_wrapper.dart

@@ -17,7 +17,7 @@ class DeviceGestureWrapper extends StatelessWidget {
   Widget build(BuildContext context) {
     return MediaQuery(
       data: const MediaQueryData(
-        gestureSettings: DeviceGestureSettings(touchSlop: 7.9),
+        gestureSettings: DeviceGestureSettings(touchSlop: 12),
       ),
       child: child,
     );

lib/screens/widgets/link_preview/link_preview.dart

@@ -233,7 +233,7 @@ class _LinkPreviewState extends State<LinkPreview> {
       secondChild: loadedWidget,
       crossFadeState:
           _loading ? CrossFadeState.showFirst : CrossFadeState.showSecond,
-      duration: const Duration(milliseconds: 500),
+      duration: Durations.ms500,
     );
   }
 }

lib/screens/widgets/link_preview/link_view.dart

@@ -237,7 +237,7 @@ class LinkView extends StatelessWidget {
                   crossAxisAlignment: CrossAxisAlignment.start,
                   children: <Widget>[
                     SizedBox(
-                      height: isUsingSerifFont! ? Dimens.pt2 : Dimens.pt4,
+                      height: isUsingSerifFont! ? Dimens.zero : Dimens.pt4,
                     ),
                     Text(
                       title,

lib/screens/widgets/onboarding_view.dart

@@ -15,7 +15,7 @@ class _OnboardingViewState extends State<OnboardingView> {
   final PageController pageController = PageController();
   final Throttle throttle = Throttle(delay: _throttleDelay);
 
-  static const Duration _throttleDelay = Duration(milliseconds: 100);
+  static const Duration _throttleDelay = Durations.ms100;
   static const double _screenshotHeight = 550;
 
   @override
@@ -80,7 +80,7 @@ class _OnboardingViewState extends State<OnboardingView> {
                 } else {
                   throttle.run(() {
                     pageController.nextPage(
-                      duration: const Duration(milliseconds: 600),
+                      duration: Durations.ms600,
                       curve: SpringCurve.underDamped,
                     );
                   });

lib/screens/widgets/tap_down_wrapper.dart

@@ -1,4 +1,5 @@
 import 'package:flutter/material.dart';
+import 'package:hacki/config/constants.dart';
 
 class TapDownWrapper extends StatefulWidget {
   const TapDownWrapper({
@@ -22,7 +23,7 @@ class _TapDownWrapperState extends State<TapDownWrapper>
   @override
   void initState() {
     controller = AnimationController(
-      duration: const Duration(milliseconds: 100),
+      duration: Durations.ms100,
       vsync: this,
     );
 

lib/services/web_analyzer.dart

@@ -5,6 +5,7 @@ import 'dart:io';
 import 'package:collection/collection.dart' show IterableExtension;
 import 'package:fast_gbk/fast_gbk.dart';
 import 'package:flutter/foundation.dart';
+import 'package:hacki/config/constants.dart';
 import 'package:hacki/config/locator.dart';
 import 'package:hacki/extensions/extensions.dart';
 import 'package:hacki/models/models.dart';
@@ -321,7 +322,7 @@ class WebAnalyzer {
     final Uri uri = Uri.parse(url);
     final HttpClient ioClient = HttpClient()
       ..badCertificateCallback = _certificateCheck
-      ..connectionTimeout = const Duration(seconds: 2);
+      ..connectionTimeout = Durations.twoSeconds;
     final IOClient client = IOClient(ioClient);
     final BaseRequest request = Request('GET', uri)
       ..followRedirects = true
@@ -337,7 +338,7 @@ class WebAnalyzer {
 
     try {
       final IOStreamedResponse stream =
-          await client.send(request).timeout(const Duration(seconds: 10));
+          await client.send(request).timeout(Durations.tenSeconds);
 
       if (stream.statusCode == HttpStatus.movedTemporarily ||
           stream.statusCode == HttpStatus.movedPermanently) {

lib/utils/theme_util.dart

@@ -1,61 +1,92 @@
 import 'dart:io';
 
 import 'package:adaptive_theme/adaptive_theme.dart';
-import 'package:device_info_plus/device_info_plus.dart';
 import 'package:flutter/services.dart';
 import 'package:hacki/styles/styles.dart';
 
 abstract class ThemeUtil {
-  /// Temp fix for the issue:
-  /// https://github.com/flutter/flutter/issues/119465
-  static Future<void> updateAndroidStatusBarSetting(
+  static Future<void> updateStatusBarSetting(
     Brightness brightness,
     AdaptiveThemeMode? mode,
   ) async {
-    if (Platform.isAndroid == false) return;
-
-    final DeviceInfoPlugin deviceInfoPlugin = DeviceInfoPlugin();
-    final AndroidDeviceInfo androidInfo = await deviceInfoPlugin.androidInfo;
-    final int sdk = androidInfo.version.sdkInt;
-
-    if (sdk > 28) return;
-    switch (mode) {
-      case AdaptiveThemeMode.light:
-        SystemChrome.setSystemUIOverlayStyle(
-          const SystemUiOverlayStyle(
-            statusBarBrightness: Brightness.dark,
-            statusBarIconBrightness: Brightness.dark,
-            statusBarColor: Palette.transparent,
-          ),
-        );
-      case AdaptiveThemeMode.dark:
-        SystemChrome.setSystemUIOverlayStyle(
-          const SystemUiOverlayStyle(
-            statusBarBrightness: Brightness.light,
-            statusBarIconBrightness: Brightness.light,
-            statusBarColor: Palette.transparent,
-          ),
-        );
-      case AdaptiveThemeMode.system:
-      case null:
-        switch (brightness) {
-          case Brightness.light:
-            SystemChrome.setSystemUIOverlayStyle(
-              const SystemUiOverlayStyle(
-                statusBarBrightness: Brightness.dark,
-                statusBarIconBrightness: Brightness.dark,
-                statusBarColor: Palette.transparent,
-              ),
-            );
-          case Brightness.dark:
-            SystemChrome.setSystemUIOverlayStyle(
-              const SystemUiOverlayStyle(
-                statusBarBrightness: Brightness.light,
-                statusBarIconBrightness: Brightness.light,
-                statusBarColor: Palette.transparent,
-              ),
-            );
-        }
+    if (Platform.isAndroid) {
+      switch (mode) {
+        case AdaptiveThemeMode.light:
+          SystemChrome.setSystemUIOverlayStyle(
+            const SystemUiOverlayStyle(
+              statusBarBrightness: Brightness.dark,
+              statusBarIconBrightness: Brightness.dark,
+              statusBarColor: Palette.transparent,
+            ),
+          );
+        case AdaptiveThemeMode.dark:
+          SystemChrome.setSystemUIOverlayStyle(
+            const SystemUiOverlayStyle(
+              statusBarBrightness: Brightness.light,
+              statusBarIconBrightness: Brightness.light,
+              statusBarColor: Palette.transparent,
+            ),
+          );
+        case AdaptiveThemeMode.system:
+        case null:
+          switch (brightness) {
+            case Brightness.light:
+              SystemChrome.setSystemUIOverlayStyle(
+                const SystemUiOverlayStyle(
+                  statusBarBrightness: Brightness.dark,
+                  statusBarIconBrightness: Brightness.dark,
+                  statusBarColor: Palette.transparent,
+                ),
+              );
+            case Brightness.dark:
+              SystemChrome.setSystemUIOverlayStyle(
+                const SystemUiOverlayStyle(
+                  statusBarBrightness: Brightness.light,
+                  statusBarIconBrightness: Brightness.light,
+                  statusBarColor: Palette.transparent,
+                ),
+              );
+          }
+      }
+    } else {
+      switch (mode) {
+        case AdaptiveThemeMode.light:
+          SystemChrome.setSystemUIOverlayStyle(
+            const SystemUiOverlayStyle(
+              statusBarBrightness: Brightness.light,
+              statusBarIconBrightness: Brightness.light,
+              statusBarColor: Palette.transparent,
+            ),
+          );
+        case AdaptiveThemeMode.dark:
+          SystemChrome.setSystemUIOverlayStyle(
+            const SystemUiOverlayStyle(
+              statusBarBrightness: Brightness.dark,
+              statusBarIconBrightness: Brightness.dark,
+              statusBarColor: Palette.transparent,
+            ),
+          );
+        case AdaptiveThemeMode.system:
+        case null:
+          switch (brightness) {
+            case Brightness.light:
+              SystemChrome.setSystemUIOverlayStyle(
+                const SystemUiOverlayStyle(
+                  statusBarBrightness: Brightness.light,
+                  statusBarIconBrightness: Brightness.light,
+                  statusBarColor: Palette.transparent,
+                ),
+              );
+            case Brightness.dark:
+              SystemChrome.setSystemUIOverlayStyle(
+                const SystemUiOverlayStyle(
+                  statusBarBrightness: Brightness.dark,
+                  statusBarIconBrightness: Brightness.dark,
+                  statusBarColor: Palette.transparent,
+                ),
+              );
+          }
+      }
     }
   }
 }

pubspec.lock

@@ -1390,4 +1390,4 @@ packages:
     version: "3.1.2"
 sdks:
   dart: ">=3.0.0 <4.0.0"
-  flutter: ">=3.10.2"
+  flutter: ">=3.10.3"

pubspec.yaml

@@ -1,11 +1,11 @@
 name: hacki
 description: A Hacker News reader.
-version: 1.7.0+111
+version: 1.7.4+115
 publish_to: none
 
 environment:
   sdk: ">=3.0.0 <4.0.0"
-  flutter: "3.10.2"
+  flutter: "3.10.3"
 
 dependencies:
   adaptive_theme: ^3.2.0