improvements

lib/screens/timeline_screen.dart

@@ -1,13 +1,125 @@
 import 'package:flutter/material.dart';
-import 'package:share_location/widgets/timeline_scroll.dart';
+import 'package:provider/provider.dart';
+import 'package:share_location/extensions/date.dart';
+import 'package:share_location/models/timeline.dart';
+import 'package:share_location/utils/loadable.dart';
+import 'package:share_location/widgets/timeline_page.dart';
+import 'package:supabase_flutter/supabase_flutter.dart';
 
-class TimelineScreen extends StatelessWidget {
+final supabase = Supabase.instance.client;
+
+class TimelineScreen extends StatefulWidget {
   static const ID = 'timeline';
 
-  const TimelineScreen({Key? key}) : super(key: key);
+  final DateTime? date;
+
+  const TimelineScreen({
+    Key? key,
+    this.date,
+  }) : super(key: key);
+
+  @override
+  State<TimelineScreen> createState() => _TimelineScreenState();
+}
+
+class _TimelineScreenState extends State<TimelineScreen> with Loadable {
+  final pageController = PageController();
+  final timeline = TimelineModel();
+  bool _ignorePageChanges = false;
+
+  Future<void> _goToPage(final int page) async {
+    _ignorePageChanges = true;
+
+    await pageController.animateToPage(
+      page,
+      duration: const Duration(milliseconds: 300),
+      curve: Curves.easeOutQuad,
+    );
+
+    _ignorePageChanges = false;
+  }
+
+  @override
+  initState() {
+    super.initState();
+
+    timeline.initialize();
+
+    // Update page view
+    timeline.addListener(() async {
+      if (timeline.currentIndex != pageController.page) {
+        _goToPage(timeline.currentIndex);
+      }
+    }, ['currentIndex']);
+
+    // Update page when initializing is done
+    timeline.addListener(() {
+      if (!mounted) {
+        return;
+      }
+
+      setState(() {});
+
+      WidgetsBinding.instance.addPostFrameCallback((_) async {
+        final initialIndex = getIndexFromDate();
+
+        await _goToPage(initialIndex);
+
+        timeline.setCurrentIndex(initialIndex);
+      });
+    }, ['isInitializing']);
+  }
+
+  @override
+  dispose() {
+    pageController.dispose();
+
+    super.dispose();
+  }
+
+  int getIndexFromDate() {
+    if (widget.date == null) {
+      return 0;
+    }
+
+    return timeline.values.keys
+        .toList()
+        .indexWhere((date) => DateTime.parse(date).isSameDay(widget.date!));
+  }
 
   @override
   Widget build(BuildContext context) {
-    return TimelineScroll();
+    if (timeline.isInitializing) {
+      return const Center(
+        child: CircularProgressIndicator(),
+      );
+    }
+
+    return Scaffold(
+      body: ChangeNotifierProvider.value(
+        value: timeline,
+        child: PageView.builder(
+          controller: pageController,
+          scrollDirection: Axis.vertical,
+          itemCount: timeline.values.length,
+          onPageChanged: (newPage) {
+            if (_ignorePageChanges) {
+              return;
+            }
+
+            if (timeline.currentIndex != newPage) {
+              // User manually changed page
+              timeline.setCurrentIndex(newPage);
+
+              timeline.setMemoryIndex(0);
+            }
+          },
+          itemBuilder: (_, index) => TimelinePage(
+            date: timeline.dateAtIndex(index),
+            memoryPack: timeline.atIndex(index),
+          ),
+        ),
+      ),
+    );
   }
 }

lib/widgets/calendar_month.dart

@@ -4,12 +4,12 @@ import 'package:flutter/material.dart';
 import 'package:flutter_calendar_widget/flutter_calendar_widget.dart';
 import 'package:intl/intl.dart';
 import 'package:share_location/constants/spacing.dart';
-import 'package:share_location/widgets/timeline_scroll.dart';
+import 'package:share_location/screens/timeline_screen.dart';
 
 class MonthCalendarBuilder extends CalendarBuilder {
   @override
   Widget buildDayOfWeek(DateTime dateTime, String weekdayString) {
-    return SizedBox.shrink();
+    return const SizedBox.shrink();
   }
 
   @override
@@ -125,8 +125,9 @@ class CalendarMonth extends StatelessWidget {
         Navigator.push(
           context,
           MaterialPageRoute(
-              builder: (context) => TimelineScroll(date: date),
+            builder: (context) => TimelineScreen(date: date),
-            ));
+          ),
+        );
       },
       style: const CalendarStyle(
         calenderMargin: EdgeInsets.symmetric(vertical: MEDIUM_SPACE),

lib/widgets/timeline_scroll.dart

@@ -1,123 +0,0 @@
-import 'package:flutter/material.dart';
-import 'package:provider/provider.dart';
-import 'package:share_location/extensions/date.dart';
-import 'package:share_location/models/timeline.dart';
-import 'package:share_location/utils/loadable.dart';
-import 'package:share_location/widgets/timeline_page.dart';
-import 'package:supabase_flutter/supabase_flutter.dart';
-
-final supabase = Supabase.instance.client;
-
-class TimelineScroll extends StatefulWidget {
-  final DateTime? date;
-
-  TimelineScroll({
-    Key? key,
-    this.date,
-  }) : super(key: key);
-
-  @override
-  State<TimelineScroll> createState() => _TimelineScrollState();
-}
-
-class _TimelineScrollState extends State<TimelineScroll> with Loadable {
-  final pageController = PageController();
-  final timeline = TimelineModel();
-  bool _ignorePageChanges = false;
-
-  Future<void> _goToPage(final int page) async {
-    _ignorePageChanges = true;
-
-    await pageController.animateToPage(
-      page,
-      duration: const Duration(milliseconds: 300),
-      curve: Curves.easeOutQuad,
-    );
-
-    _ignorePageChanges = false;
-  }
-
-  @override
-  initState() {
-    super.initState();
-
-    timeline.initialize();
-
-    // Update page view
-    timeline.addListener(() async {
-      if (timeline.currentIndex != pageController.page) {
-        _goToPage(timeline.currentIndex);
-      }
-    }, ['currentIndex']);
-
-    // Update page when initializing is done
-    timeline.addListener(() {
-      if (!mounted) {
-        return;
-      }
-
-      setState(() {});
-
-      WidgetsBinding.instance.addPostFrameCallback((_) async {
-        final initialIndex = getIndexFromDate();
-
-        await _goToPage(initialIndex);
-
-        timeline.setCurrentIndex(initialIndex);
-      });
-    }, ['isInitializing']);
-  }
-
-  @override
-  dispose() {
-    pageController.dispose();
-
-    super.dispose();
-  }
-
-  int getIndexFromDate() {
-    if (widget.date == null) {
-      return 0;
-    }
-
-    return timeline.values.keys
-        .toList()
-        .indexWhere((date) => DateTime.parse(date).isSameDay(widget.date!));
-  }
-
-  @override
-  Widget build(BuildContext context) {
-    if (timeline.isInitializing) {
-      return const Center(
-        child: CircularProgressIndicator(),
-      );
-    }
-
-    return Scaffold(
-      body: ChangeNotifierProvider.value(
-        value: timeline,
-        child: PageView.builder(
-          controller: pageController,
-          scrollDirection: Axis.vertical,
-          itemCount: timeline.values.length,
-          onPageChanged: (newPage) {
-            if (_ignorePageChanges) {
-              return;
-            }
-
-            if (timeline.currentIndex != newPage) {
-              // User manually changed page
-              timeline.setCurrentIndex(newPage);
-
-              timeline.setMemoryIndex(0);
-            }
-          },
-          itemBuilder: (_, index) => TimelinePage(
-            date: timeline.dateAtIndex(index),
-            memoryPack: timeline.atIndex(index),
-          ),
-        ),
-      ),
-    );
-  }
-}