crashlytics: log locales, timezone, route changes

lib/main.dart

@@ -1,6 +1,8 @@
 import 'dart:isolate';
+import 'dart:ui';
 
 import 'package:aves/model/settings/settings.dart';
+import 'package:aves/utils/route_tracker.dart';
 import 'package:aves/widgets/common/data_providers/settings_provider.dart';
 import 'package:aves/widgets/common/icons.dart';
 import 'package:aves/widgets/home_page.dart';
@@ -38,38 +40,11 @@ class AvesApp extends StatefulWidget {
 
 class _AvesAppState extends State<AvesApp> {
   Future<void> _appSetup;
+  final NavigatorObserver _routeTracker = CrashlyticsRouteTracker();
 
   static const accentColor = Colors.indigoAccent;
 
-  @override
+  static final darkTheme = ThemeData(
-  void initState() {
-    super.initState();
-    _appSetup = _setup();
-  }
-
-  Future<void> _setup() async {
-    await Firebase.initializeApp().then((app) {
-      FlutterError.onError = FirebaseCrashlytics.instance.recordFlutterError;
-      FirebaseCrashlytics.instance.setCustomKey(
-          'build_mode',
-          kReleaseMode
-              ? 'release'
-              : kProfileMode
-                  ? 'profile'
-                  : 'debug');
-    });
-    await settings.init();
-  }
-
-  @override
-  Widget build(BuildContext context) {
-    // place the settings provider above `MaterialApp`
-    // so it can be used during navigation transitions
-    return SettingsProvider(
-      child: OverlaySupport(
-        child: MaterialApp(
-          title: 'Aves',
-          theme: ThemeData(
     brightness: Brightness.dark,
     accentColor: accentColor,
     scaffoldBackgroundColor: Colors.grey[900],
@@ -87,22 +62,74 @@ class _AvesAppState extends State<AvesApp> {
         ),
       ),
     ),
-          ),
+  );
-          home: FutureBuilder<void>(
+
+  @override
+  void initState() {
+    super.initState();
+    _appSetup = _setup();
+  }
+
+  Future<void> _setup() async {
+    await Firebase.initializeApp().then((app) {
+      FlutterError.onError = FirebaseCrashlytics.instance.recordFlutterError;
+      FirebaseCrashlytics.instance.setCustomKey('locales', window.locales.join(', '));
+      final now = DateTime.now();
+      FirebaseCrashlytics.instance.setCustomKey('timezone', '${now.timeZoneName} (${now.timeZoneOffset})');
+      FirebaseCrashlytics.instance.setCustomKey(
+          'build_mode',
+          kReleaseMode
+              ? 'release'
+              : kProfileMode
+                  ? 'profile'
+                  : 'debug');
+    });
+    await settings.init();
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    // place the settings provider above `MaterialApp`
+    // so it can be used during navigation transitions
+    final home = FutureBuilder<void>(
       future: _appSetup,
       builder: (context, snapshot) {
-              if (snapshot.hasError) {
+        if (!snapshot.hasError && snapshot.connectionState == ConnectionState.done) {
-                return Column(
+          return settings.hasAcceptedTerms ? HomePage() : WelcomePage();
+        }
+        return Scaffold(
+          body: snapshot.hasError
+              ? Container(
+                  alignment: Alignment.center,
+                  padding: EdgeInsets.all(16),
+                  child: Column(
+                    mainAxisSize: MainAxisSize.min,
                     children: [
                       Icon(AIcons.error),
-                    Text(snapshot.error),
+                      SizedBox(height: 16),
+                      Text(snapshot.error.toString()),
                     ],
-                );
-              }
-              if (snapshot.connectionState != ConnectionState.done) return Scaffold();
-              return settings.hasAcceptedTerms ? HomePage() : WelcomePage();
-            },
                   ),
+                )
+              : SizedBox.shrink(),
+        );
+      },
+    );
+    return SettingsProvider(
+      child: OverlaySupport(
+        child: FutureBuilder<void>(
+          future: _appSetup,
+          builder: (context, snapshot) {
+            return MaterialApp(
+              home: home,
+              navigatorObservers: [
+                if (!snapshot.hasError && snapshot.connectionState == ConnectionState.done) _routeTracker,
+              ],
+              title: 'Aves',
+              darkTheme: darkTheme,
+              themeMode: ThemeMode.dark,
+            );
+          },
         ),
       ),
     );

lib/utils/route_tracker.dart

@@ -0,0 +1,18 @@
+import 'package:firebase_crashlytics/firebase_crashlytics.dart';
+import 'package:flutter/material.dart';
+
+class CrashlyticsRouteTracker extends NavigatorObserver {
+  @override
+  void didPush(Route<dynamic> route, Route<dynamic> previousRoute) => FirebaseCrashlytics.instance.log('Nav didPush to ${_name(route)}');
+
+  @override
+  void didPop(Route<dynamic> route, Route<dynamic> previousRoute) => FirebaseCrashlytics.instance.log('Nav didPop to ${_name(previousRoute)}');
+
+  @override
+  void didRemove(Route<dynamic> route, Route<dynamic> previousRoute) => FirebaseCrashlytics.instance.log('Nav didRemove to ${_name(previousRoute)}');
+
+  @override
+  void didReplace({Route<dynamic> newRoute, Route<dynamic> oldRoute}) => FirebaseCrashlytics.instance.log('Nav didReplace to ${_name(newRoute)}');
+
+  String _name(Route<dynamic> route) => route?.settings?.name ?? 'unnamed ${route?.runtimeType}';
+}